Closed Bug 645750 Opened 9 years ago Closed 9 years ago

nsCSSParser.cpp:8060:19: warning: unused variable ‘variantMask’ (also ‘table’)

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

(Whiteboard: [build_warning])

Attachments

(2 files)

Attached patch fixSplinter Review
> nsCSSParser.cpp:8060:19: warning: unused variable ‘variantMask’
> nsCSSParser.cpp:8061:26: warning: unused variable ‘table’

These variables last usages were removed in
  http://hg.mozilla.org/mozilla-central/rev/f0f1632c19e8

Fix attached (just removing the variables)
Attachment #522394 - Flags: review?(dbaron)
Assignee: nobody → dholbert
Comment on attachment 522394 [details] [diff] [review]
fix

r=dbaron, though this isn't the first time I've reviewed a fix for this warning
Attachment #522394 - Flags: review?(dbaron) → review+
(In reply to comment #1) 
> r=dbaron
thanks!

> though this isn't the first time I've reviewed a fix for this warning

Hmm, thanks for the heads-up...  I searched Bugzilla for 'variantMask' in both bug-summary & bug-comments, and I didn't find anything recent...  If I come across a dupe, I'll mark as appropriate, but otherwise I'll likely just land this today or tomorrow, with a bundle of other patches that I'm TryServer-testing right now.
(In reply to comment #2)
> I'll likely just land
> this today or tomorrow, with a bundle of other patches that I'm
> TryServer-testing right now.

D'oh, I forgot to include this in my landing.  I'll post a |hg export| version that hopefully someone can take as a ridealong.
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/8beb43b5e113
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.