search

resibots / libdynamixel

C++ interface to the dynamixel actuators
GNU General Public License v2.0
9 stars 15 forks source link

Improve exception class UnpackError, gives more info. #40

Closed dogoepp closed 6 years ago

dogoepp commented 6 years ago

Now this exception also tells how many bytes were needed/expected versus how many actually arrived.

Since this exception is only meant to be raised from within the library, and since the API is not changed (only grown), I could not see any threat for the users of this library.

costashatz commented 6 years ago

@dogoepp Thanks! I'll have a look as soon as possible..

dogoepp commented 6 years ago

Thank you for your valuable review. I will make the changes. They require more testing before this can be merged.

dogoepp commented 6 years ago

I tested as much as I could : it compiles and the utility program still works. Merging