[PATCH] D36201: Merge manifest namespaces.

[Zachary Turner via Phabricator via llvm-commits]

zturner added a comment.

Only 1 nitpick, I think reid and rui covered the rest pretty well.



================
Comment at: llvm/lib/WindowsManifest/WindowsManifestMerger.cpp:584-585
+    if (Def->prefix &&
+        (std::find(RequiredPrefixes.begin(), RequiredPrefixes.end(), Def) ==
+         RequiredPrefixes.end())) {
+      if (Def == Node->nsDef)
----------------
`!llvm::is_contained(RequiredPrefixes, Def)` will allow you to get this all on a single line.  Similar applies to other places in the file.


https://reviews.llvm.org/D36201