MantisBT - Zandronum
View Issue Details
0001804Zandronum[All Projects] Suggestionpublic2014-05-17 21:372021-01-10 20:35
needs testingopen 
0001804: cvar difference description for dmflags
oh_no duskie has been doing science again, TAKE COVER.

Basically I hacked in a little algorithm thing that shows exactly what changed in dmflags/compatflags/whatever when the server prints the "dmflags changed to blah". If too many flags change (i.e. more than 5) it just prints the amount of flags that changed so that the chat doesn't get flooded.


*** MAP01: entryway ***

sv_nojump 1
dmflags changed to 65536 (sv_nojump ON)
dmflags 123
dmflags changed to 123 (7 flags changed)
dmflags 1234
dmflags changed to 1234 (sv_nohealth OFF, sv_oldfalldamage OFF, sv_spawnfarthest ON, sv_noexit ON)
compat_limited_airmovement 1
compatflags changed to 131072 (compat_limited_airmovement ON)

Diff attached. What do you guys think?
No tags attached.
diff dmflags-data.diff (7,253) 2014-05-17 21:37
Issue History
2014-05-17 21:37DuskNew Issue
2014-05-17 21:37DuskStatusnew => assigned
2014-05-17 21:37DuskAssigned To => Dusk
2014-05-17 21:37DuskFile Added: dmflags-data.diff
2014-05-17 21:37DuskStatusassigned => needs review
2014-05-17 22:33WatermelonNote Added: 0008803
2014-05-18 07:42FritsNote Added: 0008807
2014-05-18 07:42FritsNote Deleted: 0008807
2014-05-19 05:07ZzZomboNote Added: 0008810
2014-06-21 13:31Torr SamahoNote Added: 0009612
2014-06-21 13:31Torr SamahoStatusneeds review => feedback
2021-01-10 20:34Torr SamahoNote Added: 0021587
2021-01-10 20:35Torr SamahoStatusfeedback => needs testing

2014-05-17 22:33   
This is pretty cool
2014-05-19 05:07   
I think if there more than 5 flags changed it should list the first 5 and then just add a remark about that not all were listed. Would be neat IMO.
Torr Samaho   
2014-06-21 13:31   
As we found out during an IRC discussion a while ago, the workaround of using CVAR_Dummy to determine whether a flag is of type FFlagCVar doesn't work with recent ZDoom versions anymore. Before we can add this, we need to find a way to identify these flags that also works with the current ZDoom versions.
Torr Samaho   
2021-01-10 20:34   
I added Kaminski patch for this that is building on Dusk's original patch.