https://pharo.fogbugz.com/f/cases/16073/TextEditor-initialization

On 30 Jul 2015, at 12:00, Marcus Denker <marcus.denker@inria.fr> wrote:


On 30 Jul 2015, at 11:15, Stephan Eggermont <stephan@stack.nl> wrote:

Is there a specific reason why

TextEditor>>initWithMorph: aMorph

super initialize.
morph := aMorph.
self resetState.

calls super initialize instead of self initialize?

#initialize seems to be not implemented in the hierarchy anyway.


TextMorph uses that with its subclass SmalltalkEditor as

TextEditor class>>forMorph: aMorph

^ self basicNew initWithMorph: aMorph


I would

- change basicNew to new in forMorph:
- remove the call to #initialize in #initWithMorph: as it is done by #new

Marcus