|
Rockbox mail archiveSubject: Re: dave: apps/plugins/databox edittoken.h,1.5,1.6Re: dave: apps/plugins/databox edittoken.h,1.5,1.6
From: Dave Chapman <dave_at_dchapman.com>
Date: Sat, 19 Nov 2005 12:46:36 +0000 Linus Nielsen Feltzing wrote: > cvs_at_labb.contactor.se wrote: > >> iPod: signed-char police >> > > Can't we use the -fsigned-char gcc option instead of policing the code? > > Linus As you pondered in irc, policing the code has the advantage of making it more portable. I've already fixed all the obvious problems gcc found (i.e. testing if a var of type "char" was < 0), but I think I would prefer to avoid adding -fsigned-char unless we run into problems in the future. Most of the codecs use the int8_t and uint8_t types - which are now explicitly defined to be signed or unsigned char respectively. For people who haven't been following this, the C specification says that plain "char" is either signed or unsigned depending on the platform. Up to now, all Rockbox platforms (including x86 for the sims) have had signed chars - but chars are unsigned on ARM. Dave. Received on 2005-11-19 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |