emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[elpa] externals/org bdc60fb936 3/5: org-table-make-reference: Disable c


From: ELPA Syncer
Subject: [elpa] externals/org bdc60fb936 3/5: org-table-make-reference: Disable compiler warning
Date: Thu, 11 Jan 2024 10:48:17 -0500 (EST)

branch: externals/org
commit bdc60fb936b2f28f1546b0e89243cefa53da7847
Author: Ihor Radchenko <yantar92@posteo.net>
Commit: Ihor Radchenko <yantar92@posteo.net>

    org-table-make-reference: Disable compiler warning
    
    * lisp/org-table.el (org-table-make-reference): Comment out the code
    that always returns nil, replacing it with explicit nil.  This line is
    certainly a fault in logic, but we do not know of any associated bugs.
    So, just leave a FIXME to potential future refactoring and avoid the
    compiler complaining about this code every time.
---
 lisp/org-table.el | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lisp/org-table.el b/lisp/org-table.el
index 514b275fcb..efa5425063 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2864,13 +2864,14 @@ list, `literal' is for the format specifier L."
       (if lispp
          (if (eq lispp 'literal)
              elements
-           (if (and (eq elements "") (not keep-empty))
+           (if nil
+                ;; (and (eq elements "") (not keep-empty))
                 ;; FIXME: This branch of `if' is never used because
                 ;; strings are never `eq' here.  But changing to
                 ;; `equal' breaks tests.
                 ;; See
                 ;; 
https://list.orgmode.org/orgmode/20230827214320.46754-1-salutis@me.com/
-               ""
+               ""
              (prin1-to-string
               (if numbers (string-to-number elements) elements))))
        (if (string-match "\\S-" elements)



reply via email to

[Prev in Thread] Current Thread [Next in Thread]