[Biococoa-dev] compiler warning

Koen van der Drift kvddrift at earthlink.net
Fri Dec 3 21:38:31 EST 2004


On Dec 3, 2004, at 9:23 PM, Alexander Griekspoor wrote:

> Just a few remarks while going through the code:
> - I also got a warning for a mistyped NSMutableString tempString, 
> which shouldn't be mutable.

fixed.

> - Why do you use autorelease and not release it directly right after 
> you add it to the dictionary, you know you want to loose it anyway, so 
> why don't do it directly.

which autorelease are you referring to?


> - perhaps John is about to change this already, but why not remove the 
> create prefix in the methods of the sequencefactory, thus
> - (BCSequence *) sequenceWithSequence: (BCSequence *) sequence;
> instead of
> - (BCSequence *) createSequenceWithSequence: (BCSequence *) sequence;
> it's shorter, and adheres better to the guidelines. Similar to the 
> fact that it isn't  + (NSArray *) createArrayWithCapacity; but 
> arrayWithCapacity...
>

I'll change that.

- Koen.




More information about the Biococoa-dev mailing list