Closed
Bug 708109
Opened 13 years ago
Closed 13 years ago
Eliminate THEBES_INLINE_DECL_THREADSAFE_REFCOUNTING
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(1 file)
(deleted),
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
NS_INLINE_DECL_THREADSAFE_REFCOUNTING has been added, so we should eliminate THEBES_INLINE_DECL_THREADSAFE_REFCOUNTING.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #579478 -
Flags: review?(joe)
Comment 2•13 years ago
|
||
Comment on attachment 579478 [details] [diff] [review]
fix
Review of attachment 579478 [details] [diff] [review]:
-----------------------------------------------------------------
These AddRef and Release are NS_METHOD_, which is pretty weird in thebes and layers. I don't know if it's a huge issue in practice, but I think we should be aware of this. I'll r- for now; please reflag for review if you think it's ok.
Attachment #579478 -
Flags: review?(joe) → review-
Assignee | ||
Comment 3•13 years ago
|
||
Comment on attachment 579478 [details] [diff] [review]
fix
Review of attachment 579478 [details] [diff] [review]:
-----------------------------------------------------------------
I don't see that it matters.
Attachment #579478 -
Flags: review- → review?
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 579478 [details] [diff] [review]
fix
Review of attachment 579478 [details] [diff] [review]:
-----------------------------------------------------------------
I don't see that it matters.
Attachment #579478 -
Flags: review? → review?(joe)
Assignee | ||
Comment 5•13 years ago
|
||
It might matter if this added significantly to the dependency of layers on XPCOM, but that dependency is already pretty large.
Comment 6•13 years ago
|
||
Comment on attachment 579478 [details] [diff] [review]
fix
Review of attachment 579478 [details] [diff] [review]:
-----------------------------------------------------------------
I only mentioned it because it also changes the calling convention of AddRef and Release, but it's not a big deal, you're right.
Attachment #579478 -
Flags: review?(joe) → review+
Assignee | ||
Comment 7•13 years ago
|
||
Whiteboard: [inbound]
Comment 8•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla11
You need to log in
before you can comment on or make changes to this bug.
Description
•