Closed
Bug 530361
Opened 13 years ago
Closed 11 years ago
Replace PR_MIN/PR_MAX with NS_MIN/NS_MAX in /intl
Categories
(Core :: Internationalization, defect)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
People
(Reporter: lusian, Assigned: lusian)
References
Details
Attachments
(2 files)
2.72 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
3.20 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091120 Minefield/3.7a1pre (.NET CLR 3.5.30729) Build Identifier: Please see Bug #512106. Reproducible: Always
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #413871 -
Flags: review?(smontagu)
Updated•13 years ago
|
Attachment #413871 -
Flags: review?(smontagu) → review+
Updated•13 years ago
|
Assignee: smontagu → lusian
Keywords: checkin-needed
Updated•13 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 2•13 years ago
|
||
From an OS X build: nsUCSupport.cpp /Users/philor/mc/mozilla/intl/uconv/util/nsUCSupport.cpp: In member function ‘void nsBufferDecoderSupport::FillBuffer(const char**, PRInt32)’: /Users/philor/mc/mozilla/intl/uconv/util/nsUCSupport.cpp:110: error: ‘NS_MIN’ was not declared in this scope
Keywords: checkin-needed
Assignee | ||
Comment 3•13 years ago
|
||
I have only Windows XP, so I could not test this on other platforms. I included "nsAlgorithm.h" in nsUCSupport.cpp. Will this fix the build error on OS X?
Comment 4•13 years ago
|
||
No, according to the tryserver, that breaks all three platforms - nsUnicodeToUTF8.cpp /builds/slave/sendchange-macosx-hg/build/intl/uconv/src/nsUTF8ToUnicode.cpp: In member function 'virtual nsresult nsUTF8ToUnicode::Convert(const char*, PRInt32*, PRUnichar*, PRInt32*)': /builds/slave/sendchange-macosx-hg/build/intl/uconv/src/nsUTF8ToUnicode.cpp:335: error: 'NS_MIN' was not declared in this scope nsUTF8ToUnicode.cpp /builds/slave/sendchange-linux-hg/build/intl/uconv/src/nsUTF8ToUnicode.cpp: In member function ‘virtual nsresult nsUTF8ToUnicode::Convert(const char*, PRInt32*, PRUnichar*, PRInt32*)’: /builds/slave/sendchange-linux-hg/build/intl/uconv/src/nsUTF8ToUnicode.cpp:335: error: ‘NS_MIN’ was not declared in this scope nsUTF8ToUnicode.cpp e:/builds/slave/win32-hg/build/intl/uconv/src/nsUTF8ToUnicode.cpp(335) : error C3861: 'NS_MIN': identifier not found
Assignee | ||
Comment 5•13 years ago
|
||
I updated my tree, did 'hg update -C tip' and applied the 2nd patch (Add #include); the build was successful on my computer (Windows XP), the tryserver says the otherwise, so I am not sure how to test the patches for bugs including this and others that fail on OS X. I think adding #include "nsAlgorithm.h" to files that fail will solve the problem, but is it a good solution?
Comment 6•11 years ago
|
||
Fixed by bug 661584.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•