Closed GoogleCodeExporter closed 9 years ago
Excellent catch! I can't believe I missed this! Thank you so much. I will get a
fix out this weekend.
Original comment by joshuadmorris@gmail.com
on 14 Nov 2011 at 3:12
While looking into a local fix for this, I realized that there is a fundamental
problem with storing the packet bytes in a byte array. Since Java limits
arrays to (int), a message with a 64 bit length cannot be allocated into an
array of bytes. In parseHeader() we're casting the dataLength to an int which
isn't going to work long term. In the short term I've changed my local copy to:
else if ( dataLength == 127 )
{
throw new RuntimeException("Library cannot support Long byte arrays");
}
Original comment by nw...@detroitsci.com
on 14 Nov 2011 at 3:44
Just noticed there is a snippet of code already existing that throws an
exception if the size exceeds MAX_INT.
Original comment by nw...@detroitsci.com
on 14 Nov 2011 at 4:27
Sorry for the delay, I am now working on a fix. The holidays are playing havoc
with my schedule.
Original comment by joshuadmorris@gmail.com
on 2 Dec 2011 at 1:19
I still need to fix a couple other issues and run tests, but this should be
working in trunk now.
Original comment by joshuadmorris@gmail.com
on 2 Dec 2011 at 4:44
Original comment by joshuadmorris@gmail.com
on 31 Jan 2012 at 6:32
Original issue reported on code.google.com by
nw...@detroitsci.com
on 11 Nov 2011 at 8:25