[Webkit-unassigned] [Bug 33590] [GTK] GObject DOM bindings

Tue Mar 9 07:24:14 PST 2010

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

Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #47928|review?                     |review+
               Flag|                            |

--- Comment #37 from Adam Barth <abarth at webkit.org>  2010-03-09 07:24:13 PST ---
(From update of attachment 47928)
I struggled with what to do with this patch.  On the one hand, it's much to
large to give anything but a syntactic review, but on the other hand, the patch
has been up for review for a while and a number of people have commented.  The
GDOM bindings seem to be in demand by a number of people and we've got to start
somewhere.

I'm surprised that there's so much code in the make files.  I would have
expected those snippets to be broken out into a separate file somehow.

In expanding the GDOM bindings, please consider using the generic bindings,
when possible, to improve maintainability.  The generic bindings don't have
much in them yet (and certainly nothing that's helpful for this patch), but
we're hoping to grow them over time.

On balance, I think we should take this patch and iterate on this code rather
than leaving the patch in review forever.

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