diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationInfo.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationInfo.java index 135865f271cd75..42cf3f3e169637 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationInfo.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationInfo.java @@ -174,40 +174,24 @@ public ImmutableList getSystemIncludeDirs() { } /** - * Returns the immutable set of declared include directories, relative to a - * "-I" or "-iquote" directory" (possibly empty but never null). The returned - * collection may contain duplicate elements. - * - *

Note: The iteration order of this list is preserved as ide_build_info - * writes these directories and sources out and the ordering will help when - * used by consumers. + * Returns the immutable set of declared include directories, relative to a "-I" or "-iquote" + * directory" (possibly empty but never null). */ public NestedSet getDeclaredIncludeDirs() { return declaredIncludeDirs; } /** - * Returns the immutable set of include directories, relative to a "-I" or - * "-iquote" directory", from which inclusion will produce a warning (possibly - * empty but never null). The returned collection may contain duplicate - * elements. - * - *

Note: The iteration order of this list is preserved as ide_build_info - * writes these directories and sources out and the ordering will help when - * used by consumers. + * Returns the immutable set of include directories, relative to a "-I" or "-iquote" directory", + * from which inclusion will produce a warning (possibly empty but never null). */ public NestedSet getDeclaredIncludeWarnDirs() { return declaredIncludeWarnDirs; } /** - * Returns the immutable set of headers that have been declared in the - * {@code src} or {@code headers attribute} (possibly empty but never null). - * The returned collection may contain duplicate elements. - * - *

Note: The iteration order of this list is preserved as ide_build_info - * writes these directories and sources out and the ordering will help when - * used by consumers. + * Returns the immutable set of headers that have been declared in the {@code srcs} or {@code + * hdrs} attribute (possibly empty but never null). */ public NestedSet getDeclaredIncludeSrcs() { return declaredIncludeSrcs; diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java index 98209e62d7ff6a..406bb8e8d19373 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java @@ -1076,27 +1076,22 @@ protected String getRawProgressMessage() { } /** - * Return the directories in which to look for headers (pertains to headers - * not specifically listed in {@code declaredIncludeSrcs}). The return value - * may contain duplicate elements. + * Return the directories in which to look for headers (pertains to headers not specifically + * listed in {@code declaredIncludeSrcs}). */ public NestedSet getDeclaredIncludeDirs() { return ccCompilationInfo.getDeclaredIncludeDirs(); } /** - * Return the directories in which to look for headers and issue a warning. - * (pertains to headers not specifically listed in {@code - * declaredIncludeSrcs}). The return value may contain duplicate elements. + * Return the directories in which to look for headers and issue a warning. (pertains to headers + * not specifically listed in {@code declaredIncludeSrcs}). */ public NestedSet getDeclaredIncludeWarnDirs() { return ccCompilationInfo.getDeclaredIncludeWarnDirs(); } - /** - * Return explicit header files (i.e., header files explicitly listed). The - * return value may contain duplicate elements. - */ + /** Return explicitly listed header files. */ @Override public NestedSet getDeclaredIncludeSrcs() { if (lipoScannables != null && lipoScannables.iterator().hasNext()) {