[CSV-55] Replace while(true)-loop in CSVParser.getRecord() with do-while-loop

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.0
• Fix Version/s: 1.0
• Component/s: Parser
• Assignee: Unassigned
• Reporter: Benedikt Ritter
• Created: 12/Mar/12 13:57
• Updated: 14/Aug/14 20:06
• Resolved: 12/Mar/12 15:13

Description

The current implementation of getRecords() uses a while(true) loop, that gets canceled by an if statement:

while (true) {
    reusableToken.reset();
    lexer.nextToken(reusableToken);
    // omitted
    if(reusableToken.type != TOKEN) {
        break;
    }
}

This should be replaced by a do-while-loop:

do {
    reusableToken.reset();
    lexer.nextToken(reusableToken);
    // omitted
} while (reusableToken.type == TOKEN);