Closed Bug 793440 Opened 7 years ago Closed 7 years ago
Tautological constant out of range check on DOMSVGTransform
This breaks the build on a recent Clang.
This is right but it's not enough, it shows we have other issues. mIsAnimValItem should be uint32_tmIsAnimValItem:1; in DOMSVGTransform.h also MOZ_SVG_LIST_INDEX_BIT_COUNT should be 31 (the data items should add to 32) in that class. So it looks to me like DOMSVGNumber.h and DOMSVGLength.h have correct values. Then DOMSVGNumber.h and DOMSVGLength.h should take bool arguments for aIsAnimValItem so that we can remove the aIsAnimValItem < (1 << 1) check from DOMSVGLength.cpp and DOMSVGNumber.cpp too. If you don't want to do all that, please raise a follow up so that we can track it.
make that uint32_t mIsAnimValItem:1;
Comment on attachment 663753 [details] [diff] [review] Patch Thanks Reuben.
Attachment #663753 - Flags: review?(longsonr) → review+
Green on Try. https://tbpl.mozilla.org/?tree=Try&rev=e8bb654fd567 https://hg.mozilla.org/integration/mozilla-inbound/rev/0fbabfc42867
Assignee: nobody → reuben.bmo
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.