Özkan Sezer

> Quakespasm Spike introduced the complete thing in this commit: > > [Shpoike/Quakespasm@f6d31e9](https://github.com/Shpoike/Quakespasm/commit/f6d31e9db928cb058398ba9002ca4b2094a2926f) Applying that commit to qs doesn't seem to work correctly - e.g. rockets doesn't light the path...

This #1554 and the next #1555 be combined and simplified into the following, can they not? EDIT: Even better, `< 0` should be `alloc.alloc_buffer) { ```

(See comment at https://github.com/nothings/stb/pull/1554#issuecomment-1848985016 )

This is a duplicate of https://github.com/nothings/stb/pull/1312

You are neglecting that autotools can perfectly be used to build for windows, with mingw or cygwin. Surely not msvc, but msvc isn't mingw or cygwin. As for cmake: what...

FWIW, I plan to apply something like the following to my fork (https://github.com/sezero/libmodplug) ```diff diff --git a/CMakeLists.txt b/CMakeLists.txt index 41ff174..c1b0fe0 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -176,7 +176,15 @@ install(FILES...

> mingw doesn't appear to need `-luser32` It's among its default link libraries, but not if one uses -nostdlib

Pushed https://github.com/sezero/libmodplug/commit/ff19d10348ca6ebe995d00f6880b79a5b34b2e4a in my fork.

One bad thing about this (also with my patch in my fork, and also with the existing unpatched situation) is that `-lstdc++` may actually be the wrong library: think of...

> ensuring you can include the stdlib that you want is one of the major uses of "mmacosx-version-min". Here is a discussion of a similar/but slightly different problem in another...