UNSILENCE_ALL_PLAYERS

smoothice · Post by **smoothice** » Sat Mar 07, 2009 9:23 pm

I added a new Bazaar branch today (trunk) that has some code in it that enables UNSILENCE_ALL_PLAYERS... and all it does is unsilence everyone. I thought this would be a nice feature for Armagetron. If the console out was put into the language file, it could probably be merged. (If armagetronad-dev feels it would be worthy)

-=VcL.Rajinn · Post by **-=VcL.Rajinn** » Sat Mar 07, 2009 11:04 pm

nice job smoothice =)

Post by **Tank Program** » Sat Mar 07, 2009 11:06 pm

I get the feeling that if this was all you put in your branch, a .patch file would have done just as well.

smoothice · Post by **smoothice** » Sat Mar 07, 2009 11:08 pm

Oh, I thought everything was supposed to be put in a branch for merge ease. I can remove it if it's clutter.

Luke-jr also told me a while back to put every new feature in a branch.

Post by **epsy** » Sun Mar 08, 2009 12:02 am

smoothice wrote:If the console out was put into the language file, it could probably be merged. (If armagetronad-dev feels it would be worthy)

Well, do that.

And a branch is fine.

Website · Post by **Z-Man** » Sun Mar 08, 2009 12:27 am

epsy wrote:And a branch is fine.

Me agrees 100%. The cost of a branch from trunk on LP is minimal; they are all marked as dependant onto trunk, so they only use storage for stuff not on the trunk (and a bit of management). And with a branch, it's easier to add fixes before it gets merged.

Armagetron Forums

UNSILENCE_ALL_PLAYERS

UNSILENCE_ALL_PLAYERS

Re: UNSILENCE_ALL_PLAYERS

Re: UNSILENCE_ALL_PLAYERS