[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Help-smalltalk] VisualGST UI
From: |
Holger Hans Peter Freyther |
Subject: |
Re: [Help-smalltalk] VisualGST UI |
Date: |
Thu, 24 Oct 2013 07:55:02 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Oct 23, 2013 at 12:24:32PM +0200, Gwenaël Casaccio wrote:
> HI,
Good Morning,
>
> So here is the big monster,
hehe, I really prefer evolution and small steps (as they can be
reviewed and undone more easily)
> debuggerClass [
> <category: '*VisualGST-debugger'>
>
> - ^ VisualGST.GtkDebugger
> + ^ nil
> +" ^ VisualGST.GtkDebugger"
> ]
left over. :)
> + | iter |
> + iter := self findIter: anObject ifAbsent: [ ^ aBlock value ].
> + model remove: iter
> + ]
> +
> + remove: anObject [
> + <category: 'model'>
> +
> + self remove: anObject ifAbsent: [ self error: 'item not found' ]
you appear to mix tabs and spaces. Please pick one of the two.
> - GtkAnnouncer current on: GtkNamespaceSelectionChanged do: [ :ann |
> + "GtkAnnouncer current on: GtkNamespaceSelectionChanged do: [ :ann |
> browsers updateWidget: browsers currentWidget withLabel: ann
> selectedNamespace name asString].
> GtkAnnouncer current on: GtkClassSelectionChanged do: [ :ann |
> - browsers updateWidget: browsers currentWidget withLabel: ann
> selectedClass printString]
> + browsers updateWidget: browsers currentWidget withLabel: ann
> selectedClass printString]"
> ]
left over?
This patch is impossible to review but the client code appears to get
more simple indeed. I wish you could break this into smaller logical
pieces.
thanks
holger