Zandronum Chat @ irc.zandronum.com
#zandronum
Get the latest version: 3.0
Source Code

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003153Zandronum[All Projects] Bugpublic2017-06-10 19:302017-06-12 21:39
ReporterIvan 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionout of scope 
PlatformOSOS Version
Product Version3.0-beta 
Target VersionFixed in Version 
Summary0003153: Clientside Enter script gives different value for ConsolePlayerNumber in Named and ordinary scripts
DescriptionThe example wad demonstrates the problem in greater detail. Just create a server locally and join, you'll observe the problem.
Attached Files? file icon namedtest.pk3 [^] (428 bytes) 2017-06-10 19:30
c file icon namedtest.c [^] (214 bytes) 2017-06-10 19:33
? file icon namedtest2.pk3 [^] (656 bytes) 2017-06-10 19:51
txt file icon disassembly-incorrect.txt [^] (706 bytes) 2017-06-11 14:27 [Show Content]
txt file icon disassembly-correct.txt [^] (765 bytes) 2017-06-11 14:28 [Show Content]

- Relationships

-  Notes
User avatar (0017815)
Ru5tK1ng (updater)
2017-06-10 19:51

I think it is your compiler. I took your code and compiled it in namedtest2.pk3 and both numbers were 0 on the server.
User avatar (0017817)
Korshun (reporter)
2017-06-11 14:19
edited on: 2017-06-11 14:38

Disassembling the objects using bccobjdump shows that both the incorrect scripts and all correct scripts use the same correct instructions (Full disassemblies attached).

The only notable difference is that the incorrect object uses:
-- SFLG (8)
script=800 flags=clientside
script=1 flags=clientside

While the correct one has:
-- SFLG (8)
script=800 flags=clientside
script=-1 flags=clientside

Which means named script -1 doesn't get marked as clientside. It runs serverside, where ConsolePlayerNumber() returns -1.

Instead, nonexistent script 1 is marked clientside. This is a compiler bug.

User avatar (0017821)
Edward-san (developer)
2017-06-12 21:39

I guess it's [Not a bug], then.

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: Korshun

- Issue History
Date Modified Username Field Change
2017-06-10 19:30 Ivan New Issue
2017-06-10 19:30 Ivan File Added: namedtest.pk3
2017-06-10 19:33 Ivan File Added: namedtest.c
2017-06-10 19:51 Ru5tK1ng File Added: namedtest2.pk3
2017-06-10 19:51 Ru5tK1ng Note Added: 0017815
2017-06-11 14:19 Korshun Note Added: 0017817
2017-06-11 14:26 Korshun Note Edited: 0017817 View Revisions
2017-06-11 14:27 Korshun File Added: disassembly-incorrect.txt
2017-06-11 14:28 Korshun File Added: disassembly-correct.txt
2017-06-11 14:35 Korshun Note Added: 0017818
2017-06-11 14:36 Korshun Note Deleted: 0017818
2017-06-11 14:36 Korshun Note Edited: 0017817 View Revisions
2017-06-11 14:36 Korshun Note Edited: 0017817 View Revisions
2017-06-11 14:38 Korshun Note Edited: 0017817 View Revisions
2017-06-12 21:39 Edward-san Note Added: 0017821
2017-06-12 21:39 Edward-san Status new => closed
2017-06-12 21:39 Edward-san Resolution open => out of scope






Questions or other issues? Contact Us.

Links


Copyright © 2000 - 2020 MantisBT Team
Powered by Mantis Bugtracker