Closed Bug 624442 Opened 9 years ago Closed 8 years ago

vprof ifndef-guard MIN/MAX

Categories

(Tamarin Graveyard :: Tools, defect, trivial)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: pnkfelix, Unassigned)

Details

(Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin)

vprof.cpp has macro definitions for MIN/MAX that do what you think.

But on my system, if you #include sysctl, it pulls in its own (compatible) definitions for MIN/MAX.  Not a problem, but you do get warnings from gcc.

To avoid the warnings, I'm adding #ifndef guards; there are already a bunch of similar guards strewn throughout the file.
changeset: 5754:c876056685b8
user:      Felix S Klock II <fklockii@adobe.com>
summary:   Bug 624442: ifndef-guard MIN/MAX macro definitions in vprof.cpp (r=fklockii)

http://hg.mozilla.org/tamarin-redux/rev/c876056685b8
(In reply to comment #0)
> But on my system, if you #include sysctl, it pulls in its own (compatible)
> definitions for MIN/MAX.  Not a problem, but you do get warnings from gcc.

Note that the aforementioned #include <sys/sysctl.h> is added in the patch for Bug 611232
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reopening only to track the change through nanojit-central back into TR/TM.

NJ: http://hg.mozilla.org/projects/nanojit-central/rev/17d3562dd533
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: fixed-in-nanojit
changeset: 6024:3561f821fde3
user:      Edwin Smith <edwsmith@adobe.com>
summary:   Bug 624442: ifndef-guard MIN/MAX macro definitions in vprof.cpp (r=fklockii)

http://hg.mozilla.org/tamarin-redux/rev/3561f821fde3
http://hg.mozilla.org/tracemonkey/rev/b9c97e99db42

Is Tamarin-bot going to keep closing these bugs as soon as it's merged them to TR?  That'll be annoying if so.
Whiteboard: fixed-in-nanojit → fixed-in-nanojit, fixed-in-tracemonkey
The tamarin-bot does not modify any status of the bug, all that it is doing is commenting to the bug that a submission was made to the tamarin-redux repo with this bug number referenced.
In the case of this bug, Felix closed it after fixing in TR; Felix wasn't aware vprof came from nanojit-central.  Many weeks later, I did an import from NC to TR, clobbering the changes.  I made the fix in NC then reopened this bug for tracking.  

Then I forgot to mark it fixed-in-tamarin after re-importing NC to TR... Just did it.

We still manually merge NC->TR, although Rick is working on some automation that automatically picks up NC changes, imports to TR, then sandbox-tests, to take grunt work out of the picture.  Details available if interested.
Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey → fixed-in-nanojit, fixed-in-tracemonkey, fixed-in-tamarin
Marking resolved/closed as it is fixed in TM, NJ and TR
Status: REOPENED → RESOLVED
Closed: 9 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.