Yaribz / SPADS

SpringRTS Perl Autohost for Dedicated Server
GNU General Public License v3.0
15 stars 13 forks source link

Update PerlUnitSync.pm #59

Closed abma closed 1 year ago

Yaribz commented 1 year ago

Thanks, actually I would like to keep backward compatibility with old Spring versions, that's why the unitsync header included here is an old one. Given the way SPADS interfaces with unitsync (through a compiled SWIG wrapper), any unitsync function appearing in this file will make SPADS only accept unitsync versions which implement these functions (SPADS only uses a small subset of unitsync functions anyway).

Moreover, I have a more modern way to interface SPADS with unitsync (based on libffi) ready to be deployed if needed. It won't have such restrictions and won't need access to the headers like SWIG.

Yaribz commented 1 year ago

The new unitsync interface implemented in commit a7a6f9c is now compatible with all unitsync versions.