Including ti84pce.h in multiple files causes linktime warnings. about toolchain HOT 10 CLOSED

Tagging this as an external issue until proven otherwise, as include guards are already present, so I'm not sure what to do about it...

from toolchain.

Include guards protect against multiple inclusion of a given header during a single compiler invocation, but they're powerless against single or multiple inclusion of a given header during multiple independent compiler invocations.
Now, why exactly is the toolchain complaining about __frameset specially, which is an equate in the ASM file included by ti84pce.h, I don't know.

from toolchain.

Oh, yeah, "multiple independent compiler invocations", you're probably right...

from toolchain.

WARNING (909) (00000000 Z:\opt\CEdev\lib\std\chelp.lib)--> Public symbol "__imuls" already externally defined.
WARNING (909) (00D1CEC1 Z:\opt\CEdev\lib\std\chelp.lib)--> Public symbol "__imuls" already externally defined.

I believe this is related.

from toolchain.

Perhaps turning the equates into a mix of defines and xrefs would help?

from toolchain.

I think "external issue" is not the right tag for this. It's not a compiler issue, it's a "the include file doesn't behave like an include file, because it introduces link-time symbols, rather than merely providing forward declarations" issue.

from toolchain.

Yeah, removed it.

from toolchain.

Just use _At rather than equ.

from toolchain.

The best solution would be to redo the standard libraries to use the libraries in the bootcode. This probably means modifying the crtl source so it doesn't cause these warnings.

from toolchain.

Fixed with release v2.0 (And no, changing the name to 'tice.h' wasn't the solution :P)

from toolchain.