If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Cannot compile the latest Nightly using MSVC 11

RESOLVED FIXED in mozilla18

Status

()

Core
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: emk, Assigned: emk)

Tracking

unspecified
mozilla18
x86_64
Windows 8
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
> Comment 21 Philip Chee 2012-09-29 01:09:53 PDT
>
>I'm hitting this:
>
>c:/t1/hg/comm-central/mozilla/netwerk/protocol/http/nsHttpChannel.cpp(4492) : >error C2782: 'const T &mozilla::clamped(const T &,const T &,const T &)' : >template parameter 'T' is ambiguous
>        c:\t1\hg\objdir-sm\mozilla\dist\include\nsAlgorithm.h(55) : see >declaration of 'mozilla::clamped'
>        could be 'int16_t'
(Assignee)

Comment 1

5 years ago
Created attachment 666193 [details] [diff] [review]
Add a template parameter to clamped
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Attachment #666193 - Flags: review?(ehsan)
Comment on attachment 666193 [details] [diff] [review]
Add a template parameter to clamped

Review of attachment 666193 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, if you want to take my review before Ehsan gets here :)
Attachment #666193 - Flags: review+
(Assignee)

Comment 3

5 years ago
I already got green on try along with other patches.
https://tbpl.mozilla.org/?tree=Try&rev=fd96b09d1bfa
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/42e5480fb109
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/42e5480fb109
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Attachment #666193 - Flags: review?(ehsan) → review+
You need to log in before you can comment on or make changes to this bug.