Please do, I would like to see what you fixed.
quick update (just got home a bit ago), still fixing things
so far i've came across:
- excessive amounts of singular letter variables, this doesn't speed things up or slow things down and just confuses everyone
- unneeded conversions (24h time -> 12h time), if you can't decipher 24 hour time, you shouldn't have passed 5th grade
- bad error traps
- limiting delays to 6 digit numbers (echo(mFloor(999999+1)), try it)
- not exporting actual preferences correctly (variable pattern leaves out things)
- not even using $Pref (let the game handle it, if it crashes then this should be left what it was)
- commands limited to super admins with no setting to change it
EDIT: r u srs
EDIT2: add "not even using certain prefs" to that list
EDIT3:
forgettons of for loops when you save, viso isn't checking for isPlanted on bricks to see if they're temporary, but instead loops through the clientGroup. he is checking, but still looping regardless (???????)
imagine 24 players with 20000+ bricks, yeah
(
Server_AS1_nextBrick)
EDIT4: you're saving mechanism is so confusing it's giving me a headache
EDIT5: you also know you can reuse file objects right? you don't have to delete them, just close them.
EDIT6: what is with you and your useless if statements lol
EDIT7: add "using global variables where local variables would suffice" to the list as well
EDIT8: so those of you having auto saves running at midnight really need to double check things, viso believed 00:00 was 1 am. it is 12am.
EDIT8.1: i've had to change/patch so many things i sincerely hope the original version is CRC'd
EDIT8.2: this is how you convert 24h to 12h, it is seriously this easy