Closed Bug 796164 Opened 12 years ago Closed 12 years ago

Change NECKO_UNKNOWN_APP_ID to UINT32_MAX

Categories

(Core :: Networking, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: jduell.mcbugs, Assigned: jduell.mcbugs)

References

Details

Bug 791906 changed it to #define NECKO_UNKNOWN_APP_ID std::numeric_limits<uint32_t>::max() But that is causing things like MOZ_ASSERT(appId != NECKO_UNKNOWN_APP_ID); to barf on windows with nsNetUtil.h(1362) : warning C4003: not enough actual parameters for macro 'max' nsNetUtil.h(1364) : warning C4003: not enough actual parameters for macro 'max' nsNetUtil.h(1364) : error C2589: '(' : illegal token on right side of '::' nsNetUtil.h(1364) : error C2059: syntax error : '::' nsNetUtil.h(1364) : error C2143: syntax error : missing ';' before '{'
OS: Linux → Windows 7
Bug 794023 did that.
Blocks: 794023
No longer depends on: 791906
Either (std::numeric_limits<uint32_t>::max)() or #define NOMINMAX will work.
:emk, I landed UINT32_MAX before I saw your comment (got +r from ehsan on IRC). If you feel some other fix is better, please open a bug for it. https://hg.mozilla.org/integration/mozilla-inbound/rev/064bf6c9fe16 This was blocking bug 786299 from landing (it still has other issues).
Blocks: 786299
Assignee: nobody → jduell.mcbugs
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.