Stéphane Raimbault
Stéphane Raimbault
No but it could be a good fit for promodbus spin-off before merging it later in libmodbus...
The proper way to do that would be to only use bit instead of byte for storage. Not currently on my TODO list.
OK, notes added to next version of the documentation.
Related to https://github.com/stephane/libmodbus/pull/646
No answer from reporter.
I didn't see such issue before. The internal code is: ```c if (ctx->s != -1) { tcsetattr(ctx->s, TCSANOW, &ctx_rtu->old_tios); close(ctx->s); ctx->s = -1; } ``` Do you see something missing?
It was fun to piss off the fascists but the implementation is not realistic. It will just try to rename the main branch.
See updated `is_compliant` branch.
It is fixed by https://github.com/stephane/libmodbus/commit/db1cbc593501590eef16bbf9e0746290ac9116b9?
Could you add log traces to in the libmodbus close function to be sure it's called (not early exit in your app) and to see the value of `ctx->s`?