search

R2Northstar / NorthstarLauncher

Launcher used to modify Titanfall 2 to allow mods to be loaded
MIT License
271 stars 125 forks source link

correct includes everywhere #732

Closed Jan200101 closed 2 weeks ago

Jan200101 commented 3 weeks ago

lots of headers have dependencies that are fulfilled through the precompiled header either directly or indirectly.

For the sake of code clarity translation unit dependencies should be properly marked.

Jan200101 commented 3 weeks ago

So there's a few changes in there as well such as the removal of a few cases of NOTE_UNUSED( );, changes of assert_msg to assert(), as well as introduction of [[maybe_unused]] and given my limited C++ knowledge I'm trying to figure out how that relates to the core change of the PR which is changing the includes ^^"

those were extra changes that I forgot to remove Will do so ASAP