all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lennart Borgman <lennart.borgman@gmail.com>
To: "Stephen J. Turnbull" <stephen@xemacs.org>
Cc: "Antoine Levitt" <antoine.levitt@gmail.com>,
	"Andreas Röhler" <andreas.roehler@online.de>,
	"Stefan Monnier" <monnier@iro.umontreal.ca>,
	emacs-devel@gnu.org
Subject: Re: Errors in interactive commands
Date: Wed, 3 Aug 2011 11:37:38 +0200	[thread overview]
Message-ID: <CANbX366==rRMY8TZa+=AK4FnjJWDQMay6DMtFv26Utw4d-rqDA@mail.gmail.com> (raw)
In-Reply-To: <87aabrhs2o.fsf@uwakimon.sk.tsukuba.ac.jp>

On Wed, Aug 3, 2011 at 09:28, Stephen J. Turnbull <stephen@xemacs.org> wrote:
> Andreas Röhler writes:
>
>  > A move-forward at the end of the buffer isn't a wrong command as such,
>  > just will not be successful.
>  >
>  > Therefor it should not be raised an error, just nil returned.
>
> This is problematic, IMO.  Suppose you have a buffer which is supposed
> to have an even number of objects in it, and you want to delete every
> second then.  Then
>
> (progn
>  (goto (point-min))
>  (while (not (eobp))
>    (forward-object 2)
>    (delete-object -1)))
>
> DTRTs, including detecting the "incorrect state" of an odd number of
> objects, and not deleting the first object on an incomplete last line.
>
> If you're going to change the behavior of the movement commands here,
> I think it would be better to have them return the number of objects
> left to move over.  Then 0 would mean (complete) success (how Un*x-y!)
>
> But this would be a major change.

Examples such like this one is why I propose using something like
(throw 'command-level) instead of raising an error.

Of course the catch tag 'command-level does not exist and I propose
adding it in the C code command loops in so that (throw
'command-level) gives an execution path similar to (error 'something).
In this way I think we will get rather few compatibility problems.

Maybe this can be introduced in debug-ignored-errors?



  parent reply	other threads:[~2011-08-03  9:37 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-02 17:46 Errors in interactive commands Antoine Levitt
2011-08-02 19:04 ` Andreas Röhler
2011-08-02 19:15 ` Lennart Borgman
2011-08-02 19:18   ` Lars Magne Ingebrigtsen
2011-08-03  7:13   ` Stephen J. Turnbull
2011-08-03  9:28     ` Lennart Borgman
2011-08-02 19:25 ` Stefan Monnier
2011-08-02 19:36   ` Lennart Borgman
2011-08-03  6:13     ` Andreas Röhler
2011-08-03  7:28       ` Stephen J. Turnbull
2011-08-03  7:46         ` Andreas Röhler
2011-08-03  8:08           ` David Kastrup
2011-08-03  8:34             ` Andreas Röhler
2011-08-03  9:09           ` Stephen J. Turnbull
2011-08-03 10:31             ` Andreas Röhler
2011-08-04  2:15               ` Stephen J. Turnbull
2011-08-03  9:37         ` Lennart Borgman [this message]
2011-08-03  9:45           ` Andreas Schwab
2011-08-03 14:41             ` Stefan Monnier
2011-08-03 15:01               ` Lennart Borgman

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CANbX366==rRMY8TZa+=AK4FnjJWDQMay6DMtFv26Utw4d-rqDA@mail.gmail.com' \
    --to=lennart.borgman@gmail.com \
    --cc=andreas.roehler@online.de \
    --cc=antoine.levitt@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=stephen@xemacs.org \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.