-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[iOS 15.4] Fix unused variables #212
[iOS 15.4] Fix unused variables #212
Conversation
@@ -444,6 +444,7 @@ - (void)setIsSelected:(BOOL)isSelected; | |||
|
|||
#if PLATFORM(IOS_FAMILY) | |||
[[[m_webView _UIKitDelegateForwarder] asyncForwarder] webView:m_webView addMessageToConsole:dictionary withSource:messageSource]; | |||
UNUSED_PARAM(respondsToNewSelector); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be UNUSED_VARIABLE.
@@ -221,10 +221,12 @@ void FontCascade::drawGlyphs(GraphicsContext& context, const Font& font, const G | |||
break; | |||
} | |||
|
|||
#if PLATFORM(IOS_FAMILY) | |||
UNUSED_PARAM(shouldSmoothFonts); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be UNUSED_VARIABLE.
@@ -1319,7 +1319,9 @@ static Class _WebMessageDocumentClass() | |||
NSRange rangeToReplace = NSMakeRange(textLength, 0); | |||
NSDictionary *attrs; | |||
if (fileWrapper) { | |||
#if !PLATFORM(IOS_FAMILY) | |||
#if PLATFORM(IOS_FAMILY) | |||
UNUSED_PARAM(ignoreOrientation); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be UNUSED_VARIABLE.
@@ -388,6 +388,7 @@ static AccessibilityPreferences accessibilityPreferences() | |||
} | |||
#else | |||
UNUSED_PARAM(mediaDevicesEnabled); | |||
UNUSED_PARAM(isSafari); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be UNUSED_VARIABLE (and existing code has this issue of course).
@@ -2374,7 +2374,7 @@ VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const IntPoin | |||
if (!renderView) | |||
return VisiblePosition(); | |||
|
|||
#if PLATFORM(COCOA) | |||
#if PLATFORM(MAC) | |||
FrameView* frameView = &renderView->frameView(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no real chance of a meaningful side effect here, is there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so, unless frameView()
changes state. @cdumez, any thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine.
af51a2e
to
edf2de8
Compare
@@ -2374,7 +2374,7 @@ VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const IntPoin | |||
if (!renderView) | |||
return VisiblePosition(); | |||
|
|||
#if PLATFORM(COCOA) | |||
#if PLATFORM(MAC) | |||
FrameView* frameView = &renderView->frameView(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine.
Since @aproskuryakov asked for this, here is the build output of the first error being fixed here:
|
edf2de8
to
bc487e0
Compare
bc487e0