[clang-tools-extra] two-or-so tiny documentation fixes/improvements in “main/clang-tools-extra/docs/modularize.rst” (PR #99256)

via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 16 16:22:02 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-tools-extra

Author: Abe (Abe149)

<details>
<summary>Changes</summary>

mainly, fixed “the Clang module mechanism doesn’t support headers the rely on other headers” => “the Clang module mechanism doesn’t support headers that rely on other headers”.  [emphasis on “the” versus “that”]

---
Full diff: https://github.com/llvm/llvm-project/pull/99256.diff


1 Files Affected:

- (modified) clang-tools-extra/docs/modularize.rst (+2-2) 


``````````diff
diff --git a/clang-tools-extra/docs/modularize.rst b/clang-tools-extra/docs/modularize.rst
index 64ca8c99d4e8e..97fd33b958650 100644
--- a/clang-tools-extra/docs/modularize.rst
+++ b/clang-tools-extra/docs/modularize.rst
@@ -254,8 +254,8 @@ For example, with the same header list from above::
   }
 
 Note that headers with dependents will be ignored with a warning, as the
-Clang module mechanism doesn't support headers the rely on other headers
-to be included first.
+Clang module mechanism doesn't support headers that rely on other headers
+being included first.
 
 The module map format defines some keywords which can't be used in module
 names. If a header has one of these names, an underscore ('_') will be

``````````

</details>


https://github.com/llvm/llvm-project/pull/99256


More information about the cfe-commits mailing list