Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • batsim batsim
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 38
    • Issues 38
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • batsim
  • batsimbatsim
  • Issues
  • #87
Closed
Open
Issue created Oct 16, 2018 by Millian Poquet@mpoquetOwner

Reduce error message redundancy in the protocol

The protocol.cpp file is full of redundant error messages, stating that the received message is invalid.
Such error messages should be kept, but a cleaner code would not repeat the prefix at each line...

One way to improve this is to:

  • Use asserts that throws an exception on error. Only use the meaningful message there without prefix.
  • Surround protocol functions calls with try/catch. Rethrow errors with the desired prefix.
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking