This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH/RFA] Fix ctype table and isblank


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Jeff Johnston on 4/8/2009 12:03 PM:
>> The problem are smaller targets.  Maybe it would be a feasible approach
>> to stick to one single 8 bit table (for ASCII) in case of small targets
>> and to provide everything as 16 bit tables for targets which don't care
>> the few extra K bytes.
>>
>> And widening the tables introduces a new problem for Cygwin, which has
>> to keep some of the "old" stuff to maintain backward compatibility with
>> applications built under an older version.  It would have to maintain
>> the old ASCII ctype table, and copy over the data
>> from the new tables in a more tricky way; we would have to keep the
>> meaning of the current bit values and only extend flags to the upper
>> bytes so that Cygwin can deal with that for older apps.  New apps would
>> immediately profit from the new tables, of course.
> 
> 1. Widen the tables (16 bits) and create a new ctype ptr with a new name.

If we still need more bits, even after the _U|_L improvement, why not
create two parallel 8-bit tables with all the new bits in the new table?
You don't have to widen the old tables, as that just makes life harder for
portability to the apps compiled against the older table.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             ebb9@byu.net
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkndQVoACgkQ84KuGfSFAYCsOwCeJMlpIxH7ApITFVShl/lVO6WS
HioAmQEfgcfqvp0+Ms6jLAvdFzWVvTeT
=n5cs
-----END PGP SIGNATURE-----


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]