search

Arakula / A09

A09 6800/6801/6809/6309/68HC11 Assembler
GNU General Public License v2.0
40 stars 8 forks source link

Port to MacOS #2

Closed stevebyan closed 5 years ago

stevebyan commented 5 years ago

The MacOS Clang compiler still gives one warning:

a09.c:3979:19: warning: address of array 'curline->txt' will always evaluate to 'true' [-Wpointer-bool-conversion] else if (curline->txt) / otherwise / ~~ ~~~^

It's not obvious to me what the original intent was, so I didn't make any changes to address this issue.

Arakula commented 5 years ago

OK, I integrated that, but I'll make some little changes to that in the near future.

Arakula commented 5 years ago

... and the clang warning is a coding error, as it should read "if (*curline->txt)" . Has no real consequences, but I'll fix it in the next commit.

stevebyan commented 5 years ago

Thanks. I just tested my changes on Fedora. At 1a4ef410968, your old head of master, gcc 6.2.1 picks up the implicit declaration of 'unlink':

a09.c: In function ‘main’: a09.c:7054:3: warning: implicit declaration of function ‘unlink’ [-Wimplicit-function-declaration] unlink(objname); ^~

but it has no other complaints. (Unlike MacOS, Linux has a /usr/include/malloc.h)

With the fixes, a09 compiles, installs, and runs cleanly on Fedora.