Patches item #3051663, was opened at 2010-08-23 12:45 Message generated for change (Comment added) made by mrpeach You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3051663...
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: externals Group: bugfix
Status: Pending Resolution: Accepted
Priority: 6 Private: No Submitted By: IOhannes m zmlnig (zmoelnig) Assigned to: Martin Peach (mrpeach) Summary: make slipdec reassemble arbitrary packages
Initial Comment: current slipdec doesn't work if the SLIP encoded package is not presented as a single list (which kind of defeats the idea of SLIP, which is to paketize a _stream_ of data)
with the attached patch, [slipdec] is able to decode a stream even if it was [1, 192 2, 3 4, 5 192, 6, 192(
before applying and committing the patch, please have a close look at it. for now, i just disabled the "valid" output (because it doesn't make so much sense anyhow, a SLIP stream is almost always valid) i know think that the flag could indicate whether there is still data waiting in the queue (and is updated after each message received)
----------------------------------------------------------------------
Comment By: Martin Peach (mrpeach)
Date: 2010-08-26 16:34
Message: Maybe it should output the size of the queue, so zero would mean a complete packet. Valid was supposed to mean that all the input was equivalent to bytes, i.e. no floating point, symbols, etc.
----------------------------------------------------------------------
You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3051663...