[Webkit-unassigned] [Bug 27425] adding GDOMHTMLElementWrapperFactory[.cpp/.h]

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jul 19 13:28:50 PDT 2009


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


Luke Kenneth Casson Leighton <lkcl at lkcl.net> changed:

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




--- Comment #1 from Luke Kenneth Casson Leighton <lkcl at lkcl.net>  2009-07-19 13:28:49 PDT ---
Created an attachment (id=33052)
 --> (https://bugs.webkit.org/attachment.cgi?id=33052)
adds gdomhtmlelementwrapperfactory.

these files have review comments on it (one key reason why they're submitted
under a separate bugreport).

from memory, the questions are:

* why is there "shouting" in it?

  answer: because the original files, JSHTMLElementWrapperFactory.[cpp,h] from
which they're copied, also have "shouting" in them.

* why is this not auto-generated?

  answer: because the auto-generator is written in a bug-ugly language called
perl, and it was easier to do a first revision by cut/paste using the advanced
editing features of vim than it was to mind-bend to yet another perl script [i
did try].  also, the exception handling has been completely ripped out of
CodeGeneratorGObject which makes this file quite different from its origins.

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