qcms.h defines QCMS_INTENT_ enums, and gfxPlatform.cpp defines INTENT_MIN to 0 and INTENT_MAX to 3, and defines its own INTENT_DEFAULT. These values should be based on what the qcms enums are, and should probably live where the enum is defined.
Trivial issue, but could be trouble if the enum changes and we don't know about the place in the source that "knows" what the enum range is. So, move it all in one place.
Attachment #716801 - Flags: review?(bgirard)
7 years ago
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.