all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
@ 2016-05-03 20:28 Robert Weiner
  2016-05-04 16:30 ` Lars Ingebrigtsen
  2016-05-09 15:09 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: Robert Weiner @ 2016-05-03 20:28 UTC (permalink / raw)
  To: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 322 bytes --]

It reads: "Point and before-insertion markers move forward to end up
after the inserted text."


Shouldn't it be: "Point and after-insertion markers move forward to
end up after the inserted text."


This might be even better: "Point and markers with an insertion type
of t move forward to after the inserted text."


Bob

[-- Attachment #2: Type: text/html, Size: 1977 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
  2016-05-03 20:28 Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong? Robert Weiner
@ 2016-05-04 16:30 ` Lars Ingebrigtsen
  2016-05-04 16:38   ` Eli Zaretskii
  2016-05-09 15:09 ` Lars Ingebrigtsen
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-05-04 16:30 UTC (permalink / raw)
  To: Robert Weiner; +Cc: emacs-devel

Robert Weiner <rswgnu@gmail.com> writes:

> It reads: "Point and before-insertion markers move forward to end up
> after the inserted text."
>
> Shouldn't it be: "Point and after-insertion markers move forward to
> end up after the inserted text."
>
> This might be even better: "Point and markers with an insertion type
> of t move forward to after the inserted text."

Uhm...  I think that sounds correct?  Anybody?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
  2016-05-04 16:30 ` Lars Ingebrigtsen
@ 2016-05-04 16:38   ` Eli Zaretskii
  2016-05-04 16:47     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2016-05-04 16:38 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: rswgnu, emacs-devel

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Wed, 04 May 2016 18:30:55 +0200
> Cc: emacs-devel <emacs-devel@gnu.org>
> 
> Robert Weiner <rswgnu@gmail.com> writes:
> 
> > It reads: "Point and before-insertion markers move forward to end up
> > after the inserted text."
> >
> > Shouldn't it be: "Point and after-insertion markers move forward to
> > end up after the inserted text."
> >
> > This might be even better: "Point and markers with an insertion type
> > of t move forward to after the inserted text."
> 
> Uhm...  I think that sounds correct?  Anybody?

The former wording, not the latter.

This doc bug is strange: the feature was coded from the get-go to move
the after-insertion markers after the inserted text, but the doc
string says the contrary.  So unlike Richard and those who helped him
back then.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
  2016-05-04 16:38   ` Eli Zaretskii
@ 2016-05-04 16:47     ` Lars Ingebrigtsen
  2016-05-04 16:56       ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-05-04 16:47 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: rswgnu, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

> This doc bug is strange: the feature was coded from the get-go to move
> the after-insertion markers after the inserted text, but the doc
> string says the contrary.  So unlike Richard and those who helped him
> back then.

Yeah, and it's so weird that such a blatant doc error in such a basic
function has survived for 20 years that I thought I must be misreading
the code somehow.  :-)  But it's calling insert_1_both with
before_markers false, so the doc string really is wrong.  (Besides, it
wouldn't make sense if it was doing what it was documenting.)

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
  2016-05-04 16:47     ` Lars Ingebrigtsen
@ 2016-05-04 16:56       ` Eli Zaretskii
  0 siblings, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2016-05-04 16:56 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: rswgnu, emacs-devel

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: rswgnu@gmail.com,  emacs-devel@gnu.org
> Date: Wed, 04 May 2016 18:47:17 +0200
> 
> it's calling insert_1_both with before_markers false, so the doc
> string really is wrong

It's more than that: this doc string was wrong even before the feature
that allows to move the before-insertion markers was introduced, when
the_only_ thing we could do was move only the after-insertion markers.

Strange...



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong?
  2016-05-03 20:28 Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong? Robert Weiner
  2016-05-04 16:30 ` Lars Ingebrigtsen
@ 2016-05-09 15:09 ` Lars Ingebrigtsen
  1 sibling, 0 replies; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-05-09 15:09 UTC (permalink / raw)
  To: Robert Weiner; +Cc: emacs-devel

Robert Weiner <rswgnu@gmail.com> writes:

> Shouldn't it be: "Point and after-insertion markers move forward to
> end up after the inserted text."

I've now fixed this in emacs-25.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-05-09 15:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-03 20:28 Emacs 25.1.50: Isn't 2nd line of "insert" function documentation wrong? Robert Weiner
2016-05-04 16:30 ` Lars Ingebrigtsen
2016-05-04 16:38   ` Eli Zaretskii
2016-05-04 16:47     ` Lars Ingebrigtsen
2016-05-04 16:56       ` Eli Zaretskii
2016-05-09 15:09 ` Lars Ingebrigtsen

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.