From: Miles Bader Date: Tue, 7 Feb 2006 09:08:53 +0000 (+0000) Subject: Revision: emacs@sv.gnu.org/emacs--devo--0--patch-51 X-Git-Tag: emacs-pretest-22.0.90~4330 X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=42ab8e36acd651b047b82658802cbff53474dcaa;p=emacs.git Revision: emacs@sv.gnu.org/emacs--devo--0--patch-51 Make constrain-to-field notice overlays 2006-02-08 Miles Bader * src/editfns.c (Fconstrain_to_field): Use Fget_char_property instead of Fget_text_property (other field functions word with overlays as well as text-properties). --- diff --git a/src/ChangeLog b/src/ChangeLog index fd3bdb55a40..153a0baacbd 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,9 @@ +2006-02-08 Miles Bader + + * editfns.c (Fconstrain_to_field): Use Fget_char_property instead + of Fget_text_property (other field functions word with overlays as + well as text-properties). + 2006-02-07 Kenichi Handa * dispextern.h (unibyte_display_via_language_environment): Extern diff --git a/src/editfns.c b/src/editfns.c index 37fe0f76752..b621397eda3 100644 --- a/src/editfns.c +++ b/src/editfns.c @@ -739,24 +739,24 @@ Field boundaries are not noticed if `inhibit-field-text-motion' is non-nil. */) if (NILP (Vinhibit_field_text_motion) && !EQ (new_pos, old_pos) - && (!NILP (Fget_text_property (new_pos, Qfield, Qnil)) - || !NILP (Fget_text_property (old_pos, Qfield, Qnil)) + && (!NILP (Fget_char_property (new_pos, Qfield, Qnil)) + || !NILP (Fget_char_property (old_pos, Qfield, Qnil)) /* To recognize field boundaries, we must also look at the previous positions; we could use `get_pos_property' instead, but in itself that would fail inside non-sticky fields (like comint prompts). */ || (XFASTINT (new_pos) > BEGV - && !NILP (Fget_text_property (prev_new, Qfield, Qnil))) + && !NILP (Fget_char_property (prev_new, Qfield, Qnil))) || (XFASTINT (old_pos) > BEGV - && !NILP (Fget_text_property (prev_old, Qfield, Qnil)))) + && !NILP (Fget_char_property (prev_old, Qfield, Qnil)))) && (NILP (inhibit_capture_property) /* Field boundaries are again a problem; but now we must decide the case exactly, so we need to call `get_pos_property' as well. */ || (NILP (get_pos_property (old_pos, inhibit_capture_property, Qnil)) && (XFASTINT (old_pos) <= BEGV - || NILP (Fget_text_property (old_pos, inhibit_capture_property, Qnil)) - || NILP (Fget_text_property (prev_old, inhibit_capture_property, Qnil)))))) + || NILP (Fget_char_property (old_pos, inhibit_capture_property, Qnil)) + || NILP (Fget_char_property (prev_old, inhibit_capture_property, Qnil)))))) /* It is possible that NEW_POS is not within the same field as OLD_POS; try to move NEW_POS so that it is. */ {