Manuel Lauss

Interesting, I get 2, the captions even read "Menacing electronic buzz" whey they are spawned.

> Tried to build on an M1 Mac and thought I would note a few issues here, unclear if some of these are clang related or MacOS's STL at this...

> /Users/jonathan/code/TheForceEngine/TheForceEngine/TFE_FileSystem/filestream.h:51:7: note: candidate function not viable: no known conversion from 'size_t *' (aka 'unsigned long *') How many bits does an "unsigned long" have on Apple? A bit strange...

> > The sprintf->snprintf replacement should be fine, with the understanding this is an ANSI C99 extension. > > For the size_t thing, size_t is defined as the type returned...

i.e.: ```diff --git a/TheForceEngine/TFE_Memory/memoryRegion.cpp b/TheForceEngine/TFE_Memory/memoryRegion.cpp index cf859d22..59565001 100644 --- a/TheForceEngine/TFE_Memory/memoryRegion.cpp +++ b/TheForceEngine/TFE_Memory/memoryRegion.cpp @@ -80,11 +80,11 @@ struct MemoryRegion { char name[32]; MemoryBlock** memBlocks; - size_t blockArrCapacity; + u64 blockArrCapacity; -...

> Found one more instance of this, might u64 be a little large? As far as preserving compatibility with 32-bit systems, assuming this is actually a concern? Changing them to...

> [...] there is another linker error with SDL now, CMAKE seems to be handing the include path just fine but something is going wrong with the linker library paths...

> Oops also, these changes you've got diffs up of, are these checked into a branch/fork anywhere, or just posted here? I've got a macOS box, happy to pull it...

Oh and also #282 for big-endian machines. I like it, but I can't test it since I have no BE userland for my MIPS toys.

just bikeshedding here: - Maybe add an envvar too? i.e. "TFE_PORTABLE=1 ./theforceengine", - Maybe make it configurable as default behaviour (thinking about a flatpak here again where everyhing the needs...