Closed
Bug 689367
Opened 13 years ago
Closed 13 years ago
layout/build/nsLayoutModule.cpp:761:1: warning: 'kNS_CANVASRENDERINGCONTEXT2DAZURE_CID' defined but not used
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: dholbert, Assigned: dholbert)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file)
2.15 KB,
patch
|
bas.schouten
:
review+
|
Details | Diff | Splinter Review |
bug 651858 added NS_CANVASRENDERINGCONTEXT2DAZURE_CID and a "NS_DEFINE_NAMED_CID()" line for it, as shown here: https://hg.mozilla.org/mozilla-central/rev/a8171812fe05#l6.2 ... but it never uses it beyond those two lines, as shown here: https://mxr.mozilla.org/mozilla-central/search?string=CANVASRENDERINGCONTEXT2DAZURE_CID&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central (compare to e.g. the same search with s/2DAZURE/WEBGL/, which turns up 3 usages: https://mxr.mozilla.org/mozilla-central/search?string=NS_CANVASRENDERINGCONTEXTWEBGL_CID ) It looks like we never needed this CID and can delete it. (unless we should perhaps be using it but forgot to?)
Assignee | ||
Comment 1•13 years ago
|
||
I noted in the bug title but forgot to mention in comment 0 -- this bug triggers the following build warning: > layout/build/nsLayoutModule.cpp:761:1: warning: 'kNS_CANVASRENDERINGCONTEXT2DAZURE_CID' defined but not used https://tbpl.mozilla.org/php/getParsedLog.php?id=6559103&tree=Firefox
Assignee | ||
Comment 2•13 years ago
|
||
This patch just removes the unused CID. (Bas, let me know (and feel free to take this) if there's something else that needs to be done here.)
Attachment #562584 -
Flags: review?(bas.schouten)
Assignee | ||
Updated•13 years ago
|
Blocks: buildwarning
Updated•13 years ago
|
Assignee: nobody → dholbert
Comment 3•13 years ago
|
||
Comment on attachment 562584 [details] [diff] [review] fix: remove unused CID Review of attachment 562584 [details] [diff] [review]: ----------------------------------------------------------------- Looks right to me!
Attachment #562584 -
Flags: review?(bas.schouten) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Oops, I somehow missed that this got r+. :) Sorry for delay in landing. After I do a sanity-check to be sure this still builds, I'll push.
Assignee | ||
Comment 5•13 years ago
|
||
Landed: https://hg.mozilla.org/integration/mozilla-inbound/rev/6f440ff5247f BTW, I have the 'eol' hg extension on the machine I'm pushing from, and that caught & fixed some DOS line-endings in 3 lines of contextual code (for NS_CANVASRENDERINGCONTEXT2DTHEBES_CID) when I did a final qref on the patch. (those were the only lines with DOS line-endings in the whole file) I left that line-ending-fix in the patch when pushing, since it doesn't affect behavior and DOS line endings are evil.
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → mozilla11
Comment 6•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6f440ff5247f
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•