Closed Bug 624442 Opened 9 years ago Closed 8 years ago
vprof ifndef-guard MIN/MAX
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 <firstname.lastname@example.org> 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 → ---
changeset: 6024:3561f821fde3 user: Edwin Smith <email@example.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 ago → 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.