Add: - <array> for the data member in QComboBoxPrivate, amending d8e110054876b1cbf186e95bac5561a237ee1c13 - <limit> for numerical_limits in ibid, amending f0443984b84dea782ccd06dbce59808d55b15dbe Replace with fwd decl, because the header uses in-name-only, add include to .cpp file, if not already present: - QLineEdit - QPainter - QStyleOption subclasses Remove, because after the recent moves of code from the header to the .cpp, these are no longer needed: - qapplication.h - qcompleter.h - qevent.h - qstandarditemmodel.h (moved to .cpp) In qcombobox.cpp, besides the additions implied by the changes to the _p.h, remove the qcombobox.h include and rely on the transitive include though the _p.h, thus checking the _p.h file includes for completeness. This reliance on transitive includes is ok, because they're all from the same component. Add includes in the recommended style (<>, QtModule/, grouped by module, groups ordered by generality, ascending, within groups alphabetically), staying away from the unordered pre-existing includes as much as possible. Just move the <array> includes into the right spot, amending 4a7c76d4a5365343178ea29c8a8e9b8d8acb89b5. Not picking back to not cause problems with users that rely on transitive includes, even though this is a private header. Change-Id: I3aa42daf15f51df02dda4d3ab5aad1c9fb4dfa84 Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>