Closed Bug 1119499 Opened 9 years ago Closed 9 years ago
Build warning in unified mode: 'BAD
_TLS _INDEX' macro redefined
I'm not sure if this is more fallout from my adding a new file to xpcom/base, but I saw this warning floating by. The other definition is in xpcpublic.h. I was going to take the hacky approach that xpcpublic.h takes and just add a #ifndef BAD_TLS_INDEX guard. Also, the definitions aren't exactly the same, so I'll take this chance to make them the same...
Well, XPConnect doesn't even use this, so maybe moving it into another file that the places that do use it is the better approach.
This was added to xpcpublic.h in bug 711240, when it was apparently "used all over Gecko" but now I just see it in two files, so I think we can just define it there.
This just moves the definition into the quota manager. nsTraceRefcnt can keep using its own slightly different definition.
Oops, that's a -1, so the spacing was correct.
Attachment #8546191 - Flags: review?(Jan.Varga)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.