Page MenuHomeWildfire Games

Fix no-pch build following renderer options refactoring rP22610

Authored by wraitii on Aug 4 2019, 9:30 PM.



LOGWARNING requires CLogger following

See refactoring of renderer options in rP22610.

Test Plan

Compile without precompiled headers.

Diff Detail

rP 0 A.D. Public Repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

wraitii created this revision.Aug 4 2019, 9:30 PM
Owners added a subscriber: Restricted Owners Package.Aug 4 2019, 9:30 PM
Vulcan added a comment.Aug 4 2019, 9:32 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build:

elexis accepted this revision.Aug 4 2019, 9:37 PM
elexis added a subscriber: elexis.

The inclusion is correct, just not in alphabetic order.

Seems like this file is in reverse alphebtic order, so the insertion is consistent :P

(One can be lazy and copy&paste headers to

Also newline for `RenderingOptions.h, so that it stands out as the header of the current file, see

This revision is now accepted and ready to land.Aug 4 2019, 9:37 PM
elexis retitled this revision from Fix no-pch build following rP22610 to Fix no-pch build following renderer options refactoring rP22610.Aug 4 2019, 9:40 PM
elexis edited the summary of this revision. (Show Details)

Our sorting is case-insensitive so the file was actually ordered correctly before. Will fix when committing.

This revision was landed with ongoing or failed builds.Aug 4 2019, 10:08 PM
This revision was automatically updated to reflect the committed changes.