[webkit-reviews] review requested: [Bug 15960] View source mode displays the attributes after an attribute with an empty value incorrectly. : [Attachment 18076] Proposed patch v3

[bugzilla-daemon at webkit.org]

Anyang Ren <anyang.ren at gmail.com> has asked  for review:
Bug 15960: View source mode displays the attributes after an attribute with an
empty value incorrectly.
http://bugs.webkit.org/show_bug.cgi?id=15960

Attachment 18076: Proposed patch v3
http://bugs.webkit.org/attachment.cgi?id=18076&action=edit

------- Additional Comments from Anyang Ren <anyang.ren at gmail.com>
I figured out a better fix.  It's much easier to determine when
we start processing the next attribute than to determine when we're
done with the current attribute.  So whenever we see the guide
character 'a', signaling the beginning of an attribute, we increment
currAttr.

This requires initializing currAttr to -1 so that after the first
increment it starts at 0.  So I have to declare currAttr as 'int',
and have to add a static cast to unsigned when I compare it with
token->attrs->length(), an unsigned value.