It's important to note that the cheat commands were supposed to be examples (as is said in the commit), which is why I tried to add the /give targeting initially. 

As for renaming commands, I can handle renaming some of them, but I still like instabuild. Fastbuild and instantbuild both sound too formal for me :s.

On 2014-05-10, at 4:30 AM, "Paul Chote" <notifications@github.com> wrote:

> In OpenRA.Mods.RA/DevCommands.cs:
> 
> > +
> > +			switch (name)
> > +			{
> > +				case "give":
> > +					var order = new Order("DevGiveCash", world.LocalPlayer.PlayerActor, false);
> > +					var cash = 0;
> > +
> > +					if (int.TryParse(arg, out cash))
> > +						order.ExtraData = (uint)cash;
> > +
> > +					Game.Debug("Giving {0} credits to player {1}.", (cash == 0 ? "cheat default" : cash.ToString()), world.LocalPlayer.PlayerName);
> > +					world.IssueOrder(order);
> > +
> > +					break;
> > +
> > +				case "removeshroud": world.IssueOrder(new Order("DevShroudDisable", world.LocalPlayer.PlayerActor, false)); break;
> You can avoid duplication and make these clearer by factoring out a IssueDevCommand(World world, string command) helper that does the boilerplate bits.
> 
> \
> Reply to this email directly or view it on GitHub.