[Webkit-unassigned] [Bug 12122] SVGPathElement should not inherit from SVGPathParser

Tue Jun 5 15:40:05 PDT 2007

http://bugs.webkit.org/show_bug.cgi?id=12122


eric at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14845|review?                     |review-
               Flag|                            |


------- Comment #2 from eric at webkit.org  2007-06-05 15:40 PDT -------
(From update of attachment 14845)
It's a rather incomplete fix.  Basically you're just adding another class to
codify the existing hack of needing to allocate a class to do the parsing.  If
you're going to add this PathBuilder class, i would think you would make
SVGPathElement use it too.  I don't think that this fix is really much better
than the existing hack.  If you got rid of the other instances of classes
inheriting from SVGPathParser, that would be a more complete fix.  We can chat
about this on IRC if you like.


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