schellingb / dosbox-pure

DOSBox Pure is a new fork of DOSBox built for RetroArch/Libretro aiming for simplicity and ease of use.
GNU General Public License v2.0
731 stars 61 forks source link

Compilation error since 91f3504 #409

Closed DrUm78 closed 1 year ago

DrUm78 commented 1 year ago

A compilation error now occurs because of this commit: https://github.com/schellingb/dosbox-pure/commit/91f35042ce56b7c9b33b6fd282ad6e60611d4510

Compiling src/dbp_network.cpp ...
src/dbp_network.cpp: In function ‘void ModemBufAddS(CModemBuf&, const char*, ...)’:
src/dbp_network.cpp:781:26: error: ‘va_start’ was not declared in this scope
  781 |  char m[128]; va_list v; va_start(v, fmt); int len = vsnprintf(m, sizeof(m), fmt, v); va_end(v);
      |                          ^~~~~~~~
src/dbp_network.cpp:781:87: error: ‘va_end’ was not declared in this scope
  781 |  char m[128]; va_list v; va_start(v, fmt); int len = vsnprintf(m, sizeof(m), fmt, v); va_end(v);
      |                                                                                       ^~~~~~
make: *** [Makefile:235: build/release/src~dbp_network.cpp.o] Error 1
schellingb commented 1 year ago

Oh, thanks for noticing! I'm still a bit far from release of the next version so I've been testing mostly on Windows and Android. Interestingly both compile fine without including but it should be more correct now.

Thanks again for testing and reporting this. Are you going to try the new netplay stuff? If you do, you'll need a very recent version of RetroArch, too. The support for this was merged just 3 days ago. Please report back :-)

DrUm78 commented 1 year ago

Compilation now works fine, thanks. 👍

I cannot test the netplay stuff as my target platform does not support it (FunKey S handheld with no connectivity).