L1nq0 commented on PR #109: URL: https://github.com/apache/dubbo-hessian-lite/pull/109#issuecomment-5596075545
@RainYuY You are right, and I could reproduce it: with the old check, round tripping a plain bean through a stream that hands out one byte per read() was rejected as a bad field count. The rework in 1a43ec9 swaps the order. Every name is read before any array is allocated, and the allocation follows the list length, not the wire count. A negative count fails outright, and a definition whose input stops midway throws with the number of names it reached. A complete definition still parses when it arrives one byte at a time, and a bean round trips through such a stream. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
