demntor / pymodbus

Modbus Python release (tools included)
0 stars 0 forks source link

Review changes regarding RTU frame initialization #46

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Purpose of code changes on this branch:

When the library starts reading from the serial line, it might
happen that only a part of a frame is received. In this case,
the calculation of the frame length based on the function code
will not work.

I've checked in a fix on the rtu branch which discards a
frame fragment in the case that the function code cannot
be associated with a ModbusPDU.

When reviewing my code changes, please focus on:

the stability of my solution. I'm not sure if advancing the
frame will always solve the problem.

After the review, I'll merge this branch into:
/trunk

Original issue reported on code.google.com by albert.k...@gmail.com on 4 Mar 2011 at 9:51

GoogleCodeExporter commented 8 years ago

Original comment by Bashw...@gmail.com on 4 Mar 2011 at 3:44