izzy/UnrealEngineUWP
0
0
Fork 0
mirror of https://github.com/izzy2lost/UnrealEngineUWP.git synced 2026-03-26 18:15:20 -07:00
Code Issues Packages Projects Releases Wiki Activity

[UBT]

Browse Source 
* IWYUMode - Changed so IWYU always keep includes that are needed but redundant (they are transitively included by other needed includes). To remove redundant includes there is now an option -RemoveRedundant that brings back old behavior

#preflight skipped
#rb none

[CL 23888278 by henrik karlsson in ue5-main branch]
This commit is contained in:
henrik karlsson
2023-01-27 14:11:31 -05:00
parent 170dbfc280
commit e39ac31263
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
Engine/Source/Programs/UnrealBuildTool/Modes/IWYUMode.cs
View File

@@ -251,10 +251,10 @@ namespace UnrealBuildTool
public bool bNoTransitiveIncludes = false;
/// <summary>
/// Allow files to not add includes if they are transitively included by other includes
/// Will remove headers that are needed but redundant because they are included through other needed includes
/// </summary>
[CommandLine("-KeepRedundant")]
public bool bKeepRedundantIncludes = false;
[CommandLine("-RemoveRedundant")]
public bool bRemoveRedundantIncludes = false;
/// <summary>
/// Will skip compiling before updating. Handle with care, this is dangerous since files might not match .iwyu files
@@ -1055,8 +1055,8 @@ namespace UnrealBuildTool
}
}
// If keep redundant includes is set we don't remove the includes that are included through other includes
if (bKeepRedundantIncludes)
// We don't remove seen includes that are redundant because they are included through other includes that are needed.
if (!bRemoveRedundantIncludes)
{
List<string> ToReadd = new();
foreach (var Seen in Info.IncludesSeenInFile)