1215156 - move SetPixel* functions into nsBMPDecoder.cpp

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Nathan Froyd [:froydnj]]

These functions are only used in nsBMPDecoder.cpp; we don't need them
anywhere else.  The only other place they might get used would be the
BMP encoder, but the encoder appears to have its own routines for
setting pixel data, which don't overlap very well with the decoder's.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: move SetPixel* functions into nsBMPDecoder.cpp

Comment 2

[Seth Fowler [:seth] [:s2h]]

Comment on attachment 8674306 [details] [diff] [review]
move SetPixel* functions into nsBMPDecoder.cpp

Review of attachment 8674306 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks!

::: image/decoders/nsBMPDecoder.cpp
@@ +122,5 @@
>  
> +/// Sets the pixel data in aDecoded to the given values.
> +/// @param aDecoded pointer to pixel to be set, will be incremented to point to
> +/// the next pixel.
> +static inline void

As long as we're at it, let's get rid of |inline| here since it's useless.

@@ +129,5 @@
> +{
> +  *aDecoded++ = gfxPackedPixel(aAlpha, aRed, aGreen, aBlue);
> +}
> +
> +static inline void

Same here.

@@ +141,5 @@
> +/// @param aDecoded where the data is stored. Will be moved 4 resp 8 bytes
> +/// depending on whether one or two pixels are written.
> +/// @param aData The values for the two pixels
> +/// @param aCount Current count. Is decremented by one or two.
> +static inline void

And here.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ffef339aaa4f

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/ffef339aaa4f