diff options
| author | Stefan Monnier | 2025-04-29 22:54:41 -0400 |
|---|---|---|
| committer | Stefan Monnier | 2025-04-30 12:28:35 -0400 |
| commit | 3bc1c13661a334fedd8967188715cf85ef2af992 (patch) | |
| tree | 52d1f044f1eb173f86c5dd855cca3af7fe345529 | |
| parent | fcaec1ff0d6be18d4fa3682401ced30741be6243 (diff) | |
| download | emacs-3bc1c13661a334fedd8967188715cf85ef2af992.tar.gz emacs-3bc1c13661a334fedd8967188715cf85ef2af992.zip | |
(register-preview-info): Delete `noconfirm` slot
* lisp/register.el (register-preview-info): Remove `noconfirm` slot.
(register-command-info): Delete `:noconfirm` args.
(register-read-with-preview-fancy): Hardcode the `noconfirm` setting
because it was always exactly the same anyway.
| -rw-r--r-- | lisp/register.el | 45 |
1 files changed, 15 insertions, 30 deletions
diff --git a/lisp/register.el b/lisp/register.el index a1bffb5529b..f8f6488fee4 100644 --- a/lisp/register.el +++ b/lisp/register.el | |||
| @@ -190,9 +190,8 @@ TYPES are the supported types of registers. | |||
| 190 | MSG is the minibuffer message to show when a register is selected. | 190 | MSG is the minibuffer message to show when a register is selected. |
| 191 | ACT is the type of action the command is doing on register. | 191 | ACT is the type of action the command is doing on register. |
| 192 | SMATCH accept a boolean value to say if the command accepts non-matching | 192 | SMATCH accept a boolean value to say if the command accepts non-matching |
| 193 | registers. | 193 | registers." |
| 194 | If NOCONFIRM is non-nil, request confirmation of register name by RET." | 194 | types msg act smatch) |
| 195 | types msg act smatch noconfirm) | ||
| 196 | 195 | ||
| 197 | (cl-defgeneric register-command-info (command) | 196 | (cl-defgeneric register-command-info (command) |
| 198 | "Return a `register-preview-info' object storing data for COMMAND." | 197 | "Return a `register-preview-info' object storing data for COMMAND." |
| @@ -204,8 +203,7 @@ If NOCONFIRM is non-nil, request confirmation of register name by RET." | |||
| 204 | :types '(string number) | 203 | :types '(string number) |
| 205 | :msg "Insert register `%s'" | 204 | :msg "Insert register `%s'" |
| 206 | :act 'insert | 205 | :act 'insert |
| 207 | :smatch t | 206 | :smatch t)) |
| 208 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 209 | (cl-defmethod register-command-info ((_command (eql jump-to-register))) | 207 | (cl-defmethod register-command-info ((_command (eql jump-to-register))) |
| 210 | (make-register-preview-info | 208 | (make-register-preview-info |
| 211 | ;; FIXME: This should not be hardcoded but computed based on whether | 209 | ;; FIXME: This should not be hardcoded but computed based on whether |
| @@ -214,86 +212,73 @@ If NOCONFIRM is non-nil, request confirmation of register name by RET." | |||
| 214 | file buffer file-query) | 212 | file buffer file-query) |
| 215 | :msg "Jump to register `%s'" | 213 | :msg "Jump to register `%s'" |
| 216 | :act 'jump | 214 | :act 'jump |
| 217 | :smatch t | 215 | :smatch t)) |
| 218 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 219 | (cl-defmethod register-command-info ((_command (eql view-register))) | 216 | (cl-defmethod register-command-info ((_command (eql view-register))) |
| 220 | (make-register-preview-info | 217 | (make-register-preview-info |
| 221 | :types '(all) | 218 | :types '(all) |
| 222 | :msg "View register `%s'" | 219 | :msg "View register `%s'" |
| 223 | :act 'view | 220 | :act 'view |
| 224 | :noconfirm (memq register-use-preview '(nil never)) | ||
| 225 | :smatch t)) | 221 | :smatch t)) |
| 226 | (cl-defmethod register-command-info ((_command (eql append-to-register))) | 222 | (cl-defmethod register-command-info ((_command (eql append-to-register))) |
| 227 | (make-register-preview-info | 223 | (make-register-preview-info |
| 228 | :types '(string) ;; FIXME: Fails on rectangles! | 224 | :types '(string) ;; FIXME: Fails on rectangles! |
| 229 | :msg "Append to register `%s'" | 225 | :msg "Append to register `%s'" |
| 230 | :act 'modify | 226 | :act 'modify |
| 231 | :noconfirm (memq register-use-preview '(nil never)) | ||
| 232 | :smatch t)) | 227 | :smatch t)) |
| 233 | (cl-defmethod register-command-info ((_command (eql prepend-to-register))) | 228 | (cl-defmethod register-command-info ((_command (eql prepend-to-register))) |
| 234 | (make-register-preview-info | 229 | (make-register-preview-info |
| 235 | :types '(string) ;;FIXME: Fails on rectangles! | 230 | :types '(string) ;;FIXME: Fails on rectangles! |
| 236 | :msg "Prepend to register `%s'" | 231 | :msg "Prepend to register `%s'" |
| 237 | :act 'modify | 232 | :act 'modify |
| 238 | :noconfirm (memq register-use-preview '(nil never)) | ||
| 239 | :smatch t)) | 233 | :smatch t)) |
| 240 | (cl-defmethod register-command-info ((_command (eql increment-register))) | 234 | (cl-defmethod register-command-info ((_command (eql increment-register))) |
| 241 | (make-register-preview-info | 235 | (make-register-preview-info |
| 242 | :types '(string number) ;;FIXME: Fails on rectangles! | 236 | :types '(string number) ;;FIXME: Fails on rectangles! |
| 243 | :msg "Increment register `%s'" | 237 | :msg "Increment register `%s'" |
| 244 | :act 'modify | 238 | :act 'modify |
| 245 | :noconfirm (memq register-use-preview '(nil never)) | ||
| 246 | :smatch t)) | 239 | :smatch t)) |
| 247 | (cl-defmethod register-command-info ((_command (eql copy-to-register))) | 240 | (cl-defmethod register-command-info ((_command (eql copy-to-register))) |
| 248 | (make-register-preview-info | 241 | (make-register-preview-info |
| 249 | :types '(all) | 242 | :types '(all) |
| 250 | :msg "Copy to register `%s'" | 243 | :msg "Copy to register `%s'" |
| 251 | :act 'set | 244 | :act 'set)) |
| 252 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 253 | (cl-defmethod register-command-info ((_command (eql point-to-register))) | 245 | (cl-defmethod register-command-info ((_command (eql point-to-register))) |
| 254 | (make-register-preview-info | 246 | (make-register-preview-info |
| 255 | :types '(all) | 247 | :types '(all) |
| 256 | :msg "Point to register `%s'" | 248 | :msg "Point to register `%s'" |
| 257 | :act 'set | 249 | :act 'set)) |
| 258 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 259 | (cl-defmethod register-command-info ((_command (eql number-to-register))) | 250 | (cl-defmethod register-command-info ((_command (eql number-to-register))) |
| 260 | (make-register-preview-info | 251 | (make-register-preview-info |
| 261 | :types '(all) | 252 | :types '(all) |
| 262 | :msg "Number to register `%s'" | 253 | :msg "Number to register `%s'" |
| 263 | :act 'set | 254 | :act 'set)) |
| 264 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 265 | (cl-defmethod register-command-info | 255 | (cl-defmethod register-command-info |
| 266 | ((_command (eql window-configuration-to-register))) | 256 | ((_command (eql window-configuration-to-register))) |
| 267 | (make-register-preview-info | 257 | (make-register-preview-info |
| 268 | :types '(all) | 258 | :types '(all) |
| 269 | :msg "Window configuration to register `%s'" | 259 | :msg "Window configuration to register `%s'" |
| 270 | :act 'set | 260 | :act 'set)) |
| 271 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 272 | (cl-defmethod register-command-info ((_command (eql frameset-to-register))) | 261 | (cl-defmethod register-command-info ((_command (eql frameset-to-register))) |
| 273 | (make-register-preview-info | 262 | (make-register-preview-info |
| 274 | :types '(all) | 263 | :types '(all) |
| 275 | :msg "Frameset to register `%s'" | 264 | :msg "Frameset to register `%s'" |
| 276 | :act 'set | 265 | :act 'set)) |
| 277 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 278 | (cl-defmethod register-command-info ((_command (eql copy-rectangle-to-register))) | 266 | (cl-defmethod register-command-info ((_command (eql copy-rectangle-to-register))) |
| 279 | (make-register-preview-info | 267 | (make-register-preview-info |
| 280 | :types '(all) | 268 | :types '(all) |
| 281 | :msg "Copy rectangle to register `%s'" | 269 | :msg "Copy rectangle to register `%s'" |
| 282 | :act 'set | 270 | :act 'set |
| 283 | :noconfirm (memq register-use-preview '(nil never)) | ||
| 284 | :smatch t)) | 271 | :smatch t)) |
| 285 | (cl-defmethod register-command-info ((_command (eql file-to-register))) | 272 | (cl-defmethod register-command-info ((_command (eql file-to-register))) |
| 286 | (make-register-preview-info | 273 | (make-register-preview-info |
| 287 | :types '(all) | 274 | :types '(all) |
| 288 | :msg "File to register `%s'" | 275 | :msg "File to register `%s'" |
| 289 | :act 'set | 276 | :act 'set)) |
| 290 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 291 | (cl-defmethod register-command-info ((_command (eql buffer-to-register))) | 277 | (cl-defmethod register-command-info ((_command (eql buffer-to-register))) |
| 292 | (make-register-preview-info | 278 | (make-register-preview-info |
| 293 | :types '(all) | 279 | :types '(all) |
| 294 | :msg "Buffer to register `%s'" | 280 | :msg "Buffer to register `%s'" |
| 295 | :act 'set | 281 | :act 'set)) |
| 296 | :noconfirm (memq register-use-preview '(nil never)))) | ||
| 297 | 282 | ||
| 298 | (defun register-preview-forward-line (arg) | 283 | (defun register-preview-forward-line (arg) |
| 299 | "Move to next or previous line in register preview buffer. | 284 | "Move to next or previous line in register preview buffer. |
| @@ -502,13 +487,13 @@ or `never'." | |||
| 502 | m)) | 487 | m)) |
| 503 | (data (register-command-info this-command)) | 488 | (data (register-command-info this-command)) |
| 504 | (enable-recursive-minibuffers t) | 489 | (enable-recursive-minibuffers t) |
| 505 | types msg result act win strs smatch noconfirm) | 490 | types msg result act win strs smatch |
| 491 | (noconfirm (memq register-use-preview '(nil never)))) | ||
| 506 | (if data | 492 | (if data |
| 507 | (setq types (register-preview-info-types data) | 493 | (setq types (register-preview-info-types data) |
| 508 | msg (register-preview-info-msg data) | 494 | msg (register-preview-info-msg data) |
| 509 | act (register-preview-info-act data) | 495 | act (register-preview-info-act data) |
| 510 | smatch (register-preview-info-smatch data) | 496 | smatch (register-preview-info-smatch data)) |
| 511 | noconfirm (register-preview-info-noconfirm data)) | ||
| 512 | (setq types '(all) | 497 | (setq types '(all) |
| 513 | msg "Overwrite register `%s'" | 498 | msg "Overwrite register `%s'" |
| 514 | act 'set)) | 499 | act 'set)) |
| @@ -600,7 +585,7 @@ or `never'." | |||
| 600 | (with-selected-window (minibuffer-window) | 585 | (with-selected-window (minibuffer-window) |
| 601 | (minibuffer-message | 586 | (minibuffer-message |
| 602 | msg (key-description pat))) | 587 | msg (key-description pat))) |
| 603 | ;; `:noconfirm' is specified explicitly, don't ask for | 588 | ;; `noconfirm' is specified explicitly, don't ask for |
| 604 | ;; confirmation and exit immediately (bug#66394). | 589 | ;; confirmation and exit immediately (bug#66394). |
| 605 | (setq result pat) | 590 | (setq result pat) |
| 606 | (exit-minibuffer)))))))) | 591 | (exit-minibuffer)))))))) |