search

stardot / beebasm

A portable 6502 assembler with BBC Micro style syntax
http://www.retrosoftware.co.uk/wiki/index.php/BeebAsm
GNU General Public License v3.0
83 stars 26 forks source link

Update test runners and fix new compiler warning #87

Closed mungre closed 1 year ago

mungre commented 1 year ago

Update the tests to use the latest versions of Ubuntu and macOS. Also fix the code to avoid new compiler warning.

mungre commented 1 year ago

Cheers. I tried another way of fixing this that I thought would be more readable but the compiler didn't like it. So I think we're stuck with having the null termination out of sight in Allocate.

Do you want to put this in v1.10?

chriskillpack commented 1 year ago

I think it should go in - it's a small change that addresses scary warnings from clang. Go ahead and merge when you are ready.