LineReader icon indicating copy to clipboard operation
LineReader copied to clipboard

Harmonization of the FileReader

Open johnjohndoe opened this issue 13 years ago • 0 comments

The use and meaning of local variables in FileReader readline and readLineBackwards are not consistent. Especially, the character position at which the reader sits after one while loop in readLineBackwards is another then it is in readLine. That makes it difficult to use the class without knowledge about the inner semantics. Further, readLineBackwards can be simplified by getting rid of the redundant if-clauses used before and within the while loop. That needs some brain power of course :-}

johnjohndoe avatar May 03 '11 16:05 johnjohndoe