Summary: | Unused but set variable warning in MacroAssemberX86_64 | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xan Lopez <xan.lopez> | ||||||||
Component: | JavaScriptCore | Assignee: | Xan Lopez <xan.lopez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, commit-queue, dbates, eric, menard, webkit.review.bot, xan.lopez | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | OS X 10.5 | ||||||||||
Bug Depends on: | 59568 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Xan Lopez
2011-04-26 12:08:37 PDT
Created attachment 91143 [details]
Patch
Comment on attachment 91143 [details]
Patch
I wish there was a better way we could do this. This looks sane to me.
Comment on attachment 91143 [details] Patch Landed as r84977. Closing. http://trac.webkit.org/changeset/84977 might have broken SnowLeopard Intel Release (Tests) The following tests are not passing: accessibility/anchor-linked-anonymous-block-crash.html accessibility/aria-activedescendant-crash.html accessibility/aria-checkbox-checked.html accessibility/aria-checkbox-text.html accessibility/aria-combobox.html accessibility/aria-controls-with-tabs.html accessibility/aria-describedby-on-input.html accessibility/aria-disabled.html accessibility/aria-help.html accessibility/aria-hidden-update.html accessibility/aria-hidden-with-elements.html accessibility/aria-hidden.html accessibility/aria-label.html accessibility/aria-labelledby-on-input.html accessibility/aria-labelledby-overrides-label.html accessibility/aria-labelledby-stay-within.html accessibility/aria-link-supports-press.html accessibility/aria-list-and-listitem.html accessibility/aria-option-role.html accessibility/aria-presentational-role.html This patch was rolled out in <http://trac.webkit.org/changeset/85000> (bug #59568). Created attachment 91234 [details]
Patch
OK, that was obviously broken. Unfortunately we even need an even slightly uglier patch here... I wonder if there's a way of flagging a variable as unused, we could do that for release builds here. Maybe just (void)var;, like ASSERT_UNUSED, but without using the macro because we need it in NDEBUG. Comment on attachment 91234 [details]
Patch
We have an UNUSED() macro for this, no?
Created attachment 92095 [details]
Patch
Comment on attachment 92095 [details]
Patch
OK.
Comment on attachment 92095 [details] Patch Clearing flags on attachment: 92095 Committed r85718: <http://trac.webkit.org/changeset/85718> All reviewed patches have been landed. Closing bug. The commit-queue encountered the following flaky tests while processing attachment 92095 [details]: http/tests/xmlhttprequest/cross-origin-no-authorization.html bug 33357 (author: ap@webkit.org) The commit-queue is continuing to process your patch. |