* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
[not found] ` <E1aXuK3-0005Mz-Ua@vcs.savannah.gnu.org>
@ 2016-02-22 18:50 ` Stefan Monnier
2016-02-22 19:30 ` Eli Zaretskii
0 siblings, 1 reply; 10+ messages in thread
From: Stefan Monnier @ 2016-02-22 18:50 UTC (permalink / raw
To: emacs-devel; +Cc: Eli Zaretskii
> +After a command is executed, if point moved into a region that has
> +special properties (e.g. composition, display), Emacs adjusts point to
> +the boundary of the region. But when a command binds this variable to
> +non-nil, this point adjustment is suppressed.
Why did you replace "sets" with "binds"?
AFAIK let-binding this variable has no effect because the variable is
consulted only after the command finishes, i.e. after the binding has
been undone.
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-22 18:50 ` [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment' Stefan Monnier
@ 2016-02-22 19:30 ` Eli Zaretskii
2016-02-22 20:45 ` Stefan Monnier
0 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2016-02-22 19:30 UTC (permalink / raw
To: Stefan Monnier; +Cc: emacs-devel
> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Cc: Eli Zaretskii <eliz@gnu.org>
> Date: Mon, 22 Feb 2016 13:50:37 -0500
>
> > +After a command is executed, if point moved into a region that has
> > +special properties (e.g. composition, display), Emacs adjusts point to
> > +the boundary of the region. But when a command binds this variable to
> > +non-nil, this point adjustment is suppressed.
>
> Why did you replace "sets" with "binds"?
> AFAIK let-binding this variable has no effect because the variable is
> consulted only after the command finishes, i.e. after the binding has
> been undone.
Who said anything about let-binding?
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-22 19:30 ` Eli Zaretskii
@ 2016-02-22 20:45 ` Stefan Monnier
2016-02-22 20:53 ` Eli Zaretskii
0 siblings, 1 reply; 10+ messages in thread
From: Stefan Monnier @ 2016-02-22 20:45 UTC (permalink / raw
To: Eli Zaretskii; +Cc: emacs-devel
>> > +After a command is executed, if point moved into a region that has
>> > +special properties (e.g. composition, display), Emacs adjusts point to
>> > +the boundary of the region. But when a command binds this variable to
>> > +non-nil, this point adjustment is suppressed.
>>
>> Why did you replace "sets" with "binds"?
>> AFAIK let-binding this variable has no effect because the variable is
>> consulted only after the command finishes, i.e. after the binding has
>> been undone.
> Who said anything about let-binding?
To me the difference between "sets" and "binds" is that the former
clearly means "setq" while the other can either mean "change it from
unbound to bound" or "let-binds".
So, I find "binds" to be less precise than "sets", hence my question:
Why did you replace "sets" with "binds"?
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-22 20:45 ` Stefan Monnier
@ 2016-02-22 20:53 ` Eli Zaretskii
2016-02-22 21:13 ` Stefan Monnier
0 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2016-02-22 20:53 UTC (permalink / raw
To: Stefan Monnier; +Cc: emacs-devel
> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Cc: emacs-devel@gnu.org
> Date: Mon, 22 Feb 2016 15:45:35 -0500
>
> To me the difference between "sets" and "binds" is that the former
> clearly means "setq" while the other can either mean "change it from
> unbound to bound" or "let-binds".
>
> So, I find "binds" to be less precise than "sets", hence my question:
> Why did you replace "sets" with "binds"?
Because that's how I tend to think about the act of giving a value to
a variable in Emacs Lisp.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-22 20:53 ` Eli Zaretskii
@ 2016-02-22 21:13 ` Stefan Monnier
2016-02-23 3:27 ` Eli Zaretskii
[not found] ` <<83fuwkjecd.fsf@gnu.org>
0 siblings, 2 replies; 10+ messages in thread
From: Stefan Monnier @ 2016-02-22 21:13 UTC (permalink / raw
To: Eli Zaretskii; +Cc: emacs-devel
>> To me the difference between "sets" and "binds" is that the former
>> clearly means "setq" while the other can either mean "change it from
>> unbound to bound" or "let-binds".
>> So, I find "binds" to be less precise than "sets", hence my question:
>> Why did you replace "sets" with "binds"?
> Because that's how I tend to think about the act of giving a value to
> a variable in Emacs Lisp.
OK. To me, the fact that disable-point-adjustment needs to be setq'd by
the command rather than let-bound is an important point (I'd expect this
to surprise many coders since the usual way for a command to affect the
behavior of Emacs "for this command only" is to let-bind a variable), so
I find the new wording to encourage confusion rather than trying to
avoid it.
But clearly you have a different opinion, and I honestly can't claim to
know better, especially when it comes to writing docs, so I'll shut up
now,
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-22 21:13 ` Stefan Monnier
@ 2016-02-23 3:27 ` Eli Zaretskii
2016-02-23 3:40 ` Lars Ingebrigtsen
[not found] ` <<83fuwkjecd.fsf@gnu.org>
1 sibling, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2016-02-23 3:27 UTC (permalink / raw
To: Stefan Monnier; +Cc: emacs-devel
> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Cc: emacs-devel@gnu.org
> Date: Mon, 22 Feb 2016 16:13:36 -0500
>
> >> Why did you replace "sets" with "binds"?
> > Because that's how I tend to think about the act of giving a value to
> > a variable in Emacs Lisp.
>
> OK. To me, the fact that disable-point-adjustment needs to be setq'd by
> the command rather than let-bound is an important point (I'd expect this
> to surprise many coders since the usual way for a command to affect the
> behavior of Emacs "for this command only" is to let-bind a variable), so
> I find the new wording to encourage confusion rather than trying to
> avoid it.
If this is confusing, I will changing it back. What do others think?
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-23 3:27 ` Eli Zaretskii
@ 2016-02-23 3:40 ` Lars Ingebrigtsen
2016-02-23 4:16 ` Kaushal Modi
0 siblings, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2016-02-23 3:40 UTC (permalink / raw
To: Eli Zaretskii; +Cc: Stefan Monnier, emacs-devel
Eli Zaretskii <eliz@gnu.org> writes:
> If this is confusing, I will changing it back. What do others think?
I think "set" is clearer in this instance.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-23 3:40 ` Lars Ingebrigtsen
@ 2016-02-23 4:16 ` Kaushal Modi
0 siblings, 0 replies; 10+ messages in thread
From: Kaushal Modi @ 2016-02-23 4:16 UTC (permalink / raw
To: Lars Ingebrigtsen; +Cc: Eli Zaretskii, Stefan Monnier, Emacs developers
[-- Attachment #1: Type: text/plain, Size: 123 bytes --]
On Feb 22, 2016 10:41 PM, "Lars Ingebrigtsen" <larsi@gnus.org> wrote:
>
> I think "set" is clearer in this instance.
Yes.
[-- Attachment #2: Type: text/html, Size: 233 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
[not found] ` <<83fuwkjecd.fsf@gnu.org>
@ 2016-02-23 7:10 ` Drew Adams
2016-02-23 17:37 ` Eli Zaretskii
0 siblings, 1 reply; 10+ messages in thread
From: Drew Adams @ 2016-02-23 7:10 UTC (permalink / raw
To: Eli Zaretskii, Stefan Monnier; +Cc: emacs-devel
> > >> Why did you replace "sets" with "binds"?
> > > Because that's how I tend to think about the act of giving a value to
> > > a variable in Emacs Lisp.
> >
> > OK. To me, the fact that disable-point-adjustment needs to be setq'd by
> > the command rather than let-bound is an important point (I'd expect this
> > to surprise many coders since the usual way for a command to affect the
> > behavior of Emacs "for this command only" is to let-bind a variable), so
> > I find the new wording to encourage confusion rather than trying to
> > avoid it.
>
> If this is confusing, I will changing it back. What do others think?
I agree with Stefan here. Generally use "set" for assignment
(regardless of whether the symbol has already been bound in any
way). Use "bind" for a `let' binding.
It's true that we also speak of a symbol being unbound or bound
depending on whether its value cell is void. But most of the
uses of "bound" and "bind" for symbols in Emacs have to do with
`let' bindings.
However, if it is important in this particular context to make
clear that "disable-point-adjustment needs to be setq'd by the
command rather than let-bound" then I'd say that, to remove any
ambiguity.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment'
2016-02-23 7:10 ` Drew Adams
@ 2016-02-23 17:37 ` Eli Zaretskii
0 siblings, 0 replies; 10+ messages in thread
From: Eli Zaretskii @ 2016-02-23 17:37 UTC (permalink / raw
To: monnier; +Cc: emacs-devel
OK, I clarified the doc strings wrt this aspect.
Thanks for the feedback.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-02-23 17:37 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20160222173059.20603.67344@vcs.savannah.gnu.org>
[not found] ` <E1aXuK3-0005Mz-Ua@vcs.savannah.gnu.org>
2016-02-22 18:50 ` [Emacs-diffs] emacs-25 6bd9d69: Fix documentation of 'global-disable-point-adjustment' Stefan Monnier
2016-02-22 19:30 ` Eli Zaretskii
2016-02-22 20:45 ` Stefan Monnier
2016-02-22 20:53 ` Eli Zaretskii
2016-02-22 21:13 ` Stefan Monnier
2016-02-23 3:27 ` Eli Zaretskii
2016-02-23 3:40 ` Lars Ingebrigtsen
2016-02-23 4:16 ` Kaushal Modi
[not found] ` <<83fuwkjecd.fsf@gnu.org>
2016-02-23 7:10 ` Drew Adams
2016-02-23 17:37 ` Eli Zaretskii
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.