AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

QDateTimeParser: simplify switch by exploiting fall-through

Browse Source 
Several cases all fell together; their code tested away one of them;
so putting it last and falling through to it avoids the need to test.

Change-Id: I0bf555710604a56faa61b60bfe5484d5c5a1ee6b
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
Edward Welbourne 2016-12-15 10:57:09 +01:00
parent 7243b7cbf2
commit 5bc814ac69
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/corelib/tools/qdatetimeparser.cpp
View File

@ -1461,15 +1461,13 @@ QDateTimeParser::FieldInfo QDateTimeParser::fieldInfo(int index) const
case MinuteSection:
case Hour24Section:
case Hour12Section:
case YearSection:
case YearSection2Digits:
ret |= AllowPartial;
Q_FALLTHROUGH();
case YearSection:
ret |= Numeric;
if (sn.type != YearSection) {
ret |= AllowPartial;
}
if (sn.count != 1) {
if (sn.count != 1)
ret |= FixedWidth;
}
break;
case MonthSection:
case DaySection: