FairMQ icon indicating copy to clipboard operation
FairMQ copied to clipboard

Improve `bool ChangeState()` API

Open dennisklein opened this issue 1 year ago • 0 comments

Based on user feedback it seems easy to miss a return value of false (which usually means that the requested transition is illegal in the current state). Possible ideas to improve here were discussed:

  • Add [[nodiscard]]
  • Throw instead (would also naturally lead to error state if not handled - which the user is in favor for)

dennisklein avatar Aug 26 '22 12:08 dennisklein