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

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

[elpa] externals/org fad90df535 4/5: Revert "org-table-make-reference: D


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

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

    Revert "org-table-make-reference: Disable compiler warning"
    
    This reverts commit bdc60fb936b2f28f1546b0e89243cefa53da7847.
    
    Apparently, the branch of if used sometimes and that sometimes is
    covered by tests.  Strange.  Leave it be for now.
---
 lisp/org-table.el | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lisp/org-table.el b/lisp/org-table.el
index efa5425063..514b275fcb 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2864,14 +2864,13 @@ list, `literal' is for the format specifier L."
       (if lispp
          (if (eq lispp 'literal)
              elements
-           (if nil
-                ;; (and (eq elements "") (not keep-empty))
+           (if (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]