Generated methods that return BOOL should return NO instead of 0.

[seanm]

AnalysisTool (http://www.karppinen.fi/analysistool/) which in turn uses the llvm static analyzer generates warnings for mogenerated code. ex:

• (BOOL)primitiveIsLeafValue {
  NSNumber *result = [self primitiveIsLeaf];
  return result ? [result boolValue] : 0;
  }

It warns "coercion from 'int' to 'BOOL' may alter its value". Of course, there isn't really anything wrong with the code above, but the warning could be easily silenced by changing that 0 to a NO. That would fix it because the definition of NO includes a cast to BOOL.

This warning is useful in other situations but all these false positives are generating a lot of noise.

[rentzsch]

This is a little hard to deal with since we'd need to special-case BOOLs in the machine.m template

[seanm]

:( That's a pity. Combined with bug #2, I'm starting to have the impression that mogenerator is difficult to expand upon... :(

[rentzsch]

It's not difficult to expound on, it just has to deal with asymmetry between Obj-C types and their NSValue/NSNumber method names. I haven't written a mapper since I haven't needed it for myself yet.

[seanm]

Fixed: http://github.com/seanm/mogenerator/commit/fbb156268d2a6fd5668fe43dc4c815ddeca4b1d8

[rentzsch]

Thanks! Closing.