@RN
@RN_
@StormyCloud
@T3s|4
@T3s|4_
@eyedeekay
@postman
@zzz
+FreefallHeavens
+Xeha
+acetone
+cumlord
+hk
+poriori
+profetikla
+r00tobo
+snex
+uop23ip
An0nm0n
Arch
Danny
DeltaOreo
Irc2PGuest17611
Irc2PGuest1833
Irc2PGuest64356
Irc2PGuest91362
Leopold_
Meow
Nausicaa
Onn4l7h
Onn4|7h
anu3
ardu
boonst
deadface
dickless
dr|z3d
itsjustme
mareki2pb
not_bob_afk
onon_1
orignal_
pisslord
qend-irc2p
r3med1tz-
radakayot_
segfault
shiver_sc
simprelay
solidx66_
thetia
u5657
woodwose
dr|z3d
does this look saner, or am I making the same mistake here?
dr|z3d
b32 = reader.readLine();
dr|z3d
if (b32 == null) {return;}
zzz
same
dr|z3d
yeah, figured.
dr|z3d
so, totally pointless then, as you say. this should cover it: while((b32 = reader.readLine()) != null) {
dr|z3d
well, not totally pointless, as it prevents the issue from triggering, but not the correct fix.
dr|z3d
what about checking the filesize instead?
dr|z3d
int fileLength = (int) file.length();
dr|z3d
if (fileLength == 0) {return;}
zzz
better
zzz
but the reason for a ticket was that this is low priority, not for today, so if you want my analysis you'll have to sit tight ))
dr|z3d
sure, I don't think it's something that'll affect most people.
dr|z3d
if there's no file at all, non-issue. only zero length file, or presumably a file containing no b32s.
dr|z3d
got a stacktrace zzz, bug report updated.