Staffan, Two bytes len prefix is a part of JDP protocol specification, so I would prefer to write it explicitly.
But changed to *writeUTF()* as recommended. see: http://cr.openjdk.java.net/~dsamersoff/8015604/webrev.02/ -Dmitry On 2013-06-04 11:48, Staffan Larsen wrote: > Looks good. > > Even better would have been to use DataOutputStream.writeUTF(). > > /Staffan > > On 3 jun 2013, at 23:33, Dmitry Samersoff <dmitry.samers...@oracle.com> wrote: > >> Hi Everybody, >> >> The problem: >> >> Code uses string length rather than byte array length to write length >> field of JDP packet entry. So if an entry contains non-ASCII characters >> code produces invalid packet. >> >> >> http://cr.openjdk.java.net/~dsamersoff/8015604/webrev.01/ >> >> -Dmitry >> >> >> -- >> Dmitry Samersoff >> Oracle Java development team, Saint Petersburg, Russia >> * I would love to change the world, but they won't give me the sources. > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.