Closed Bug 645750 Opened 9 years ago Closed 9 years ago
CSSParser .cpp:8060:19: warning: unused variable ‘variant Mask’ (also ‘table’)
> 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)
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.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.