Eric Wasylishen wrote:

>
> > I might miss something, but I think this class should really be named
> > ETHistory and not ETHistoryManager. Each instance manage history
> > entries and not histories, in other words each ETHistoryManager object
> > just represents a simple history.
> > For example,  in the WebKit there is WebHistory which contains
> > WebHistoryItem as elements.
> > imo manager classes should only be created when they have no specific
> > corresponding names in the real-word.
>
> Sounds good, I could rename it if you like.
>

Sounds right to me too.


> Does this set of classes look more sensible to move to a "Utilities"
> (or maybe "Convenience") subproject?:
>
> ETTranscript - transcript for Smalltalk


This should be moved to SmalltalkSupport, shouldn't it? Maybe even rename it
to just Transcript?

-Truls
_______________________________________________
Etoile-dev mailing list
[email protected]
https://mail.gna.org/listinfo/etoile-dev

Reply via email to