search

astuff / kvaser_interface

A Kvaser CAN interface ROS abstraction layer.
MIT License
84 stars 52 forks source link

Fix/memory management #16

Closed JWhitleyWork closed 5 years ago

JWhitleyWork commented 5 years ago

Already approved by @Daniel-Stanek and @zoakesAStuff. Will force-merge when @rgunjala93 completes testing.

JWhitleyWork commented 5 years ago

Re: fixed bug, see 10cd698. Since handle int value is actually initialized to a valid value (-1) and returned as an actual handle number later in the code, the check at the beginning of open() is unnecessary.