Closed Bug 31664 Opened 25 years ago Closed 25 years ago

Threadsafe assertions when running table regression tests

Categories

(Core :: Graphics: ImageLib, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: troy, Assigned: pnunn)

Details

I'm getting threadsafe assertions on the addref/release functions of NetReaderImpl (file ilNetReader.cpp in modules/libimg). The following patch eliminates the assertions. I can check it in for you if you want: Index: src/ilNetReader.cpp =================================================================== RCS file: /cvsroot/mozilla/modules/libimg/src/ilNetReader.cpp,v retrieving revision 3.10 diff -r3.10 ilNetReader.cpp 70c70 < NS_IMPL_ISUPPORTS(NetReaderImpl, kINetReaderIID) --- > NS_IMPL_THREADSAFE_ISUPPORTS(NetReaderImpl, kINetReaderIID)
Troy: thanks for the find. I'll check it in as soon as I get a good fresh tree. If you have a window of opportunity to check it in before then, super. -P
Status: NEW → ASSIGNED
Target Milestone: M15
checked in. Thanks Troy. -P
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
QA is presently unequipped to verify this bug; would anyone care to verify it, or shall I rubber-stamp it as verified without inspection?
No follow-up received after five days; rubber-stamping as Verified Without Inspection, since QA can't verify code-level bug reports. Troy, please feel free to re-open this bug if any issues remain.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.