branch: externals/hyperbole
commit 7e6d2e9659dac4310dd5d92d87724c0528fb351f
Author: Mats Lidell <mats.lid...@lidells.se>
Commit: GitHub <nore...@github.com>

    Remove flymake/flycheck/byte-compile warnings (#431)
---
 hbut.el         |  5 ++++-
 hibtypes.el     |  3 ++-
 hui-mouse.el    |  4 ++--
 hui.el          |  8 +++++---
 hypb-ert.el     |  5 +++--
 hypb.el         |  7 ++++---
 hyrolo-logic.el |  7 +++++--
 hyrolo.el       | 29 +++++++++++++++++++----------
 hywconfig.el    |  9 ++++++---
 kotl/kview.el   |  5 ++++-
 10 files changed, 54 insertions(+), 28 deletions(-)

diff --git a/hbut.el b/hbut.el
index 1b6d5862b0..2f55428706 100644
--- a/hbut.el
+++ b/hbut.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    18-Sep-91 at 02:57:09
-;; Last-Mod:      3-Jan-24 at 02:31:24 by Bob Weiner
+;; Last-Mod:      3-Jan-24 at 23:32:27 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -67,6 +67,9 @@ Use the function, (hbut:max-len), to read the proper value.")
 
 (declare-function hpath:file-position-to-line-and-column "hpath")
 
+(declare-function hyrolo-hdr-move-after-p "hyrolo")
+(declare-function smart-eolp "hui-mouse")
+
 ;;; ************************************************************************
 ;;; Private variables
 ;;; ************************************************************************
diff --git a/hibtypes.el b/hibtypes.el
index a3e81f9ed7..e1e9bee453 100644
--- a/hibtypes.el
+++ b/hibtypes.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    19-Sep-91 at 20:45:31
-;; Last-Mod:      3-Jan-24 at 01:38:08 by Bob Weiner
+;; Last-Mod:      3-Jan-24 at 23:34:58 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -76,6 +76,7 @@
 (declare-function symtable:add "hact")
 
 (declare-function hyrolo-get-file-list "hyrolo")
+(declare-function ert-test-boundp "ert")
 
 ;;; ************************************************************************
 ;;; Public implicit button types
diff --git a/hui-mouse.el b/hui-mouse.el
index 39cc36efdc..7781a6a650 100644
--- a/hui-mouse.el
+++ b/hui-mouse.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    04-Feb-89
-;; Last-Mod:      3-Jan-24 at 13:54:31 by Bob Weiner
+;; Last-Mod:      3-Jan-24 at 23:43:10 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -115,7 +115,7 @@ In other context signal an error."
     (hypb:error "(Hyperbole Action Key): No action defined for this context; 
try another location")))
 
 (defun assist-key-error ()
-  "If in Org mode and Hyperbole shares {M-RET}, run 'org-meta-return'.
+  "If in Org mode and Hyperbole shares {M-RET}, run `org-meta-return'.
 In other context, signal an error."
   (if (and (funcall hsys-org-mode-function)
           (hsys-org-meta-return-shared-p))
diff --git a/hui.el b/hui.el
index 987864f5e6..a2d7db4bd5 100644
--- a/hui.el
+++ b/hui.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    19-Sep-91 at 21:42:03
-;; Last-Mod:      2-Jan-24 at 23:02:06 by Bob Weiner
+;; Last-Mod:      3-Jan-24 at 23:42:18 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -1141,7 +1141,8 @@ link button and return t; otherwise, return nil.
 With optional DEPRESS-WINDOW and RELEASE-WINDOW, use the points
 from those instead.  See also documentation for `hui:link-possible-types'.
 
-An Assist Mouse Key drag between windows (when not on an item) runs this 
command."
+An Assist Mouse Key drag between windows (when not on an item)
+runs this command."
   (interactive (progn
                 ;; Clear smart key variables so this does not
                 ;; improperly reference values left over from a prior
@@ -1243,7 +1244,8 @@ With optional NAME-ARG-FLAG (interactively, the prefix 
argument set to
 anything other than a single C-u (list 4)), prompt for a name to precede
 the implicit button.
 
-An Action Mouse Key drag between windows (when not on an item) runs this 
command."
+An Action Mouse Key drag between windows (when not on an item)
+runs this command."
   (interactive (progn
                 ;; Clear smart key variables so this does not
                 ;; improperly reference values left over from a prior
diff --git a/hypb-ert.el b/hypb-ert.el
index 52b7c7170d..389461633c 100644
--- a/hypb-ert.el
+++ b/hypb-ert.el
@@ -3,7 +3,7 @@
 ;; Author:       Mats Lidell <ma...@gnu.org> and Bob Weiner <r...@gnu.org>
 ;;
 ;; Orig-Date:    31-Mar-21 at 21:11:00
-;; Last-Mod:      3-Jan-24 at 03:37:28 by Bob Weiner
+;; Last-Mod:      4-Jan-24 at 14:10:39 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -40,7 +40,8 @@
   (if (memq 'message-fn (actype:params #'ert-run-tests-interactively))
       ;; Suppress ert messages so last test case message stays in the 
minibuffer;
       ;; 3rd arg message-fn available only in Emacs 27 and earlier
-      (ert selector nil #'hypb-ert-message-function)
+      (with-suppressed-warnings ((callargs ert))
+        (ert selector nil #'hypb-ert-message-function))
     (ert selector))
   ;; ERT can display a long internal data structure as a result, so
   ;; replace it in the minibuffer with a blank message.
diff --git a/hypb.el b/hypb.el
index 798337fd21..538107e289 100644
--- a/hypb.el
+++ b/hypb.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:     6-Oct-91 at 03:42:38
-;; Last-Mod:     23-Dec-23 at 23:57:52 by Bob Weiner
+;; Last-Mod:      3-Jan-24 at 23:44:37 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -852,8 +852,9 @@ PACKAGE-NAME may be a symbol or a string."
 ;;;###autoload
 (defmacro hypb:remove-from-plist (place name)
   "Remove from property list PLACE a NAME string.
-PLACE may be a symbol, or any generalized variable allowed by `setf'.
-The form generated by the macro returns true if NAME was found and removed, 
nil otherwise."
+PLACE may be a symbol, or any generalized variable allowed by
+`setf'.  The form generated by the macro returns true if NAME was
+found and removed, nil otherwise."
   (declare (debug (place form)))
   (gv-letplace (tval setter) place
     (macroexp-let2 macroexp-copyable-p tname name
diff --git a/hyrolo-logic.el b/hyrolo-logic.el
index f96b779fb0..9171b3106b 100644
--- a/hyrolo-logic.el
+++ b/hyrolo-logic.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    13-Jun-89 at 22:57:33
-;; Last-Mod:     26-Dec-23 at 20:46:10 by Bob Weiner
+;; Last-Mod:      4-Jan-24 at 14:00:13 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -89,7 +89,10 @@ flag is non-nil.  With optional KOUTLINE-FLAG, map entries 
with
 `hyrolo-map-kotl' rather than `hyrolo-map-entries'.
 
 A complex example of EXPR might be:
-  (and (or (not time card) (xor (and french \"red balloons\") spanish)) 
teacher pet)
+  (and
+   (or (not time card) (xor (and french \"red balloons\") spanish))
+   teacher
+   pet)
 which means:
   (Match neither `time' nor `card'
      or
diff --git a/hyrolo.el b/hyrolo.el
index 54bd1e4a55..07f21d3abe 100644
--- a/hyrolo.el
+++ b/hyrolo.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:     7-Jun-89 at 22:08:29
-;; Last-Mod:     31-Dec-23 at 11:53:19 by Bob Weiner
+;; Last-Mod:      4-Jan-24 at 00:12:17 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -102,6 +102,7 @@
 (defvar org-outline-regexp-bol)         ; "org.el"
 (defvar markdown-regex-header)          ; "markdown-mode.el"
 (defvar google-contacts-buffer-name)    ; "ext:google-contacts.el"
+(defvar hbut:source-prefix)             ; "hbut.el"
 
 ;; Forward declarations
 (defvar hyrolo--wconfig)
@@ -185,7 +186,7 @@ See `hyrolo-hdr-and-entry-regexp'")
   (concat hyrolo-hdr-regexp
          "\\|^" (if (boundp 'hbut:source-prefix) hbut:source-prefix "@loc> ")
          "\\|")
-  "Regular expression to prefix to `hyrolo-hdr-and-entry-regexp' and 
`outline-regexp'.
+  "Regexp to prefix to `hyrolo-hdr-and-entry-regexp' and `outline-regexp'.
 It must not contain any parenthesized match groupings.")
 
 (defvar hyrolo-entry-regexp "^\\([*\^L]+\\)\\([ \t\n\r]+\\)"
@@ -248,7 +249,8 @@ It must contain a %s indicating where to put the entry name 
and a second
   :group 'hyperbole-hyrolo)
 
 (defvar hyrolo-entry-name-regexp "[-_a-zA-Z0-9@.]+\\( ?, ?[-_a-zA-Z0-9@.]+\\)?"
-  "*Regexp matching a hyrolo entry name after matching to 
`hyrolo-hdr-and-entry-regexp'.")
+  "*Regexp matching a hyrolo entry name.
+The match is after matching to `hyrolo-hdr-and-entry-regexp'.")
 
 (defconst hyrolo-markdown-suffix-regexp 
"md\\|markdown\\|mkd\\|mdown\\|mkdn\\|mdwn"
   "Regexp matching Markdown file suffixes.")
@@ -1064,7 +1066,8 @@ any rolo entry of the given level, not the beginning of a 
line (^); an
 example, might be (regexp-quote \"**\") to match level two.  Return number
 of groupings sorted.
 
-Current buffer should be an editable HyRolo source location, not the match 
buffer."
+Current buffer should be an editable HyRolo source location, not
+the match buffer."
   (interactive "sRegexp for level's entries: \nP")
   ;; Divide by 2 in next line because each asterisk character is preceded
   ;; by a regexp-quote backslash character.
@@ -1780,7 +1783,8 @@ Return number of matching entries found."
 
 (defun hyrolo-map-level (func level-regexp &optional max-groupings)
   "Perform FUNC in current buffer on groupings of entries at level 
LEVEL-REGEXP.
-Current buffer should be an editable HyRolo source location, not the match 
buffer.
+Current buffer should be an editable HyRolo source location, not
+the match buffer.
 
 Limit to a maximum of optional argument MAX-GROUPINGS.  Nil value
 of MAX-GROUPINGS means all groupings at the given level.  FUNC
@@ -1845,7 +1849,10 @@ Return number of groupings matched."
 This mode does not add any outline-related font-locking.
 
 See the command `outline-mode' for more information on this mode."
-  nil " Outl" nil
+  ;; nil " Outl" nil ;; FIXME: From when is this obsolete?
+  :init-value nil
+  :lighter " Outl"
+  :keymap nil
   (if hyrolo-outline-minor-mode
       ;; enable minor mode
       (progn
@@ -2860,7 +2867,8 @@ Both positions may be nil if there are no matches yet 
found."
       (list nil nil))))
 
 (defun hyrolo--cache-get-major-mode-from-index (major-mode-index)
-  "Return `major-mode' key from hash table entry with key MAJOR-MODE-INDEX, 
else nil."
+  "Return `major-mode' key from hash table entry with key MAJOR-MODE-INDEX.
+Return nil if not found."
   (gethash major-mode-index hyrolo--cache-index-to-major-mode-hasht))
 
 (defun hyrolo-cache-get-major-mode-from-pos (pos)
@@ -2894,9 +2902,10 @@ Call whenever `hyrolo-display-buffer' is changed."
   "Cache buffer `major-mode' for MATCHED-BUF with point in HyRolo display 
buffer.
 MATCHED-BUF must be a live buffer, not a buffer name.
 
-Push (point-max) of `hyrolo-display-buffer' onto 
`hyrolo--cache-loc-match-bounds'.
-Push hash table's index key to `hyrolo--cache-major-mode-indexes'.
-Ensure MATCHED-BUF's `major-mode' is stored in the hash table."
+Push (point-max) of `hyrolo-display-buffer' onto
+`hyrolo--cache-loc-match-bounds'.  Push hash table's index key to
+`hyrolo--cache-major-mode-indexes'.  Ensure MATCHED-BUF's
+`major-mode' is stored in the hash table."
   (with-current-buffer hyrolo-display-buffer
     (let* ((matched-buf-major-mode (buffer-local-value 'major-mode 
matched-buf))
           (matched-buf-major-mode-name (symbol-name matched-buf-major-mode))
diff --git a/hywconfig.el b/hywconfig.el
index 673b10acc6..fa75fbfb38 100644
--- a/hywconfig.el
+++ b/hywconfig.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    15-Mar-89
-;; Last-Mod:     23-Dec-23 at 01:21:23 by Bob Weiner
+;; Last-Mod:      4-Jan-24 at 14:15:30 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -204,7 +204,8 @@ oldest one comes the newest one."
 
 (defun hywconfig-named-get (name)
   "Return the selected frame's window configuration with NAME."
-  (lax-plist-get (hywconfig-named-get-entries) name))
+  (with-suppressed-warnings ((obsolete lax-plist-get)) ;; Obsolete since 29.1, 
use plist-get
+    (lax-plist-get (hywconfig-named-get-entries) name)))
 
 (defun hywconfig-named-get-names ()
   "Return the selected frame's list of window configuration names."
@@ -216,7 +217,9 @@ oldest one comes the newest one."
 
 (defun hywconfig-named-put (name wconfig)
   "Add NAMEd WCONFIG to selected frame's plist of named window configurations."
-  (hywconfig-named-set-entries (lax-plist-put (hywconfig-named-get-entries) 
name wconfig)))
+  (hywconfig-named-set-entries
+   (with-suppressed-warnings ((obsolete lax-plist-put)) ;; Obsolete since 
29.1, use plist-get
+     (lax-plist-put (hywconfig-named-get-entries) name wconfig))))
 
 (defun hywconfig-named-remove (name)
   "Remove the selected frame's stored window configuration with NAME."
diff --git a/kotl/kview.el b/kotl/kview.el
index 63f8a7edca..26b444bc53 100644
--- a/kotl/kview.el
+++ b/kotl/kview.el
@@ -3,7 +3,7 @@
 ;; Author:       Bob Weiner
 ;;
 ;; Orig-Date:    6/30/93
-;; Last-Mod:     20-Dec-23 at 17:27:44 by Bob Weiner
+;; Last-Mod:      4-Jan-24 at 13:59:44 by Mats Lidell
 ;;
 ;; SPDX-License-Identifier: GPL-3.0-or-later
 ;;
@@ -26,6 +26,9 @@
 ;;; ************************************************************************
 
 (defvar kotl-mode:refill-flag)
+(defvar hyrolo-display-buffer)
+(defvar hyrolo-hdr-regexp)
+(defvar hbut:source-prefix)
 
 (declare-function klabel:format "klabel.el")
 (declare-function klabel:idstamp-p "klabel.el")

Reply via email to