openoffice-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Don Lewis <truck...@apache.org>
Subject what is this supposed to do?
Date Tue, 05 Nov 2019 18:54:17 GMT
I don't understand what this strange little bit of code (line 611 of
basebmp/inc/basebmp/packedpixeliterator.hxx) is supposed to do:

    value_type get(difference_type const & d) const
    {
        const int remainder( x(d.x) % num_intraword_positions );
                             ^^^^^^
        return (unsigned_cast<value_type>(*current(d.x,d.y) &
                                          get_mask<value_type, bits_per_pixel, MsbFirst>(remainder))
                >> get_shift<num_intraword_positions, bits_per_pixel, MsbFirst>(remainder));
    }

I've never seen any compiler complaints about it before, but gcc 9
throws an error:

../inc/basebmp/packedpixeliterator.hxx: In member function 'basebmp::PackedPixel
Iterator<Valuetype, bits_per_pixel, MsbFirst>::value_type basebmp::PackedPixelIt
erator<Valuetype, bits_per_pixel, MsbFirst>::get(const difference_type&) const':
../inc/basebmp/packedpixeliterator.hxx:611:35: error: expression cannot be used
as a function
  611 |         const int remainder( x(d.x) % num_intraword_positions );
      |                                   ^


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org


Mime
View raw message