clugg / sm-json

A pure SourcePawn JSON encoder/decoder.
GNU General Public License v3.0
83 stars 8 forks source link

sourcemod plugin #37

Open Neptune196 opened 4 days ago

Neptune196 commented 4 days ago

//// sm_translator.sp // // C:\Program Files (x86)\steamcmd\l4d2sv\left4dead2\addons\sourcemod\scripting\include\json\helpers\decode.inc(399) : warning 203: symbol is never used: "is_array" // 399 | int pos, // --------------------^ // Code size: 69368 bytes // Data size: 12304 bytes // Stack/heap size: 18520 bytes // Total requirements: 100192 bytes // // 1 Warning. // // Compilation Time: 0.2 sec // ----------------------------------------

Press enter to exit ...

clugg commented 1 day ago

Hey @Neptune196, Thanks for raising this. I am surprised that I've never seen this warning before! This parameter has existed (and been unused) for as long as the function has. Would you mind sharing what version of spcomp you're compiling with and any relevant CLI arguments? I made a brief attempt but have been unable to reproduce the warning on my end, and it'd be nice to have warnings like these showing in the pipeline.

Also, I've pushed a fix to this branch - would you be able to confirm that this solves the warning and everything still works as expected? I'm sure it'll be fine, but I don't have any Source servers on hand to run json_test at the moment to confirm.

Thanks again!