Skip to content

- fixed two memory leaks (unnecessary strdup's); - removed ser_flush calls that were causing the driver to desync; - refactored code, mostly bit mainpulation functions; - other minor improvements in source code. - #511

Merged
aquette merged 1 commit into
networkupstools:masterfrom
rpvelloso:master
Feb 25, 2019

Conversation

@rpvelloso

Copy link
Copy Markdown
Contributor
  • removed ser_flush calls that were causing the driver to desync;
  • refactored code, mostly bit mainpulation functions;
  • other minor improvements in source code.

- removed ser_flush calls that were causing the driver to desync;
- refactored code, mostly bit mainpulation functions;
- other minor improvements in source code.
@rpvelloso

Copy link
Copy Markdown
Contributor Author

Does anybody have any idea when this is going to the master branch? @clepple?

@aquette

aquette commented Feb 25, 2019

Copy link
Copy Markdown
Member

hi @rpvelloso
sorry for the eternity it took, I'm about to merge your PR.
Side notes:

thanks for your contribution

@aquette
aquette merged commit 7de6654 into networkupstools:master Feb 25, 2019
aquette pushed a commit that referenced this pull request Feb 26, 2019
jimklimov pushed a commit to jimklimov/nut that referenced this pull request Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants