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

Reply via email to