Use -DUNICODE and -D_UNICODE everywhere

NEW
Unassigned

Status

()

defect
7 years ago
2 years ago

People

(Reporter: joe, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
We currently don't compile with -DUNICODE and -D_UNICODE everywhere, which can lead to problems when we compile one piece of code that exports a Windows type (like LOGFONT) with the define, and use it elsewhere without the define. This happens when the definition of code includes the chromium-config.mk file, but the user of that code doesn't.

We should just globally define -DUNICODE and -D_UNICODE everywhere on Windows.
We have to sanity check that we're not using non-W-prefixed APIs that would break with this. I *think* we're probably okay at this point, and any errors would just be compile errors.

Comment 2

7 years ago
wtf, I could have sworn we did this years ago.

Note that there are a few places which *remove* -DUNICODE from the build flags: gfx/gl/Makefile.in and gfx/thebes/Makefile.in
(Reporter)

Comment 3

7 years ago
It's entirely possible we do this everywhere except gfx/; I know for certain that including chromium-config.mk in gfx/2d caused types to change due to -DUNICODE, though.
You need to log in before you can comment on or make changes to this bug.