Closed
Bug 518502
Opened 15 years ago
Closed 13 years ago
Replace PR_MIN/PR_MAX with NS_MIN/NS_MAX, in mozilla-central
Categories
(Core :: General, defect)
Core
General
Tracking
()
VERIFIED
FIXED
People
(Reporter: sgautherie, Assigned: lusian)
References
(Blocks 1 open bug, )
Details
(Keywords: meta)
Attachments
(1 obsolete file)
+++ This bug was initially created as a clone of Bug #512106 +++ See that bug about why and how to do it...
Reporter | ||
Updated•15 years ago
|
Whiteboard: [good first bug] [first, file blocking bugs]
Assignee | ||
Comment 1•15 years ago
|
||
Automatically replaced PR_MIN/PR_MAX, and then had to add changes manually (#include "nsAlgorithm.h", type conversions). I could build Firefox on Windows XP. The patch needs to be tested on the Try Server.
Reporter | ||
Updated•15 years ago
|
Assignee: nobody → lusian
Status: NEW → ASSIGNED
Keywords: checkin-needed
Whiteboard: [good first bug] [first, file blocking bugs] → [c-n: to try] [first, file blocking bugs] [good first bug]
Comment 2•15 years ago
|
||
This patch doesn't apply anymore: patching file xpcom/string/src/nsTSubstring.cpp Hunk #1 succeeded at 475 (offset 1 line). Hunk #3 succeeded at 526 (offset 1 line). Hunk #4 FAILED at 544. http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTry/1258635928.1258636234.11984.gz
Keywords: checkin-needed
Whiteboard: [c-n: to try] [first, file blocking bugs] [good first bug] → [first, file blocking bugs] [good first bug]
Assignee | ||
Updated•15 years ago
|
Attachment #402894 -
Attachment is obsolete: true
Reporter | ||
Comment 4•13 years ago
|
||
To be explicit, what about (other) remaining cases? (Mostly in (NSPR and) NSS.) "Found 38 matching lines in 11 files "
Comment 5•13 years ago
|
||
The others are in NSS/NSPR (which I was under the impression can't use a core specific include, due to third party users of them) & non C++ code. As such, I don't believe any of the others can be changed - though if that is not the case, I'm happy to try and implement whatever you suggest.
Reporter | ||
Comment 6•13 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #5) > The others are in NSS/NSPR (which I was under the impression can't use a > core specific include, due to third party users of them) & non C++ code. As > such, I don't believe any of the others can be changed - though if that is > not the case, I'm happy to try and implement whatever you suggest. Right: "Found 39 matching lines in 12 files" /modules/libreg/src/nr_bufio.c /ipc/chromium/src/base/third_party/nspr/prtypes.h 2 * /nsprpub/pr/... /docshell/shistory/src/nsSHistory.cpp : I filed bug 707395. 7 * /security/nss/ V.Fixed
Status: RESOLVED → VERIFIED
Whiteboard: [first, file blocking bugs] [good first bug]
Reporter | ||
Comment 7•13 years ago
|
||
(In reply to Serge Gautherie (:sgautherie) from comment #6) > V.Fixed Arg, I probably checked PR_MIN only :-/ I filed 2 new blocking bugs!
You need to log in
before you can comment on or make changes to this bug.
Description
•