LuckyBro / sgfork

Automatically exported from code.google.com/p/sgfork
GNU General Public License v2.0
0 stars 0 forks source link

Item configuration files #66

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Overall description of the new feature:
Place bg_itemlist table into files like that : "configs/$classname.item".

Variables to control the new feature:
N/A

Original issue reported on code.google.com by kikc...@gmail.com on 3 Oct 2009 at 11:34

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Some additional fixes are applied in r114.
Still there are some stress tests to be performed:
1. Loading/saving items, weapons, configs.
2. Changing them via files and in game.

Also I would like to add some command like Conq` has told:
config [items|weapons|gravity] list
This one will printout current configuration.
And probably this command should be in CGAME instead of GAME to allow every 
player to 
know current configuration (but there is a side impact - some players may want 
to 
request this list too often to harm the game, so the game will be lagged).
And in this case I suggest the server variable "g_allowConfigListToClient" will 
be 
very helpful.

Original comment by igorpana...@gmail.com on 24 Nov 2009 at 1:58

GoogleCodeExporter commented 8 years ago
And as overall remark - this thing is huge.
Is it be possible to make it shorter?
Probably with some macro or array of structures like in server commands.
I mean there are too much typings of the same actions. During 
loading/saving/modifying/sending. And again too much code duplication - once we 
need to 
add new item/weapon member to structure - we have about 4 places where we can 
make a 
mistake.

Original comment by igorpana...@gmail.com on 24 Nov 2009 at 4:11