On Tue, Aug 10, 2010 at 10:12 AM, St�phane Ducasse <stephane.ducasse@inria.fr> wrote:

>
> What's CUIS? :(

another squeak fork

> � � � �- then we should probably rewrite some parts and do not inherit from StringHolder
> � � � �I never understood why MessageSet is a subclass of Browser :)
> � � � �But I would focus on the
>
>
> on the ...?

paragraphEditor kicking out

>
>
> > StringHolder class comment says: "I am a kind of Model that includes a piece of text. �In some cases, the text can be edited, and in some the text is a method."
> >
> > And I smell that its main responsibility is to hold a Text or a String in its contents variable, and all its subclasses are there because of that... �And if so, I don't like that inheritance approach :).
>
> This is exact.
>
> > I want to destroy (literally) that hierarchy :). �And maybe it helps us in coming refactors and cleanings, so we can do them with less fear :P.
>
> So could you
> � � � �review benjamin code?
>
> This evening I'll check it and compare it with the existant RecentMessageSet and that stuff.
>
> � � � �then check the texteditor/paragraphEditor
>
> I'll write it down in my TODO list :). �But the idea is to replace ParagraphEditor with them, is it?

yes

>
> >
> > Well, this mail is just for ask for advice and argue about the correctness of this change (or not).
> >
> > Thanks for reading up to here!
> > Guille
> > _______________________________________________
> > Pharo-project mailing list
> > Pharo-project@lists.gforge.inria.fr
> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> _______________________________________________
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

_______________________________________________
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project