search

QW-Group / mvdsv

MVDSV: a QuakeWorld server
GNU General Public License v2.0
58 stars 56 forks source link

TRAPS: Implement Map_Extension, GetExtField and SetExtField #117

Closed dsvensson closed 4 weeks ago

dsvensson commented 1 year ago

As this is completely separate from the alpha related protocol changes, perhaps it's easier to deal with this in a separate PR even if it doesn't really do anything on its own.

tcsabina commented 1 year ago

Nano, The progs.h in this PR, I think is a good candidate to move to the new qwprot repo. The on in the ezquake repo and the one here in mvdsv are almost identical. I haven't looked into which one should we take to be the main one in qwprot. Do you want to play around with it? Making a PR in qwprot to tack this progs.h change?

dsvensson commented 1 year ago

What stands out is edict_t which in mvdsv has entvars_t* while in ezq is inlined in a nice little horror story. Kind of ties into bumping mvdsv in ezquake which has all sorts of benefits, but is a pretty large task it feels like.

dsvensson commented 4 weeks ago

Not going to pursue this feature.