On Thu, 2011-07-14 at 14:26 +0200, Martin Nordholts wrote:
> Hi Enrico,
> I've made a first review-round of some of your new code. It's not a
> complete review, but it's a start. I hope the to-the-point comments
> are OK, I don't mean to be rude.
> Note that I'm CCing gimp-developer to keep our correspondence public.
> I've done the review by diffing origin/master with
> origin/soc-2011-gimpunitentry (after locally merging master to your
> branch) and put the result in a text file, then adding comments
> inline. The patch has been indented 8 spaces to make the comments
> stand out more. The diff with comments is found here:
> If you have comments on my comments, just continue this email thread.
I have an additional comment:
All these changes:
- GtkWidget *size_se;
+ GimpUnitEntryTable *size_se;
The variable must stay a GtkWidget, it a convention for all widgets,
since the widget_new() functions also by convention return a GtkWidget.
Gimp-developer mailing list