Re: [cc65] Memory impact of Atari library compile time defines

Hi!

On Mon, Nov 23, 2009 at 7:31 PM, Christian Grössler wrote:
> Hi,
>
> Daniel Serpell wrote:
>> With the attached 3 programs, the size went from:
>> -rw-r--r-- 1 daniel daniel 291 2009-11-22 23:06 empty-atari
>> -rw-r--r-- 1 daniel daniel 2254 2009-11-22 23:06 fileio-atari
>> -rw-r--r-- 1 daniel daniel 3150 2009-11-22 23:06 printf-atari
>> -rw-r--r-- 1 daniel daniel 1243 2009-11-22 23:06 put-atari
>>
>> To:
>> -rw-r--r-- 1 daniel daniel 290 2009-11-22 23:46 empty-atari
>> -rw-r--r-- 1 daniel daniel 2253 2009-11-22 23:46 fileio-atari
>> -rw-r--r-- 1 daniel daniel 2744 2009-11-22 23:46 printf-atari
>> -rw-r--r-- 1 daniel daniel 837 2009-11-22 23:46 put-atari
>>

I rewrote "__rwsetup", see the patch attached, now the sizes are:

-rw-r--r-- 1 daniel daniel 284 2009-11-23 23:30 empty-atari
-rw-r--r-- 1 daniel daniel 2230 2009-11-23 23:30 fileio-atari
-rw-r--r-- 1 daniel daniel 2716 2009-11-23 23:30 printf-atari
-rw-r--r-- 1 daniel daniel 786 2009-11-23 23:30 put-atari

>
> Ok, I overlooked this part when I first read the mail, so I
> manually made the change to put the fdtoiocb function into
> its own file (since the patch didn't apply). Please take
> a look. I did some sanity checks but nothing more yet.

I searched for some testsuite to run, and found the tests
in the cc65 source, but two of them failed to build because
a missing include, attached is a patch to add unistd.h.

Seems all still works ok.

Thanks,

    Daniel.

----------------------------------------------------------------------
To unsubscribe from the list send mail to majordomo@musoftware.de with
the string "unsubscribe cc65" in the body(!) of the mail.