search

RealNeGate / Cuik

A Modern C11 compiler (STILL EARLY)
MIT License
656 stars 32 forks source link

Fix some bugs, add small improvements #8

Closed flysand7 closed 2 years ago

flysand7 commented 2 years ago

Changes:

  1. If the user who is compiling Cuik already has vcvars set, vcvars won't need to be initialized again. Helps with extra error message like "vcvars not found" in case of Visual Studio Developer Prompt that doesn't keep vcvars in it's PATH by default.
  2. If vcvars weren't found in PATH a nice error message is displayed telling the user to either add it to PATH or use the Visual Studio Develeoper Command Prompt.
  3. /Z7 doesn't emit an extra pdb (that's not essential for normal debugging) also fuck visual studio
  4. Fixed a case of UB on calling strcmp with NULL argument that would trigger a crash on Windows if the argument to an option is not specified.
RealNeGate commented 2 years ago

Forgot to close this but yea, you made them to a branch that was outdated at that time so I took the changes in now: https://github.com/RealNeGate/Cuik/commit/9fb90dca0bea758cbe89c194a78fbbec7db9a0a5