Closed Bug 689367 Opened 8 years ago Closed 8 years ago
Layout Module .cpp:761:1: warning: 'k NS _CANVASRENDERINGCONTEXT2DAZURE _CID' defined but not used
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?)
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
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)
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+
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.
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
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.