[Biococoa-dev] compiler warning

[Alexander Griekspoor]

Just a few remarks while going through the code:
- I also got a warning for a mistyped NSMutableString tempString, which 
shouldn't be mutable.
- 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. In fact I have seen major speedups/memory 
footprint reductions in my apps by a simple direct release instead of 
waiting for the autorelease if you use many objects. A common thing I 
do in a so-called double loop is to setup an autorelease pool for each 
inner loop cycle and release that everytime the outerloop continues, it 
can mean the difference between a nicely running app and a one that 
locks up the complete computer.
- 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...
Cheers,
Alex


Op 4-dec-04 om 3:01 heeft Koen van der Drift het volgende geschreven:

> Hi,
>
> I get the following compiler warning in BCSequenceReader:
>
> BCFoundation/BCSequenceIO/BCSequenceReader.m:418: warning: 
> `BCSequence' may not respond to `-autorelease'
>
>
> BCSequence is a subclass of NSObject, so it should inherit that 
> method, right?
>
>
> - Koen.
>
> _______________________________________________
> Biococoa-dev mailing list
> Biococoa-dev at bioinformatics.org
> https://bioinformatics.org/mailman/listinfo/biococoa-dev
>
>
*********************************************************
                      ** Alexander Griekspoor **
*********************************************************
                The Netherlands Cancer Institute
                Department of Tumorbiology (H4)
          Plesmanlaan 121, 1066 CX, Amsterdam
                   Tel:  + 31 20 - 512 2023
                   Fax:  + 31 20 - 512 2029
                  AIM: mekentosj at mac.com
                   E-mail: a.griekspoor at nki.nl
               Web: http://www.mekentosj.com

    The requirements said: Windows 2000 or better.
    So I got a Macintosh.

*********************************************************
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: text/enriched
Size: 2413 bytes
Desc: not available
URL: <http://www.bioinformatics.org/pipermail/biococoa-dev/attachments/20041204/a6910cf6/attachment.bin>