MantisBT - Zandronum
View Issue Details
0002858Zandronum[All Projects] Bugpublic2016-10-06 04:072018-09-30 21:59
unknownna 
Edward-san 
lowtextalways
closedfixed 
3.0-beta 
3.03.0 
0002858: compatflags2 is changed twice on client after changemap map changes in 3.0
See attached screenshot. Not sure whether this is a bug, but noticed the difference and decided to report it.
No tags attached.
png difference.png (146,226) 2016-10-06 04:07
/tracker/file_download.php?file_id=1907&type=bug
png
Issue History
2016-10-06 04:07unknownnaNew Issue
2016-10-06 04:07unknownnaFile Added: difference.png
2016-10-06 04:23WaTaKiDNote Added: 0015766
2016-10-06 06:44Edward-sanNote Added: 0015767
2016-10-06 06:44Edward-sanAssigned To => Edward-san
2016-10-06 06:44Edward-sanStatusnew => assigned
2016-10-06 06:52Edward-sanNote Edited: 0015767bug_revision_view_page.php?bugnote_id=15767#r9592
2016-10-06 06:52Edward-sanStatusassigned => needs review
2016-10-08 17:10Edward-sanNote Added: 0015808
2016-10-08 17:10Edward-sanStatusneeds review => needs testing
2016-10-09 17:56Torr SamahoProduct Version3.0-beta => 3.0
2016-10-09 18:40Edward-sanProduct Version3.0 => 3.0-beta
2016-10-09 18:40Edward-sanTarget Version => 3.0
2016-10-09 19:20unknownnaNote Added: 0015858
2016-10-09 21:07unknownnaStatusneeds testing => resolved
2016-10-09 21:07unknownnaFixed in Version => 3.0
2016-10-09 21:07unknownnaResolutionopen => fixed
2018-09-30 21:59Blzut3Statusresolved => closed

Notes
(0015766)
WaTaKiD   
2016-10-06 04:23   
imo those messages shouldnt even print if the flags dont actually change, would cut down on a wee bit of spam on long running servers
(0015767)
Edward-san   
2016-10-06 06:44   
(edited on: 2016-10-06 06:52)
It's the server which prints those strings and shares them to the clients. It seem the double compatflags2 problem arises because of this code change. The fix is easy enough.

[edit] Done.

(0015808)
Edward-san   
2016-10-08 17:10   
Merged to 3.0 upstream:'https://bitbucket.org/Torr_Samaho/zandronum/commits/26dc34682512 [^]'
(0015858)
unknownna   
2016-10-09 19:20   
The double message is gone. Good work, Edward-san.