Created attachment 33803 [details]
GCC does not like multi-character character constants, so it produces the following warning for BMPImageDecoder.cpp:
../../WebCore/platform/image-decoders/bmp/BMPImageDecoder.cpp:128:16: warning: multi-character character constant
The attached patch fixes the warning, but since this might be a bit controversial *and* I can't run the tests to verify that this does not break anything (they all fail with 'timed out' now, something must be broken somewhere...), I'm opening a bug for this.
Why do you think this is controversial?
(In reply to comment #1)
> Why do you think this is controversial?
Well, at least for me it's not 100% obvious that this is correct, won't break anything, that the old code is not somehow relying on the use of multi-character character constants...
That being said the value of multi-character character constants is not guaranteed to be the same across platforms (according to Google), so I think a fix similar to this one (if not this one) should be applied.
I landed this as r46626, closing.
If you believe the bots this caused:
- A crash in GTK+ in test svg/dom/SVGScriptElement/script-load-and-error-events.svg (it works fine in my box).
- A layout failure in Qt for test animations/change-keyframes.html.
- A crash in Windows Debug for test http/tests/security/aboutBlank/xss-DENIED-navigate-opener-document-write.html (!?).
So I've rolled it out.
Comment on attachment 33803 [details]
rolled out, marking r-
Created attachment 34171 [details]
Fix two different warnings
The original patch was sound. I think the test failures encountered must have been due to flakiness or another change. Verified that this still decodes BMPs OK.
I also fixed another warning I just introduced (sorry!).
Created attachment 34172 [details]
Fix two different warnings (correct patch)
Oops, attached wrong patch. I have too many patches in flight :(
Fixed in r46814.