all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Robert Goldman <rpgoldman@sift.info>
To: Carsten Dominik <dominik@science.uva.nl>
Cc: emacs-orgmode@gnu.org
Subject: Re: question about org-region-active-p
Date: Wed, 17 Dec 2008 08:12:21 -0600	[thread overview]
Message-ID: <494908C5.5050702@sift.info> (raw)
In-Reply-To: <EE43E2A6-9393-49C3-825A-084912C7F26F@uva.nl>

Carsten Dominik wrote:
> Hi Robert,
> 
> no, I do not think this would make sense, to step out with
> an error, because all the commands that need a region will
> also work without one.  This is how regions work in Emacs.
> 
> If you find places in the documentation where this could
> be made clearer, let me know.

Thank you Carsten for the response.  I am still inclined to think that
this case is one where the user should get some warning because it's in
that bad class of bug-like situations where code quietly does something
that the user does not expect and that contradicts the documentation.

On the other hand, I can't see where one would raise an error.  It's not
an error to call org-region-active-p without transient-mark-mode; indeed
some libraries turn this on and off, and the user is entitled not to use it.

The call to export isn't right, because the export command does not give
the system any opportunity to infer whether the user intended to operate
on a region or on the whole.  So there's no way to tell that the user's
expectations are going to be violated.

The loading of org-mode isn't the right place, either, since the vast
majority of org-mode works just fine with t-m-m turned off.

I'm inclined to think that the answer is simply that one shouldn't turn
transient-mark-mode off, ever.  This works for me, but I hate to think
of the next person coming along to fall into this same trap, from which
the only means of escape is to inspect the source code and use the debugger.

I would suggest a modification to the manual and will submit one, but
need clarification about one issue:  the use of t-m-m seems to be a
fallback for older emacsen that don't have region-active-p.  I have such
an emacs, so I don't know --- does this mean that on newer emacsen
org-region-active-p will work even without transient-mark-mode?

Best,
Robert

  reply	other threads:[~2008-12-17 14:18 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-16  1:51 question about org-region-active-p Robert Goldman
2008-12-16  2:20 ` Samuel Wales
2008-12-16  7:53 ` Carsten Dominik
2008-12-16 15:43   ` Robert Goldman
2008-12-17  8:13     ` Carsten Dominik
2008-12-17 14:12       ` Robert Goldman [this message]
2008-12-17 16:29         ` Carsten Dominik
2008-12-17 16:38           ` Robert Goldman
2008-12-17 16:44           ` Bernt Hansen
2008-12-17 17:01             ` Carsten Dominik
2008-12-18 16:48               ` Robert Goldman
2008-12-19  8:39                 ` Carsten Dominik
2008-12-16 15:52 ` William Henney

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=494908C5.5050702@sift.info \
    --to=rpgoldman@sift.info \
    --cc=dominik@science.uva.nl \
    --cc=emacs-orgmode@gnu.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.