search

APN-Pucky / tyrant_optimize

MIT License
15 stars 7 forks source link

Add support of Visual Studio #75

Closed vakomash closed 7 months ago

APN-Pucky commented 7 months ago

I am testing it on linux in vscode with the integration, but gcc-13 complains about missing braces with the added if(tokenParsed). So it should be if(tokenParsed) { ... }:

[build] /usr/bin/g++-13 -DBOOST_ATOMIC_DYN_LINK -DBOOST_ATOMIC_NO_LIB -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_FILESYSTEM_NO_LIB -DBOOST_REGEX_DYN_LINK -DBOOST_REGEX_NO_LIB -DBOOST_SYSTEM_DYN_LINK -DBOOST_SYSTEM_NO_LIB -DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_NO_LIB -DBOOST_TIMER_DYN_LINK -DBOOST_TIMER_NO_LIB -DNQUEST -DTYRANT_OPTIMIZER_VERSION=\"\" -I/home/apn/git/tyrant_optimize/extern/hPMML/include -Wall -Werror -Ofast -g -std=gnu++14 -MD -MT CMakeFiles/tuo.dir/tyrant_optimize.cpp.o -MF CMakeFiles/tuo.dir/tyrant_optimize.cpp.o.d -o CMakeFiles/tuo.dir/tyrant_optimize.cpp.o -c /home/apn/git/tyrant_optimize/tyrant_optimize.cpp
[build] /home/apn/git/tyrant_optimize/tyrant_optimize.cpp: In function ‘DeckResults run(int, const char**)’:
[build] /home/apn/git/tyrant_optimize/tyrant_optimize.cpp:3270:12: error: suggest explicit braces to avoid ambiguous ‘else’ [-Werror=dangling-else]
[build]  3270 |         if (!tokenParsed)
[build]       |            ^
[build] cc1plus: all warnings being treated as errors
[build] ninja: build stopped: subcommand failed.
[proc] The command: /usr/bin/cmake --build /home/apn/git/tyrant_optimize/build --config Debug --target all -- exited with code: 1
APN-Pucky commented 7 months ago

LGTM, thanks!