modularize - Update main comment.
llvm-svn: 189968
diff --git a/clang-tools-extra/modularize/Modularize.cpp b/clang-tools-extra/modularize/Modularize.cpp
index 8b68e48..db0bb69 100644
--- a/clang-tools-extra/modularize/Modularize.cpp
+++ b/clang-tools-extra/modularize/Modularize.cpp
@@ -79,21 +79,17 @@
//
// Some ideas:
//
-// 1. Add options to disable any of the checks, in case
+// 1. Check for and warn about "#include" directives inside 'extern "C/C++" {}'
+// and "namespace (name) {}" blocks.
+//
+// 2. Omit duplicate "not always provided" messages
+//
+// 3. Add options to disable any of the checks, in case
// there is some problem with them, or the messages get too verbose.
//
-// 2. Try to figure out the preprocessor conditional directives that
+// 4. Try to figure out the preprocessor conditional directives that
// contribute to problems and tie them to the inconsistent definitions.
//
-// 3. Check for correct and consistent usage of extern "C" {} and other
-// directives. Warn about #include inside extern "C" {}.
-//
-// 4. There seem to be a lot of spurious "not always provided" messages,
-// and many duplicates of these, which seem to occur when something is
-// defined within a preprocessor conditional block, even if the conditional
-// always evaluates the same in the stand-alone case. Perhaps we could
-// collapse the duplicates, and add an option for disabling the test (see #4).
-//
// 5. There are some legitimate uses of preprocessor macros that
// modularize will flag as errors, such as repeatedly #include'ing
// a file and using interleaving defined/undefined macros