[Webkit-unassigned] [Bug 73275] Upstream BlackBerry porting of WebCore/editing
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Nov 28 20:26:56 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=73275
Daniel Bates <dbates at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #116876|review? |review+, commit-queue-
Flag| |
--- Comment #2 from Daniel Bates <dbates at webkit.org> 2011-11-28 20:26:56 PST ---
(From update of attachment 116876)
View in context: https://bugs.webkit.org/attachment.cgi?id=116876&action=review
> Source/WebCore/editing/blackberry/EditorBlackBerry.cpp:23
> +#include "ClipboardAccessPolicy.h"
This include is unnecessary since ClipboardBlackBerry.h (below) includes Clipboard.h, which includes this file.
> Source/WebCore/editing/blackberry/EditorBlackBerry.cpp:25
> +#include "NotImplemented.h"
Remove this include since we aren't using its functionality in this file.
> Source/WebCore/editing/blackberry/EditorBlackBerry.cpp:26
> +#include "PassRefPtr.h"
This should be #include <wtf/PassRefPtr.h>.
> Source/WebCore/editing/blackberry/EditorBlackBerry.cpp:30
> +WTF::PassRefPtr<Clipboard> Editor::newGeneralClipboard(ClipboardAccessPolicy policy, Frame*)
Do we need the WTF:: prefix?
> Source/WebCore/editing/blackberry/SmartReplaceBlackBerry.cpp:26
> +bool isCharacterSmartReplaceExempt(UChar32 c, bool isPreviousCharacter)
Neither of these parameters are used in this function. If we want to keep the names of these parameters, don't we need to use UNUSED_PARAM() (*) here so as to avoid compiler warnings? The name of the first parameter doesn't seem very meaningful so I suggest omitting its name. You may want to consider using UNUSED_PARAM(isPreviousCharacter) so as to keep the parameter name isPreviousCharacter even though we don't use it.
(*) Defined in JavaScriptCore/wtf/UnusedParam.h, <http://trac.webkit.org/browser/trunk/Source/JavaScriptCore/wtf/UnusedParam.h?rev=74855#L32>.
> Source/WebCore/editing/blackberry/SmartReplaceBlackBerry.cpp:32
> +}
Nit: This is a short file. It's convention to put an inline comment here, // namespace WebCore, to demarcate the end of the WebCore namespace.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list