[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
This revision was automatically updated to reflect the committed changes. Closed by commit rGc3dc6b081da6: [NFC][clang-tools-extra] Inclusive language: replace master with main (authored by quinnp). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 Files: clang-tools-extra/test/modularize/SubModule2.h Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Sorry, LG! (I'm not very familiar with clang-modularize, but this seems to be the only place the term is used) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
quinnp added a comment. @sammccall could you take a look at the other changes when you have time? Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
quinnp updated this revision to Diff 387990. quinnp marked an inline comment as done. quinnp added a comment. Addressing review comments. Reverting change to `Protocol.h`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 Files: clang-tools-extra/test/modularize/SubModule2.h Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
quinnp marked an inline comment as done. quinnp added inline comments. Comment at: clang-tools-extra/clangd/Protocol.h:141 /// the server can send `null` to indicate that the version is known and the - /// content on disk is the master (as speced with document content ownership). + /// content on disk is the main (as speced with document content ownership). /// sammccall wrote: > This is quoting > https://microsoft.github.io/language-server-protocol/specification. > I think they accept PRs though. Thanks @sammccall, I'll remove this change from the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
sammccall added inline comments. Comment at: clang-tools-extra/clangd/Protocol.h:141 /// the server can send `null` to indicate that the version is known and the - /// content on disk is the master (as speced with document content ownership). + /// content on disk is the main (as speced with document content ownership). /// This is quoting https://microsoft.github.io/language-server-protocol/specification. I think they accept PRs though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114100/new/ https://reviews.llvm.org/D114100 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114100: [NFC][clang-tools-extra] Inclusive language: replace master with main
quinnp created this revision. Herald added subscribers: usaxena95, kadircet, arphaman. quinnp requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. [NFC] As part of using inclusive language within the llvm project, this patch replaces master with main in these comments. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D114100 Files: clang-tools-extra/clangd/Protocol.h clang-tools-extra/test/modularize/SubModule2.h Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" Index: clang-tools-extra/clangd/Protocol.h === --- clang-tools-extra/clangd/Protocol.h +++ clang-tools-extra/clangd/Protocol.h @@ -138,7 +138,7 @@ /// identifier is sent from the server to the client and the file is not open /// in the editor (the server has not received an open notification before) /// the server can send `null` to indicate that the version is known and the - /// content on disk is the master (as speced with document content ownership). + /// content on disk is the main (as speced with document content ownership). /// /// The version number of a document will increase after each change, /// including undo/redo. The number doesn't need to be consecutive. Index: clang-tools-extra/test/modularize/SubModule2.h === --- clang-tools-extra/test/modularize/SubModule2.h +++ clang-tools-extra/test/modularize/SubModule2.h @@ -1,3 +1,3 @@ -// SubModule2.h - Master header with same name as directory. +// SubModule2.h - Main header with same name as directory. #include "SubModule2/Header3.h" #include "SubModule2/Header4.h" Index: clang-tools-extra/clangd/Protocol.h === --- clang-tools-extra/clangd/Protocol.h +++ clang-tools-extra/clangd/Protocol.h @@ -138,7 +138,7 @@ /// identifier is sent from the server to the client and the file is not open /// in the editor (the server has not received an open notification before) /// the server can send `null` to indicate that the version is known and the - /// content on disk is the master (as speced with document content ownership). + /// content on disk is the main (as speced with document content ownership). /// /// The version number of a document will increase after each change, /// including undo/redo. The number doesn't need to be consecutive. ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
