[Webkit-unassigned] [Bug 27410] [Gtk] adding GObject Language bindings auto-generator

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Jul 18 08:42:56 PDT 2009


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





--- Comment #2 from Luke Kenneth Casson Leighton <lkcl at lkcl.net>  2009-07-18 08:42:55 PDT ---
(In reply to comment #1)
> (From update of attachment 33020 [details])
> The patch is still is huge. 

 yes.  i know.

> If possible please trim it down to something that
> can build the bindings for Node.idl only. The point here is to review the
> generation process (and code) and not to make an outright usable GLib bindings.

 sadly, as already discussed, and to reiterate here, which i am very
disappointed to be having to do:

 * milestone 1 has been reached with patch #16401.  i am already in
"maintenance" mode on this work - not "development".  the development milestone
(1) is _completed_.

 * the generation process (and code) was developed using high-level testing on
top of extreme rapid iterative process.  therefore, even _i_ don't know exactly
why many of these things are in there: all i can tell you is, "it works", and,
if you are willing to take the original patch, and apply it, and then also
build pywebkitgtk as well, you too can prove to yourself that "it works".

 * line-by-line, blow-by-blow, bit-by-bit, piece-by-piece review is going to
bore the pants off of me _and_ any reviewers wishing to take it on.  and take
absolutely forever.  i've said - already - i'm _not_ willing to commit that
level of time.

 * "trimming down" requires development, going "backwards" not forwards. 
development "backwards" is not something that not one but _two_ separate
developers have already said "no" to.


so.

if you would like less lines, i can simply cut the file in half, if you like. 
if that's too few lines, i'm happy to cut it in half again, until you have an
amount of lines of code that is acceptable.

but i have already made it clear: there will be _no_ time spent "developing". 
and "going backwards" requires "development.

i will re-enable the review request and await your decision.

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