08.01.2010, 13:57
It's good, much better than my one that was released, but a few pointers.
1)A #define to alow users to change whether the ramp shows for all players or not, rather than releasing two seperate versions.
2)Sort out your indentation, it's really bad.
3) Instead of using seperate commands for "/my ramp 1-4" you could either use a dialog under a "myramp" command or use "/myramp" andd add the numbers as parameter's using sscanf.
Also on a not so important note, I reccommend you use ZCMD rather than strcmp, more people need to start switching to faster methods.
Note, I haven't actually tested the script, just gone through the code.
I like your idea about using different ramps, I just think it could have been coded better.
1)A #define to alow users to change whether the ramp shows for all players or not, rather than releasing two seperate versions.
2)Sort out your indentation, it's really bad.
3) Instead of using seperate commands for "/my ramp 1-4" you could either use a dialog under a "myramp" command or use "/myramp" andd add the numbers as parameter's using sscanf.
Also on a not so important note, I reccommend you use ZCMD rather than strcmp, more people need to start switching to faster methods.
Note, I haven't actually tested the script, just gone through the code.
I like your idea about using different ramps, I just think it could have been coded better.