summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKévin Le Gouguec <kevin.legouguec@gmail.com>2023-05-06 09:36:25 +0200
committerKévin Le Gouguec <kevin.legouguec@gmail.com>2023-05-06 09:39:12 +0200
commitaff208ffe98e7cf48339b1f415f7d18ea44da558 (patch)
tree10f2da050a99e6df9b80cdd0022cee530118ee19
parent1c6ad1435ce0453a8a7641461032b9fb5edd26e4 (diff)
downloadmemory-leaks-aff208ffe98e7cf48339b1f415f7d18ea44da558.tar.xz
Shuffle more itches around
-rw-r--r--itches/emacs/tracker.org204
1 files changed, 103 insertions, 101 deletions
diff --git a/itches/emacs/tracker.org b/itches/emacs/tracker.org
index 2240b69..a7e3f5b 100644
--- a/itches/emacs/tracker.org
+++ b/itches/emacs/tracker.org
@@ -158,83 +158,6 @@ Fixed by Andrea (see update 9 on his [[https://akrl.sdf.org/gccemacs.html][progr
:Role: author
:END:
Unnoticed for a year, then merged with subsequent duplicate [[bug:33400]].
-*** TODO [[bug:31586]] ~frame-title-format~ doesn't save match data
-:PROPERTIES:
-:Role: watcher
-:END:
-*** DONE [[bug:33697]] file-truename messes with match data
-:PROPERTIES:
-:Role: author
-:END:
-Merged with [[bug:31586]], a more general issue dealing with match data
-being "poisoned" by a user's =:eval= form in frame-title-format.
-*** DONE [[bug:36803]] Update mode-line of every window when compilation ends
-:PROPERTIES:
-:Role: author
-:END:
-Fixed by Stefan.
-*** DONE [[bug:57733]] Fix ~replace-*-in-region~ crash
-:PROPERTIES:
-:Role: author
-:PatchApplied: t
-:TestAdded: t
-:END:
-*** TODO Add command to toggle password visibility
-**** cover letter
-Hello Emacs,
-
-On occasion, e.g. when saving .authinfo.gpg, I'd like to briefly peek at
-the password I'm typing. Entering the password twice is an appreciated
-safeguard, but I know I'm perfectly able to make the same typo twice in
-a row; I'd love for read-passwd to offer a command to toggle the
-password visibility.
-
-(I'm dismissing the "write it in the clear, then cut it and paste it"
-solution, since I don't want to add the kill ring nor my desktop
-environment's clipboard manager to the equation)
-
-I'm attaching a proof-of-concept. Additional considerations, in no
-particular order:
-
-- For the binding, I wanted something easy to mash quickly, but
- something more mnemonic might be in order?
-
-- Some applications out there show the password while the keyboard
- shortcut (or the mouse button) remains pressed, and hide it back as
- soon as it is released; I like the idea, but I have no idea whether
- that's feasible with Emacs.
-
-- The (member …) check could be replaced with e.g. (get-char-property
- (minibuffer-prompt-end) 'display); that's how I implemented it at
- first. It would fail to turn hiding off when the user has not entered
- anything though.
-
-- I don't know how much documentation this new command would warrant:
-
- - Should it be mentioned in read-passwd's docstring? C-u isn't now,
- but it was way back when, before 2012-04-11 "* lisp/subr.el
- (read-passwd): Use read-string." (088be6fbd2).
-
- - Should it be mentioned in the manual? delete-minibuffer-contents
- is documented of course, but not under "(elisp) Reading a
- Password".
-
- - a NEWS entry, maybe? AFAICT C-u was never announced there.
-
-(On a side-note, which might warrant another bug report, I wonder if
-anything could be done to advertise read-passwd-map bindings more.
-Sure, C-h b shows these bindings, but users have no reason to assume
-that read-passwd has extra bindings in addition to regular minibuffer
-ones, so why would they reach for C-h b?)
-
-Let me know if this sounds like something useful. If so, and if the
-implementation is acceptable as-is, I'll work on documentation as soon
-as I have a clearer idea of where I should add some (and whether some
-documentation should be added for C-u as well).
-
-Thank you for your time.
-
-TODO:[[file:read-passwd-reveal.patch][attach]].
*** TODO Make cursor appearance context-dependent
E.g. add a customizable list of (PREDICATE . PROPERTIES) pairs; the
cursor gets the PROPERTIES from the first PREDICATE which matches.
@@ -242,6 +165,12 @@ cursor gets the PROPERTIES from the first PREDICATE which matches.
Use-cases:
- org-speed-commands
- repeat-mode
+*** compile
+**** DONE [[bug:36803]] Update mode-line of every window when compilation ends
+:PROPERTIES:
+:Role: author
+:END:
+Fixed by Stefan.
*** completion
**** DONE [[bug:52169]] "Args out of range" when completing shell command
:PROPERTIES:
@@ -275,30 +204,6 @@ Cf. [[https://bugs.archlinux.org/task/58886][Arch Linux]], [[https://bugs.debian
:Role: author
:PatchApplied: t
:END:
-*** kill-ring
-**** DONE [[bug:60841]] ~copy-region-blink-delay~ spuriously kicks in when the =region= face has =:inverse-video=
-:PROPERTIES:
-:Role: author
-:PatchApplied: t
-:END:
-From ~emacs -Q~:
-#+begin_src elisp
-M-: (custom-set-faces '(region ((t (:foreground "blue" :inverse-video t)))))
-C-x h
-M-w
-; ⇒ point moves to mark then back after copy-region-blink-delay seconds.
-#+end_src
-
-This is caused by ~indicate-copied-region~ checking for =region=
-having a =:background= to assess whether it's "highlighted".
-
-The bug report snowballed into
-
-- fixing a couple of corner-cases that made
- ~face-differs-from-default-p~ less useful than it could be,
-- adding a user-option for users who actually want that feedback
- unconditionally,
-- my first patch committed [[https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=c4988840598b7da84b25d21a1936ce1ab6f6d666][in my own name]] 😲
*** imenu
**** TODO Add grouping/sorting predicates to ~imenu~ completion
So that ~imenu~ + ~icomplete-vertical-mode~ becomes a built-in,
@@ -336,13 +241,110 @@ Simple example of these predicates in action:
string
pred))))
#+end_src
+*** kill-ring
+**** DONE [[bug:60841]] ~copy-region-blink-delay~ spuriously kicks in when the =region= face has =:inverse-video=
+:PROPERTIES:
+:Role: author
+:PatchApplied: t
+:END:
+From ~emacs -Q~:
+#+begin_src elisp
+M-: (custom-set-faces '(region ((t (:foreground "blue" :inverse-video t)))))
+C-x h
+M-w
+; ⇒ point moves to mark then back after copy-region-blink-delay seconds.
+#+end_src
+
+This is caused by ~indicate-copied-region~ checking for =region=
+having a =:background= to assess whether it's "highlighted".
+
+The bug report snowballed into
+
+- fixing a couple of corner-cases that made
+ ~face-differs-from-default-p~ less useful than it could be,
+- adding a user-option for users who actually want that feedback
+ unconditionally,
+- my first patch committed [[https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=c4988840598b7da84b25d21a1936ce1ab6f6d666][in my own name]] 😲
+*** minibuffer
+**** TODO Add command to toggle password visibility
+***** cover letter
+Hello Emacs,
+
+On occasion, e.g. when saving .authinfo.gpg, I'd like to briefly peek at
+the password I'm typing. Entering the password twice is an appreciated
+safeguard, but I know I'm perfectly able to make the same typo twice in
+a row; I'd love for read-passwd to offer a command to toggle the
+password visibility.
+
+(I'm dismissing the "write it in the clear, then cut it and paste it"
+solution, since I don't want to add the kill ring nor my desktop
+environment's clipboard manager to the equation)
+
+I'm attaching a proof-of-concept. Additional considerations, in no
+particular order:
+
+- For the binding, I wanted something easy to mash quickly, but
+ something more mnemonic might be in order?
+
+- Some applications out there show the password while the keyboard
+ shortcut (or the mouse button) remains pressed, and hide it back as
+ soon as it is released; I like the idea, but I have no idea whether
+ that's feasible with Emacs.
+
+- The (member …) check could be replaced with e.g. (get-char-property
+ (minibuffer-prompt-end) 'display); that's how I implemented it at
+ first. It would fail to turn hiding off when the user has not entered
+ anything though.
+
+- I don't know how much documentation this new command would warrant:
+
+ - Should it be mentioned in read-passwd's docstring? C-u isn't now,
+ but it was way back when, before 2012-04-11 "* lisp/subr.el
+ (read-passwd): Use read-string." (088be6fbd2).
+
+ - Should it be mentioned in the manual? delete-minibuffer-contents
+ is documented of course, but not under "(elisp) Reading a
+ Password".
+ - a NEWS entry, maybe? AFAICT C-u was never announced there.
+
+(On a side-note, which might warrant another bug report, I wonder if
+anything could be done to advertise read-passwd-map bindings more.
+Sure, C-h b shows these bindings, but users have no reason to assume
+that read-passwd has extra bindings in addition to regular minibuffer
+ones, so why would they reach for C-h b?)
+
+Let me know if this sounds like something useful. If so, and if the
+implementation is acceptable as-is, I'll work on documentation as soon
+as I have a clearer idea of where I should add some (and whether some
+documentation should be added for C-u as well).
+
+Thank you for your time.
+
+TODO:[[file:read-passwd-reveal.patch][attach]].
*** project
**** DONE [[bug:45765]] Change default-directory before prompting in project-compile
:PROPERTIES:
:Role: author
:PatchApplied: t
:END:
+*** regexp matching
+**** TODO [[bug:31586]] ~frame-title-format~ doesn't save match data
+:PROPERTIES:
+:Role: watcher
+:END:
+**** DONE [[bug:33697]] file-truename messes with match data
+:PROPERTIES:
+:Role: author
+:END:
+Merged with [[bug:31586]], a more general issue dealing with match data
+being "poisoned" by a user's =:eval= form in frame-title-format.
+**** DONE [[bug:57733]] Fix ~replace-*-in-region~ crash
+:PROPERTIES:
+:Role: author
+:PatchApplied: t
+:TestAdded: t
+:END:
*** shr
**** DONE [[bug:39504]] Ensure faces of enclosing elements apply to <code> elements
:PROPERTIES: