llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Roi Martin (jroimartin) <details> <summary>Changes</summary> This is a follow-up to commit f69110dcc973 ("Check for a 'buffer' type instead of 'buffer-live'."). In Emacs 29, 'buffer-live' is no longer recognized as a type and generates a compilation warning. Every function that requires a live buffer already checks whether the buffer is live, so we don't need to check ourselves. --- Full diff: https://github.com/llvm/llvm-project/pull/128385.diff 1 Files Affected: - (modified) clang-tools-extra/clang-include-fixer/tool/clang-include-fixer.el (+1-1) ``````````diff diff --git a/clang-tools-extra/clang-include-fixer/tool/clang-include-fixer.el b/clang-tools-extra/clang-include-fixer/tool/clang-include-fixer.el index f3a949f8c1b55..1cb9a0340b5e3 100644 --- a/clang-tools-extra/clang-include-fixer/tool/clang-include-fixer.el +++ b/clang-tools-extra/clang-include-fixer/tool/clang-include-fixer.el @@ -249,7 +249,7 @@ Add a missing header if there is any. If there are multiple possible headers the user can select one of them to be included. Temporarily highlight the affected symbols. Asynchronously call clang-include-fixer to insert the selected header." - (cl-check-type stdout buffer-live) + (cl-check-type stdout buffer) (let ((context (clang-include-fixer--parse-json stdout))) (let-alist context (cond `````````` </details> https://github.com/llvm/llvm-project/pull/128385 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits