From: Marcin Borkowski <mbork@mbork.pl>
To: Drew Adams <drew.adams@oracle.com>
Cc: rfflrccrd@gmail.com, 21072@debbugs.gnu.org
Subject: bug#21072: 24.5; inconsistent behaviour of `C-M-h (mark-defun)' in Emacs Lisp
Date: Fri, 04 Nov 2016 08:48:19 +0100 [thread overview]
Message-ID: <87k2cj8z3w.fsf@mbork.pl> (raw)
In-Reply-To: <c33f960f-a234-48b8-8181-c3f199592efe@default>
On 2016-10-28, at 16:32, Drew Adams <drew.adams@oracle.com> wrote:
> Thanks for considering it. And congratulations on young Borkowski!
Thanks!
On 2016-11-02, at 19:25, Drew Adams <drew.adams@oracle.com> wrote:
>> Well, both these behaviors are manifestations of the same bug.
>> Below is the corrected version. (And below that a question.)
>
> The test cases I mentioned work now. Thx. I didn't try anything
> beyond those cases. Hopefully others will test a bit more.
I would hope so.
> No, not really. But use your own judgment, I guess.
I guess your version is simpler (no need to use raw prefix arg), and
hence better.
> This is the kind of behavior I had in mind. This is for
> `transpose-sexps', but it shows the behavior. _Any_ negative
> arg flips the direction. At the outset, a negative arg means
> move backward. The absolute value of ARG is the number of
> sexps to move over.
>
> (defun reversible-transpose-sexps (arg)
> "Reversible and repeatable `transpose-sexps'.
> Like `transpose-sexps', but:
> 1. Leaves point after the moved sexp.
> 2. When repeated, a negative prefix arg flips the direction."
> (interactive "p")
> (when (eq last-command 'rev-transp-sexps-back) (setq arg (- arg)))
> (transpose-sexps arg)
> (unless (natnump arg)
> (backward-sexp (abs arg))
> (skip-syntax-backward " .")
> (setq this-command 'rev-transp-sexps-back)))
Very nice trick with the 'last-command, thanks! I included this in my
code. I will also write some tests for that (it seems to work, but...)
and send the code soon.
>> I'm also wondering whether to allow that for
>> non-interactive use, too: I'm pretty sure nobody would want to call
>> (mark-defun '-) from Lisp code, and it might make testing slightly
>> easier.
>
> I think the behavior should be the same. But see above. The
> arg passed should be numeric (positive, zero, or negative), IMO.
Again - I agree, this makes coding (though not necessarily testing!)
simpler.
Best,
--
Marcin Borkowski
next prev parent reply other threads:[~2016-11-04 7:48 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-16 6:12 bug#21072: 24.5; inconsistent behaviour of `C-M-h (mark-defun)' in Emacs Lisp Raffaele Ricciardi
2016-04-25 11:11 ` Marcin Borkowski
2016-05-01 17:17 ` Eli Zaretskii
2016-05-01 17:49 ` Marcin Borkowski
2016-05-01 18:09 ` Eli Zaretskii
2016-05-01 19:45 ` Marcin Borkowski
2016-05-01 19:56 ` Eli Zaretskii
2016-05-03 18:58 ` Marcin Borkowski
2016-05-04 14:45 ` Eli Zaretskii
2016-05-06 12:27 ` Marcin Borkowski
2016-05-06 12:59 ` Eli Zaretskii
2016-05-07 3:47 ` Marcin Borkowski
2016-05-07 5:07 ` Drew Adams
2016-10-11 12:31 ` Marcin Borkowski
2016-10-11 15:30 ` Drew Adams
2016-10-11 17:07 ` Marcin Borkowski
2016-10-11 17:52 ` Clément Pit--Claudel
2016-10-11 20:26 ` Drew Adams
2016-10-11 21:15 ` Drew Adams
2016-10-28 5:35 ` Marcin Borkowski
2016-10-28 14:32 ` Drew Adams
2016-11-02 7:28 ` Marcin Borkowski
2016-11-02 18:25 ` Drew Adams
2016-11-04 7:48 ` Marcin Borkowski [this message]
2016-11-27 7:40 ` Marcin Borkowski
2016-11-27 18:51 ` Drew Adams
2017-02-07 6:22 ` Marcin Borkowski
2017-02-07 16:14 ` Drew Adams
2016-05-07 6:47 ` Eli Zaretskii
2016-06-05 6:30 ` Marcin Borkowski
2016-06-05 7:01 ` bug#21072: Forgotten attachment (was: bug#21072: 24.5; inconsistent behaviour of `C-M-h (mark-defun)' in Emacs Lisp) Marcin Borkowski
2016-06-09 11:56 ` bug#21072: 24.5; inconsistent behaviour of `C-M-h (mark-defun)' in Emacs Lisp Marcin Borkowski
2016-06-21 7:58 ` Marcin Borkowski
2016-06-21 9:05 ` Andreas Röhler
2016-06-21 10:07 ` Marcin Borkowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k2cj8z3w.fsf@mbork.pl \
--to=mbork@mbork.pl \
--cc=21072@debbugs.gnu.org \
--cc=drew.adams@oracle.com \
--cc=rfflrccrd@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).