[Webkit-unassigned] [Bug 90544] Stop tracking line number in tokenizer

Wed Jul 4 08:20:37 PDT 2012

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

Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #150775|review?                     |review+
               Flag|                            |

--- Comment #7 from Adam Barth <abarth at webkit.org>  2012-07-04 08:20:37 PST ---
(From update of attachment 150775)
View in context: https://bugs.webkit.org/attachment.cgi?id=150775&action=review

This looks great.

> Source/WebCore/html/parser/HTMLDocumentParser.cpp:452
> +    const SegmentedString& currentString = m_input.current();
> +    return currentString.currentLine();

I would just combine these two lines to avoid having the temporary.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.