[Webkit-unassigned] [Bug 9191] JS*ElementWrapperFactory should be autogenerated
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed May 21 12:51:03 PDT 2008
http://bugs.webkit.org/show_bug.cgi?id=9191
------- Comment #5 from julien.chaffraix at gmail.com 2008-05-21 12:51 PDT -------
(In reply to comment #4)
> (From update of attachment 21207 [edit])
> Wow. Totally awesome.
>
> I assume you've diff'd the generated file from the old manual file to confirm
> that there are no bad changes?
Yes (it is not exactly a diff as the old code is using some macros to generate
the code). The preparatory work I checked in last week was to guarantee that
all tags would be guarded by the right #ifdefine guard and are not taken into
account if the guard is not matched.
I have also tested the build thoroughly with different options to confirm that
I had not forgotten some generated code.
> I think long term we want to move away from hacks in the perl file, and instead
> use a more advanced data file for generation. For example, you could encode
> all of the information from the perl hashes in the htmltags.in file using
> proper comments. (Or we could turn things like htmltags.in into XML like
> mozilla has for their bindings stuff.)
I think the best way here is to opt for XML files. Some of make_names.pl
parameters could also be included (like guardCppWith...) in the *.in files. It
would also clean and simplify make_names.pl.
--
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.
More information about the webkit-unassigned
mailing list