all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: [elpa] master 3d0a12e 13/24: Fix el-search--ensure-sexp-start error at bob
       [not found] ` <20160519204638.0003A220159@vcs.savannah.gnu.org>
@ 2016-05-20  0:10   ` Stefan Monnier
  2016-05-20 14:20     ` Michael Heerdegen
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Monnier @ 2016-05-20  0:10 UTC (permalink / raw)
  To: emacs-devel; +Cc: Michael Heerdegen

> +                                    (and (> (point) n)
> +                                         (save-excursion
> +                                           (backward-char n)

If you want to avoid the error that backward-char can signal, you need
to take into account that the buffer might be narrowed, so (> (point) n)
is not sufficient, i.e. you need something like (<= n (- (point) (point-min)))


        Stefan



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

* Re: [elpa] master 3d0a12e 13/24: Fix el-search--ensure-sexp-start error at bob
  2016-05-20  0:10   ` [elpa] master 3d0a12e 13/24: Fix el-search--ensure-sexp-start error at bob Stefan Monnier
@ 2016-05-20 14:20     ` Michael Heerdegen
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Heerdegen @ 2016-05-20 14:20 UTC (permalink / raw)
  To: emacs-devel

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> > +                                    (and (> (point) n)
> > +                                         (save-excursion
> > +                                           (backward-char n)
>
> If you want to avoid the error that backward-char can signal, you need
> to take into account that the buffer might be narrowed, so (> (point)
> n) is not sufficient, i.e. you need something like (<= n (- (point)
> (point-min)))

Indeed.  I'll upload the fix soon.


Thanks,

Michael.




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

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

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20160519204636.10168.30801@vcs.savannah.gnu.org>
     [not found] ` <20160519204638.0003A220159@vcs.savannah.gnu.org>
2016-05-20  0:10   ` [elpa] master 3d0a12e 13/24: Fix el-search--ensure-sexp-start error at bob Stefan Monnier
2016-05-20 14:20     ` Michael Heerdegen

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.