Remove unused parameter warning. WebKit/Source/WebCore/rendering/RenderObject.cpp:127:13: warning: unused parameter ‘frameView’ [-Wunused-parameter]
Created attachment 208579 [details] Patch
Comment on attachment 208579 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208579&action=review > Source/WebCore/rendering/RenderObject.cpp:129 > -#if !ENABLE(FAST_MOBILE_SCROLLING) && !PLATFORM(QT) > +#if !ENABLE(FAST_MOBILE_SCROLLING) || (ENABLE(FAST_MOBILE_SCROLLING) && !PLATFORM(QT)) That’s the same as just change && to ||, why make it so complicated?
Comment on attachment 208579 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208579&action=review >> Source/WebCore/rendering/RenderObject.cpp:129 >> +#if !ENABLE(FAST_MOBILE_SCROLLING) || (ENABLE(FAST_MOBILE_SCROLLING) && !PLATFORM(QT)) > > That’s the same as just change && to ||, why make it so complicated? In other words, this is: #if !A || (A && !B) Which is the same as: #if !A || !B
(In reply to comment #3) > (From update of attachment 208579 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=208579&action=review > > >> Source/WebCore/rendering/RenderObject.cpp:129 > >> +#if !ENABLE(FAST_MOBILE_SCROLLING) || (ENABLE(FAST_MOBILE_SCROLLING) && !PLATFORM(QT)) > > > > That’s the same as just change && to ||, why make it so complicated? > > In other words, this is: > > #if !A || (A && !B) > > Which is the same as: > > #if !A || !B Just check it explicitly. You're right. It could be so complicated. Thanks.
Created attachment 208580 [details] Patch
(In reply to comment #4) > (In reply to comment #3) > > (From update of attachment 208579 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=208579&action=review > > > > >> Source/WebCore/rendering/RenderObject.cpp:129 > > >> +#if !ENABLE(FAST_MOBILE_SCROLLING) || (ENABLE(FAST_MOBILE_SCROLLING) && !PLATFORM(QT)) > > > > > > That’s the same as just change && to ||, why make it so complicated? > > > > In other words, this is: > > > > #if !A || (A && !B) > > > > Which is the same as: > > > > #if !A || !B > > Just check it explicitly. You're right. > It could be so complicated. Thanks. Fixed it.
Comment on attachment 208580 [details] Patch Clearing flags on attachment: 208580 Committed r153974: <http://trac.webkit.org/changeset/153974>
All reviewed patches have been landed. Closing bug.