Closed Bug 765038 Opened 8 years ago Closed 8 years ago

The fix for bug 761890 broke clang builds in gfx/skia/src/sfnt/SkOTTable_head.h

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: jwatt, Assigned: nrc)

References

Details

Attachments

(1 file)

gfx/skia/src/sfnt/SkOTTable_head.h:117:76: error: non-type template argument evaluates to -1, which cannot be narrowed to type 'uint16_t' (aka 'unsigned short')
      [-Wc++11-narrowing]
SkPathEffect.cpp
            ((OnlyStronglyRTL, static_cast<SK_OT_SHORT>(SkTEndian_SwapBE16(-1))))
gfx/skia/src/sfnt/SkOTTable_head.h:118:72: error: non-type template argument evaluates to -2, which cannot be narrowed to type 'uint16_t' (aka 'unsigned short')
      [-Wc++11-narrowing]
            ((StronglyRTL, static_cast<SK_OT_SHORT>(SkTEndian_SwapBE16(-2))))
Additional context (via IRC):
jwatt hit this while doing a local clobber build of mozilla-inbound, on Mac OS X, using clang.
Assignee: nobody → ncameron
Attached patch patchSplinter Review
builds with Visual Studio, hope this fixes the Clang issue
Attachment #633401 - Flags: review?(jwatt)
Disclaimer: I'm pretty sure this is innocuous, but I haven't tried any testing at all.
Comment on attachment 633401 [details] [diff] [review]
patch

Looks fine to me.

I filed an upstream bug on this too, noting your patch here:

http://code.google.com/p/skia/issues/detail?id=649
Attachment #633401 - Flags: review?(jwatt) → review+
Seems to work fine too.

I pushed this to m-i:

https://hg.mozilla.org/integration/mozilla-inbound/rev/3ec9d53ce65c

Thanks, nrc!
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/3ec9d53ce65c
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.