Make it clearer what's happening with a fall-through.
An if () {...}'s close-brace was hiding on the end of a break; line, an idiom used in several places for braces that existed to contain a case (for the sake of local declarations). This made it hard to see that there was an if() whose else was the resulting (commented) fall-through. So put that close-brace on the comment's line and make the comment clearly indicate it's an else. Change-Id: Ie7e7c7063bef96536d6231297b083fc384f2363e Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
parent
1967accb64
commit
d8c0bd4207
@ -765,8 +765,8 @@ int QDateTimeParser::parseSection(const QDateTime ¤tValue, int sectionInde
|
||||
} else {
|
||||
state = Intermediate;
|
||||
}
|
||||
break; }
|
||||
// fall through
|
||||
break;
|
||||
} // else: fall through
|
||||
case DaySection:
|
||||
case YearSection:
|
||||
case YearSection2Digits:
|
||||
|
Loading…
x
Reference in New Issue
Block a user