21 Jul 13, 01:08AM
(This post was last modified: 21 Jul 13, 01:17AM by TheNihilanth.)
(21 Jul 13, 12:54AM)Luc@s Wrote:Oh wow, how far superior seems your code.... :)(21 Jul 13, 12:03AM)TheNihilanth Wrote: In fact, thinking it better, I can't really accept that.
No way. Too much time for such a simple feature to add!
It's kind of disappointing to see 70+ pages with ideas most them not implemented.
But that is going to change. In my limited spare time I'll code the ideas (from the Ideas thread) I like and think can be funny/etc/etc (I have some in progress) and post the patches.
If you have by any chance and artistic skill send me pm if you want to help in some way.
To begin with, below is the code patch which allows the addbot and addnbot commands to add a bot which spawns with a specific weapon.
Valid weapon parameters are (case-insensitive): knife, pistol, carbine,
shotgun, subgun, sniper, assault, grenade and akimbo. Default is 'assault'.
Example:
[SELECT ALL] Code:/addbot CLA bad kkk sniper
[SELECT ALL] Code:/addnbot 300 CLA bad shotgun
[For Rev. 7505]
Sorry but i can't accept that either.
The main reason is that it was not implemented because it's not our priority. We would like next version to be released ASAP, hence feature freeze.
Actually the code for the following features is already done (it's somewhere on my computer), (however untested)
- spawn a bot with the desired primary weapon (which can be later changed, once he is dead)
- spawn a bot at a desired location (x, y)
- disable auto respawn for a bot
- prevent a bot from attacking other bots (in this case, he will only attack the local player)
The other reason is that your code can't be accepted now :
- You're using iostream while using it for 1 single little feature is useless.
- Actually there is no point storing the primary weapon as a string
- some pieces of your code could be reduced to one line
- i doubt that void ReAddBot(CStoredBot *bot) will work as you wish because it doesn't send correct weapon names (but their IDs (GUN_SNIPER, etc.) as a 1 char string (...))
- you're allowing non primary weapons as primary weapons !!
- etc.;
- anyway my code is ready and offers many more possibilites
Maybe i sound harsh, i'm happy that you're willing to help, but /you/ did sound harsh.
Well this is my implementation I did see on the ideas thread, if there is another implementation you talked I don't know about that. Sorry not to fulfill your desires, it not my intention to do so, just to contribute in some way, if lucas had told he was implementing it I would have saved me the bother doing it for myself.
The code can be iterated and it is a good starting point, considering it took me lees than 3 days to what (it seems) it took you more than 3+ years. And if we are going to talk about 'how to code' Why don't you put your name to changes you make so I can evaluate you too?
I didn't mean to sound harsh maybe you are too sensible and took it this way.