From Marcin Cieslak to ~rabbits/public-inbox
On Sun, 19 May 2024, ~xsova wrote: > From: Bryce Schultz <bryce@xsova.dev> > > -RELEASE_flags=-std=c89 -Os -DNDEBUG -g0 -s -Wall -Wno-unknown-pragmas > +RELEASE_flags=-std=c89 -Os -DNDEBUG -g0 -Wall -Wno-unknown-pragmas > DEBUG_flags=-std=c89 -DDEBUG -Wall -Wno-unknown-pragmas -Wpedantic -Wshadow -Wextra -Werror=implicit-int -Werror=incompatible-pointer-types -Werror=int-conversion -Wvla -g -Og -fsanitize=address -fsanitize=undefined > bin/uxnasm: src/utils/uxnasm.c > @ mkdir -p bin > @ cc ${RELEASE_flags} src/utils/uxnasm.c -o ${ASM} > - > + @ strip ${ASM} > +
From Marcin Cieslak to ~rabbits/public-inbox
On Tue, 26 Dec 2023, Kyle Perik wrote: > Do you think there's a way to avoid a different .s file? In theory, there should be a way... > I'm a C/ASM > novice so forgive my ignorance We are all learning here.... Is it possible to check what assembler ("as" command maybe) is used on your system? I try to reproduce the problem with a recent clang compiler and everything works as expected (but I'm not on a Mac).
From Marcin Cieslak to ~rabbits/public-inbox
On Sat, 23 Dec 2023, Kyle Perik wrote:
>> +.section __TEXT,__text
What happens if you leave it out completely?
(gcc seems to use text segment here by default, without .section for me)
What is the error you are getting with .rodata here? or is it about the symbol underscore?
saper
From Marcin Cieslak to ~rabbits/public-inbox
On Tue, 2 May 2023, Sigrid Solveig Haflínudóttir wrote:
> It'd also work, but only because snprintf is not a define. Imo, this is just wrong way to fix this.
Sorry, forget my babble. I've played too much with the macros that day. You are of course correct.
I just dislike the code sprinkled with some crazy system-specific #ifdefs
From Marcin Cieslak to ~rabbits/public-inbox
On Mon, 1 May 2023, Hundred Rabbits wrote: > Thanks Matus, > > Someone was hitting that same issue just yesterday, hopefully this > will help them with their issue, merged :) Would #ifndef snprintf also work?
From Marcin Cieslak to ~rabbits/public-inbox
On Sun, 19 Mar 2023, Deadly Headshot wrote: > - strcat(p, "/"); /* TODO: use a macro instead of '/' for the path delimiter */ > + strcat(p, DIR_SEP_STR); /* TODO: use a macro instead of '/' for the path delimiter */ I think we can remove the TODO comments here?
From Marcin Cieslak to ~rabbits/public-inbox
On Fri, 25 Nov 2022, Hundred Rabbits wrote: > That's no the latest version of uxnasm, you should update and it'll be fine :) Ouch, right: > uxnasm src/left.tal l.rom -- Unused label: draw-dec -- Unused label: phex Assembled l.rom in 9003 bytes(13.79% used), 424 labels, 0 macros. Thanks!
From Marcin Cieslak to ~rabbits/public-inbox
Hello, I have built uxn from source (1b2049e238df96f32335edf1c6db35bd09f8b42d) on FreeBSD and then I have tried to compile left using uxnasm (commit 5fc8cc1d847a4b95dc6e09ea467bdc3c7729b973): > uxnasm src/left.tal l.rom Unknown token: =file-new Unknown token: =file-new Assembly: Failed to assemble rom. I have downloaded https://rabbits.srht.site/drifblim/drifblim.rom and it worked: > uxncli ../play/drifblim.rom src/left.tal l.rom Loaded ../play/drifblim.rom Input(.tal): Output(.rom): -- Unused label: selection