[Pharo-project] Let's change DamageRecorder>>new

2 views
Skip to first unread message

Fernando Olivero

unread,
Jan 25, 2012, 9:32:20 AM1/25/12
to pharo-...@lists.gforge.inria.fr
Found a really bad looking code here...

DamageRecorder class>>new
^ super new reset

Should be:
Delete: DamageRecorder class>>new.
DamageRecorder>>initialize
super initialize.
self reset.


Right?
Fernando

Stéphane Ducasse

unread,
Jan 25, 2012, 9:47:22 AM1/25/12
to Pharo-...@lists.gforge.inria.fr, fernando...@usi.ch
does it solve some redraw problems we experienced?

Stef

Fernando Olivero

unread,
Jan 25, 2012, 11:25:23 AM1/25/12
to Stéphane Ducasse, Pharo-...@lists.gforge.inria.fr
no, I think it wont change the behavior, the change would just fix a
bad style of coding.
Fernando


On Wed, Jan 25, 2012 at 3:47 PM, Stéphane Ducasse
<stephane...@inria.fr> wrote:
> does it solve some redraw problems we experienced?
>
> Stef
>
> On Jan 25, 2012, at 3:32 PM, Fernando Olivero wrote:
>

Stéphane Ducasse

unread,
Jan 25, 2012, 11:33:54 AM1/25/12
to Pharo-...@lists.gforge.inria.fr, fernando...@usi.ch
add an entry that we do not forget.

Stef

On Jan 25, 2012, at 3:32 PM, Fernando Olivero wrote:

Henrik Johansen

unread,
Jan 25, 2012, 12:58:57 PM1/25/12
to Pharo Development

+1 from me at least…

Cheers,
Henry

Reply all
Reply to author
Forward
0 new messages