[webkit-reviews] review denied: [Bug 46679] Delete and forward delete shouldn't start autocorrection panel timer. : [Attachment 69099] Proposed patch (v2)
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Sep 28 14:17:36 PDT 2010
mitz at webkit.org has denied jpu at apple.com's request for review:
Bug 46679: Delete and forward delete shouldn't start autocorrection panel
timer.
https://bugs.webkit.org/show_bug.cgi?id=46679
Attachment 69099: Proposed patch (v2)
https://bugs.webkit.org/attachment.cgi?id=69099&action=review
------- Additional Comments from mitz at webkit.org
View in context: https://bugs.webkit.org/attachment.cgi?id=69099&action=review
> WebCore/editing/TypingCommand.cpp:321
> +#if PLATFORM(MAC) && !defined(BUILDING_ON_TIGER) &&
!defined(BUILDING_ON_LEOPARD)
> void TypingCommand::typingAddedToOpenCommand(ETypingCommand
commandTypeForAddedTyping)
> +#else
> +void TypingCommand::typingAddedToOpenCommand(ETypingCommand)
> +#endif
The complaint was about markMisspellingsAfterTyping() having an unused (in some
configurations) parameter, not about this method, so I’m not sure why you’re
changing this one. I think it will break all builds.
The way to fix the warning about markMisspellingsAfterTyping() is to add
UNUSED_PARAM(commandType)
for the configurations that don’t use commandType.
You’ll need to make sure that you’re including <wtf/UnusedParam.h>—but I think
you have it by way of including Editor.h.
More information about the webkit-reviews
mailing list