[Webkit-unassigned] [Bug 54443] Textarea maxlength doesn't account for newlines

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 15 22:20:09 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=54443





--- Comment #32 from Ryosuke Niwa <rniwa at webkit.org>  2011-03-15 22:20:09 PST ---
(From update of attachment 85907)
View in context: https://bugs.webkit.org/attachment.cgi?id=85907&action=review

>> Source/WebCore/editing/TypingCommand.cpp:50
>> +// Helper function to check if we can insert new line feed with \n(line feed) string.
> 
> This comment seems to repeat what code does.  In general, we'd like to make code self-explanatory and add as little comment as possible.

I forgot to say that you should remove this comment.

-- 
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