Closed
Bug 31664
Opened 25 years ago
Closed 25 years ago
Threadsafe assertions when running table regression tests
Categories
(Core :: Graphics: ImageLib, defect, P3)
Tracking
()
VERIFIED
FIXED
M15
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
Comment 3•25 years ago
|
||
QA is presently unequipped to verify this bug; would anyone care to verify it,
or shall I rubber-stamp it as verified without inspection?
Comment 4•25 years ago
|
||
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.
Description
•