Closed Bug 602044 Opened 9 years ago Closed 8 years ago

compile warnings with harfbuzz

Categories

(Core :: Graphics, defect)

x86
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: vlad, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

(Whiteboard: [build_warning])

Attachments

(1 obsolete file)

building with vs2010:

c:\proj\firefox\dist\include\harfbuzz\hb-common.h(34) : Warning Msg: Not using stdint.h; integer types may have wrong size
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(216) : warning C4018: '>=' : signed/unsigned mismatch
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(227) : warning C4244: '=' : conversion from 'float' to 'hb_position_t', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(633) : warning C4244: 'return' : conversion from 'float' to 'hb_position_t', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(798) : warning C4244: 'argument' : conversion from 'gfxFloat' to 'unsigned
int', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(798) : warning C4244: 'argument' : conversion from 'gfxFloat' to 'unsigned
int', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(799) : warning C4244: 'initializing' : conversion from 'gfxFloat' to 'PRUint32', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(900) : warning C4018: '<' : signed/unsigned mismatch
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(909) : warning C4244: 'initializing' : conversion from 'double' to 'float', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(983) : warning C4018: '<' : signed/unsigned mismatch
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(990) : warning C4018: '>=' : signed/unsigned mismatch
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(996) : warning C4018: '<' : signed/unsigned mismatch
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(1012) : warning C4244: 'initializing' : conversion from 'float' to 'nscoord', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(1039) : warning C4244: '-=' : conversion from 'float' to 'nscoord', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(1044) : warning C4244: '=' : conversion from 'float' to 'nscoord', possible loss of data
c:/proj/firefox/gfx/thebes/../../../mozilla-central/gfx/thebes/gfxHarfBuzzShaper.cpp(1058) : warning C4018: '<' : signed/unsigned mismatch
This should fix the warnings, except for the one about "hb-common.h(34) : Warning Msg: Not using stdint.h; integer types may have wrong size." This is an intentional message to warn us that the code is using a collection of hard-coded typedefs for [u]int*_t types, as the standard <stdint.h> header isn't available in MSVC. (It is explicitly printed using #pragma message rather than being an actual compiler warning.)
Assignee: nobody → jfkthame
Attachment #481233 - Flags: review?(vladimir)
Comment on attachment 481233 [details] [diff] [review]
patch, v1 - add typecasts etc to avoid compiler warnings

Great, thanks!  thebes isn't exactly warning-free, but we should at least head in the right direction :)
Attachment #481233 - Flags: review?(vladimir) → review+
Whiteboard: [build_warning]
Happy for me to land this? (after unrotting)
Hmm.... judging by the current code, it looks like the warnings here should have been fixed already through other changes - I believe bug 605043 dealt with most of them, at least.

Are there still build warnings being generated in this code (besides the known message mentioned in comment #1)?
Comment on attachment 481233 [details] [diff] [review]
patch, v1 - add typecasts etc to avoid compiler warnings

This patch is obsolete, as bug 605043 fixed (at least most of) these issues; if anything remains, we should create a new patch based on current code.
Attachment #481233 - Attachment is obsolete: true
Closing command per Comment 5, re-open if there are still warnings we want to take a patch for.
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.