Re: empty document at Graphos startup

[Ivan Vučica]

On Mon, Dec 19, 2011 at 18:50, Wolfgang Lux <wolfgang.lux@gmail.com> wrote:

BTW, I've noticed this bogus code in the -init method:
   [super init];
   if (self)
This should read
   self = [super init];
   if (self)
or more compactly
   if ((self = [super init]) != nil)

 

Or even more compactly:

  if((self = [super init])

I'll go a bit more offtopic... 

To be honest, I find the following the nicest style, since the body of the initializer is on the first level of indentation, not on the second level:

//////

self = [super init];

if(!self)

  return nil;

// body here

return self;

//////

-- 
Ivan Vučica - ivan@vucica.net