Re: [Fx2lib-devel] Deprecation warnings

[Daniel O'C. <doc...@gs...>]

On 16/07/2010, at 21:45, Dennis Muhlestein wrote:
> Cloning your repository  worked just fine.  I went ahead and tagged  
> the current repository state as 0.2.  It's been quite stable for a  
> while and seemed a good place to start off on a new set of changes.   
> Your changes merged into my repo just fine but for some reason I'm  
> getting a seg fault on my version on my mac.  I'll try a different  
> version on Mac and also compile on Linux and then push the changes and  
> update the Changelog.

OK.
I got seg faults until I used a sdcc snapshot, however I saw on sf.net there was a bug report about it, ie
http://sourceforge.net/tracker/?func=detail&aid=2909385&group_id=247216&atid=1126193

> I'm not sure about past compatibility with other sdcc versions, but it  
> might be a better idea to add BIT to fx2types as a typedef, SFR could  
> go in fx2regs as a macro at the top (actually, there is already a  
> standard sdcc cross register header for cross compiling register  
> entries, would be nice to convert to that one of these days.  I'm not  
> sure where I like __interrupt, __at et al.   Perhaps just changing  
> them is best.

I would be inclined to add something to the makefiles like...
.ifdef (OLDSDCC)
CFLAGS+=-D__xdata=xdata
CFLAGS+=-D__sfr=sfr
...
.endif
(Dunno if that's legal GNUmake I usually use BSDmake)

I am not sure what else you would want to be compatible with. It is an intrinsically unportable thing because the C standard doesn't dictate how any of this stuff works.

FWIW avr-gcc (the only other microcontroller C compiler I am familiar with) does it differently (ie interrupt handling and so on). There aren't any special reserved words in the compiler itself (I believe anyway) - it's just macros.

--
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
  -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C