CommandPtr didn't already have AndThen, DeadlineFor, AlongWith, and RaceWith decorators that take multiple commands, so I didn't add equivalents of those to Command. (I did add the single command argument versions, though) This unfortunately affected the DeadlineFor test, so if people want me to add those as well in this PR then I'm fine with doing that. (Or if they want to do it themselves in a separate PR, that's cool too)
Defines consistent execution order of Until, OnlyWhile, DeadlineFor, AlongWith, RaceWith, where the textual order of arguments matches the order of processing. (The decorated command is processed first (execute() run and isFinished() checked, optionally running end()), then the arguments to the decorator)
Notes:
CommandPtrdidn't already haveAndThen,DeadlineFor,AlongWith, andRaceWithdecorators that take multiple commands, so I didn't add equivalents of those toCommand. (I did add the single command argument versions, though) This unfortunately affected theDeadlineFortest, so if people want me to add those as well in this PR then I'm fine with doing that. (Or if they want to do it themselves in a separate PR, that's cool too)Until,OnlyWhile,DeadlineFor,AlongWith,RaceWith, where the textual order of arguments matches the order of processing. (The decorated command is processed first (execute()run andisFinished()checked, optionally runningend()), then the arguments to the decorator)