The default bug view has changed. See this FAQ.

layout/build/nsLayoutModule.cpp:761:1: warning: 'kNS_CANVASRENDERINGCONTEXT2DAZURE_CID' defined but not used

RESOLVED FIXED in mozilla11

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks: 1 bug)

Trunk
mozilla11
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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

6 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

6 years ago
Created attachment 562584 [details] [diff] [review]
fix: remove unused CID

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

6 years ago
Blocks: 187528
Assignee: nobody → dholbert
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

5 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

5 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
https://hg.mozilla.org/mozilla-central/rev/6f440ff5247f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.