* lisp.h (struct Lisp_Symbol): Replace field "name" with a lisp
[bpt/emacs.git] / src / intervals.c
index 1e57fd1..2aaaad1 100644 (file)
@@ -1,11 +1,11 @@
 /* Code for doing intervals.
-   Copyright (C) 1993 Free Software Foundation, Inc.
+   Copyright (C) 1993, 1994, 1995, 1997, 1998, 2002 Free Software Foundation, Inc.
 
 This file is part of GNU Emacs.
 
 GNU Emacs is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 1, or (at your option)
+the Free Software Foundation; either version 2, or (at your option)
 any later version.
 
 GNU Emacs is distributed in the hope that it will be useful,
@@ -15,7 +15,8 @@ GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 along with GNU Emacs; see the file COPYING.  If not, write to
-the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
+the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+Boston, MA 02111-1307, USA.  */
 
 
 /* NOTES:
@@ -38,41 +39,53 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 */
 
 
-#include "config.h"
+#include <config.h>
 #include "lisp.h"
 #include "intervals.h"
 #include "buffer.h"
+#include "puresize.h"
+#include "keyboard.h"
+#include "keymap.h"
 
-/* The rest of the file is within this conditional. */
-#ifdef USE_TEXT_PROPERTIES
+/* Test for membership, allowing for t (actually any non-cons) to mean the
+   universal set.  */
 
-/* Factor for weight-balancing interval trees. */
-Lisp_Object interval_balance_threshold;
+#define TMEM(sym, set) (CONSP (set) ? ! NILP (Fmemq (sym, set)) : ! NILP (set))
+
+Lisp_Object merge_properties_sticky ();
+static INTERVAL reproduce_tree P_ ((INTERVAL, INTERVAL));
+static INTERVAL reproduce_tree_obj P_ ((INTERVAL, Lisp_Object));
 \f
-/* Utility functions for intervals. */
+/* Utility functions for intervals.  */
 
 
-/* Create the root interval of some object, a buffer or string. */
+/* Create the root interval of some object, a buffer or string.  */
 
 INTERVAL
 create_root_interval (parent)
      Lisp_Object parent;
 {
-  INTERVAL new = make_interval ();
+  INTERVAL new;
+
+  CHECK_IMPURE (parent);
+
+  new = make_interval ();
 
-  if (XTYPE (parent) == Lisp_Buffer)
+  if (BUFFERP (parent))
     {
-      new->total_length = BUF_Z (XBUFFER (parent)) - 1;
-      XBUFFER (parent)->intervals = new;
+      new->total_length = (BUF_Z (XBUFFER (parent))
+                          - BUF_BEG (XBUFFER (parent)));
+      BUF_INTERVALS (XBUFFER (parent)) = new;
+      new->position = 1;
     }
-  else if (XTYPE (parent) == Lisp_String)
+  else if (STRINGP (parent))
     {
       new->total_length = XSTRING (parent)->size;
       XSTRING (parent)->intervals = new;
+      new->position = 0;
     }
 
-  new->parent = (INTERVAL) parent;
-  new->position = 1;
+  SET_INTERVAL_OBJECT (new, parent);
 
   return new;
 }
@@ -124,14 +137,14 @@ merge_properties (source, target)
 }
 
 /* Return 1 if the two intervals have the same properties,
-   0 otherwise. */
+   0 otherwise.  */
 
 int
 intervals_equal (i0, i1)
      INTERVAL i0, i1;
 {
   register Lisp_Object i0_cdr, i0_sym, i1_val;
-  register i1_len;
+  register int i1_len;
 
   if (DEFAULT_INTERVAL_P (i0) && DEFAULT_INTERVAL_P (i1))
     return 1;
@@ -146,60 +159,85 @@ intervals_equal (i0, i1)
   i0_cdr = i0->plist;
   while (!NILP (i0_cdr))
     {
-      /* Lengths of the two plists were unequal */
+      /* Lengths of the two plists were unequal */
       if (i1_len == 0)
        return 0;
 
       i0_sym = Fcar (i0_cdr);
       i1_val = Fmemq (i0_sym, i1->plist);
 
-      /* i0 has something i1 doesn't */
+      /* i0 has something i1 doesn't */
       if (EQ (i1_val, Qnil))
        return 0;
 
-      /* i0 and i1 both have sym, but it has different values in each */
+      /* i0 and i1 both have sym, but it has different values in each */
       i0_cdr = Fcdr (i0_cdr);
-      if (NILP (Fequal (i1_val, Fcar (i0_cdr))))
+      if (! EQ (Fcar (Fcdr (i1_val)), Fcar (i0_cdr)))
        return 0;
 
       i0_cdr = Fcdr (i0_cdr);
       i1_len--;
     }
 
-  /* Lengths of the two plists were unequal */
+  /* Lengths of the two plists were unequal */
   if (i1_len > 0)
     return 0;
 
   return 1;
 }
 \f
-static int icount;
-static int idepth;
-static int zero_length;
 
 /* Traverse an interval tree TREE, performing FUNCTION on each node.
+   No guarantee is made about the order of traversal.
    Pass FUNCTION two args: an interval, and ARG.  */
 
 void
-traverse_intervals (tree, position, depth, function, arg)
+traverse_intervals_noorder (tree, function, arg)
      INTERVAL tree;
-     int position, depth;
-     void (* function) ();
+     void (* function) P_ ((INTERVAL, Lisp_Object));
      Lisp_Object arg;
 {
-  if (NULL_INTERVAL_P (tree))
-    return;
+  /* Minimize stack usage.  */
+  while (!NULL_INTERVAL_P (tree))
+    {
+      (*function) (tree, arg);
+      if (NULL_INTERVAL_P (tree->right))
+       tree = tree->left;
+      else
+       {
+         traverse_intervals_noorder (tree->left, function, arg);
+         tree = tree->right;
+       }
+    }
+}
+
+/* Traverse an interval tree TREE, performing FUNCTION on each node.
+   Pass FUNCTION two args: an interval, and ARG.  */
 
-  traverse_intervals (tree->left, position, depth + 1, function, arg);
-  position += LEFT_TOTAL_LENGTH (tree);
-  tree->position = position;
-  (*function) (tree, arg);
-  position += LENGTH (tree);
-  traverse_intervals (tree->right, position, depth + 1,  function, arg);
+void
+traverse_intervals (tree, position, function, arg)
+     INTERVAL tree;
+     int position;
+     void (* function) P_ ((INTERVAL, Lisp_Object));
+     Lisp_Object arg;
+{
+  while (!NULL_INTERVAL_P (tree))
+    {
+      traverse_intervals (tree->left, position, function, arg);
+      position += LEFT_TOTAL_LENGTH (tree);
+      tree->position = position;
+      (*function) (tree, arg);
+      position += LENGTH (tree); tree = tree->right;
+    }
 }
 \f
 #if 0
-/* These functions are temporary, for debugging purposes only. */
+
+static int icount;
+static int idepth;
+static int zero_length;
+
+/* These functions are temporary, for debugging purposes only.  */
 
 INTERVAL search_interval, found_interval;
 
@@ -221,7 +259,7 @@ search_for_interval (i, tree)
   icount = 0;
   search_interval = i;
   found_interval = NULL_INTERVAL;
-  traverse_intervals (tree, 1, 0, &check_for_interval, Qnil);
+  traverse_intervals_noorder (tree, &check_for_interval, Qnil);
   return found_interval;
 }
 
@@ -243,7 +281,7 @@ count_intervals (i)
   icount = 0;
   idepth = 0;
   zero_length = 0;
-  traverse_intervals (i, 1, 0, &inc_interval_count, Qnil);
+  traverse_intervals_noorder (i, &inc_interval_count, Qnil);
 
   return icount;
 }
@@ -255,7 +293,7 @@ root_interval (interval)
   register INTERVAL i = interval;
 
   while (! ROOT_INTERVAL_P (i))
-    i = i->parent;
+    i = INTERVAL_PARENT (i);
 
   return i;
 }
@@ -270,40 +308,43 @@ root_interval (interval)
      c           c
 */
 
-static INTERVAL
+static INLINE INTERVAL
 rotate_right (interval)
      INTERVAL interval;
 {
   INTERVAL i;
   INTERVAL B = interval->left;
-  int len = LENGTH (interval);
+  int old_total = interval->total_length;
 
-  /* Deal with any Parent of A;  make it point to B. */
+  /* Deal with any Parent of A;  make it point to B.  */
   if (! ROOT_INTERVAL_P (interval))
-    if (AM_LEFT_CHILD (interval))
-      interval->parent->left = interval->left;
-    else
-      interval->parent->right = interval->left;
-  interval->left->parent = interval->parent;
-
-  /* B gets the same length as A, since it get A's position in the tree. */
-  interval->left->total_length = interval->total_length;
+    {
+      if (AM_LEFT_CHILD (interval))
+       INTERVAL_PARENT (interval)->left = B;
+      else
+       INTERVAL_PARENT (interval)->right = B;
+    }
+  COPY_INTERVAL_PARENT (B, interval);
 
-  /* B becomes the parent of A. */
-  i = interval->left->right;
-  interval->left->right = interval;
-  interval->parent = interval->left;
+  /* Make B the parent of A */
+  i = B->right;
+  B->right = interval;
+  SET_INTERVAL_PARENT (interval, B);
 
-  /* A gets c as left child. */
+  /* Make A point to c */
   interval->left = i;
   if (! NULL_INTERVAL_P (i))
-    i->parent = interval;
-  interval->total_length = (len + LEFT_TOTAL_LENGTH (interval)
-                           + RIGHT_TOTAL_LENGTH (interval));
+    SET_INTERVAL_PARENT (i, interval);
+
+  /* A's total length is decreased by the length of B and its left child.  */
+  interval->total_length -= B->total_length - LEFT_TOTAL_LENGTH (interval);
+
+  /* B must have the same total length of A.  */
+  B->total_length = old_total;
 
   return B;
 }
-\f
+
 /* Assuming that a right child exists, perform the following operation:
 
     A               B   
@@ -313,43 +354,141 @@ rotate_right (interval)
     c               c
 */
 
-static INTERVAL
+static INLINE INTERVAL
 rotate_left (interval)
      INTERVAL interval;
 {
   INTERVAL i;
   INTERVAL B = interval->right;
-  int len = LENGTH (interval);
+  int old_total = interval->total_length;
 
-  /* Deal with the parent of A. */
+  /* Deal with any parent of A;  make it point to B.  */
   if (! ROOT_INTERVAL_P (interval))
-    if (AM_LEFT_CHILD (interval))
-      interval->parent->left = interval->right;
-    else
-      interval->parent->right = interval->right;
-  interval->right->parent = interval->parent;
-
-  /* B must have the same total length of A. */
-  interval->right->total_length = interval->total_length;
+    {
+      if (AM_LEFT_CHILD (interval))
+       INTERVAL_PARENT (interval)->left = B;
+      else
+       INTERVAL_PARENT (interval)->right = B;
+    }
+  COPY_INTERVAL_PARENT (B, interval);
 
   /* Make B the parent of A */
-  i = interval->right->left;
-  interval->right->left = interval;
-  interval->parent = interval->right;
+  i = B->left;
+  B->left = interval;
+  SET_INTERVAL_PARENT (interval, B);
 
   /* Make A point to c */
   interval->right = i;
   if (! NULL_INTERVAL_P (i))
-    i->parent = interval;
-  interval->total_length = (len + LEFT_TOTAL_LENGTH (interval)
-                           + RIGHT_TOTAL_LENGTH (interval));
+    SET_INTERVAL_PARENT (i, interval);
+
+  /* A's total length is decreased by the length of B and its right child.  */
+  interval->total_length -= B->total_length - RIGHT_TOTAL_LENGTH (interval);
+
+  /* B must have the same total length of A.  */
+  B->total_length = old_total;
 
   return B;
 }
 \f
-/* Split INTERVAL into two pieces, starting the second piece at character
-   position OFFSET (counting from 1), relative to INTERVAL.  The right-hand
-   piece (second, lexicographically) is returned.
+/* Balance an interval tree with the assumption that the subtrees
+   themselves are already balanced.  */
+
+static INTERVAL
+balance_an_interval (i)
+     INTERVAL i;
+{
+  register int old_diff, new_diff;
+
+  while (1)
+    {
+      old_diff = LEFT_TOTAL_LENGTH (i) - RIGHT_TOTAL_LENGTH (i);
+      if (old_diff > 0)
+       {
+         new_diff = i->total_length - i->left->total_length
+           + RIGHT_TOTAL_LENGTH (i->left) - LEFT_TOTAL_LENGTH (i->left);
+         if (abs (new_diff) >= old_diff)
+           break;
+         i = rotate_right (i);
+         balance_an_interval (i->right);
+       }
+      else if (old_diff < 0)
+       {
+         new_diff = i->total_length - i->right->total_length
+           + LEFT_TOTAL_LENGTH (i->right) - RIGHT_TOTAL_LENGTH (i->right);
+         if (abs (new_diff) >= -old_diff)
+           break;
+         i = rotate_left (i);
+         balance_an_interval (i->left);
+       }
+      else
+       break;
+    }
+  return i;
+}
+
+/* Balance INTERVAL, potentially stuffing it back into its parent
+   Lisp Object.  */
+
+static INLINE INTERVAL
+balance_possible_root_interval (interval)
+     register INTERVAL interval;
+{
+  Lisp_Object parent;
+  int have_parent = 0;
+
+  if (!INTERVAL_HAS_OBJECT (interval) && !INTERVAL_HAS_PARENT (interval))
+    return interval;
+
+  if (INTERVAL_HAS_OBJECT (interval))
+    {
+      have_parent = 1;
+      GET_INTERVAL_OBJECT (parent, interval);
+    }
+  interval = balance_an_interval (interval);
+
+  if (have_parent)
+    {
+      if (BUFFERP (parent))
+       BUF_INTERVALS (XBUFFER (parent)) = interval;
+      else if (STRINGP (parent))
+       XSTRING (parent)->intervals = interval;
+    }
+
+  return interval;
+}
+
+/* Balance the interval tree TREE.  Balancing is by weight
+   (the amount of text).  */
+
+static INTERVAL
+balance_intervals_internal (tree)
+     register INTERVAL tree;
+{
+  /* Balance within each side.  */
+  if (tree->left)
+    balance_intervals_internal (tree->left);
+  if (tree->right)
+    balance_intervals_internal (tree->right);
+  return balance_an_interval (tree);
+}
+
+/* Advertised interface to balance intervals.  */
+
+INTERVAL
+balance_intervals (tree)
+     INTERVAL tree;
+{
+  if (tree == NULL_INTERVAL)
+    return NULL_INTERVAL;
+
+  return balance_intervals_internal (tree);
+}
+\f
+/* Split INTERVAL into two pieces, starting the second piece at
+   character position OFFSET (counting from 0), relative to INTERVAL.
+   INTERVAL becomes the left-hand piece, and the right-hand piece
+   (second, lexicographically) is returned.
 
    The size and position fields of the two intervals are set based upon
    those of the original interval.  The property list of the new interval
@@ -357,7 +496,7 @@ rotate_left (interval)
    result.
 
    Note that this does not change the position of INTERVAL;  if it is a root,
-   it is still a root after this operation. */
+   it is still a root after this operation.  */
 
 INTERVAL
 split_interval_right (interval, offset)
@@ -366,32 +505,35 @@ split_interval_right (interval, offset)
 {
   INTERVAL new = make_interval ();
   int position = interval->position;
-  int new_length = LENGTH (interval) - offset + 1;
+  int new_length = LENGTH (interval) - offset;
 
-  new->position = position + offset - 1;
-  new->parent = interval;
+  new->position = position + offset;
+  SET_INTERVAL_PARENT (new, interval);
 
-  if (LEAF_INTERVAL_P (interval) || NULL_RIGHT_CHILD (interval))
+  if (NULL_RIGHT_CHILD (interval))
     {
       interval->right = new;
       new->total_length = new_length;
-
-      return new;
     }
-
-  /* Insert the new node between INTERVAL and its right child. */
-  new->right = interval->right;
-  interval->right->parent = new;
-  interval->right = new;
-
-  new->total_length = new_length + new->right->total_length;
+  else
+    {
+      /* Insert the new node between INTERVAL and its right child.  */
+      new->right = interval->right;
+      SET_INTERVAL_PARENT (interval->right, new);
+      interval->right = new;
+      new->total_length = new_length + new->right->total_length;
+      balance_an_interval (new);
+    }
+  
+  balance_possible_root_interval (interval);
 
   return new;
 }
 
-/* Split INTERVAL into two pieces, starting the second piece at character
-   position OFFSET (counting from 1), relative to INTERVAL.  The left-hand
-   piece (first, lexicographically) is returned.
+/* Split INTERVAL into two pieces, starting the second piece at
+   character position OFFSET (counting from 0), relative to INTERVAL.
+   INTERVAL becomes the right-hand piece, and the left-hand piece
+   (first, lexicographically) is returned.
 
    The size and position fields of the two intervals are set based upon
    those of the original interval.  The property list of the new interval
@@ -399,7 +541,7 @@ split_interval_right (interval, offset)
    result.
 
    Note that this does not change the position of INTERVAL;  if it is a root,
-   it is still a root after this operation. */
+   it is still a root after this operation.  */
 
 INTERVAL
 split_interval_left (interval, offset)
@@ -407,62 +549,103 @@ split_interval_left (interval, offset)
      int offset;
 {
   INTERVAL new = make_interval ();
-  int position = interval->position;
-  int new_length = offset - 1;
+  int new_length = offset;
 
   new->position = interval->position;
-  interval->position = interval->position + offset - 1;
-  new->parent = interval;
+  interval->position = interval->position + offset;
+  SET_INTERVAL_PARENT (new, interval);
 
   if (NULL_LEFT_CHILD (interval))
     {
       interval->left = new;
       new->total_length = new_length;
-
-      return new;
     }
-
-  /* Insert the new node between INTERVAL and its left child. */
-  new->left = interval->left;
-  new->left->parent = new;
-  interval->left = new;
-  new->total_length = new_length + LEFT_TOTAL_LENGTH (new);
+  else
+    {
+      /* Insert the new node between INTERVAL and its left child.  */
+      new->left = interval->left;
+      SET_INTERVAL_PARENT (new->left, new);
+      interval->left = new;
+      new->total_length = new_length + new->left->total_length;
+      balance_an_interval (new);
+    }
+  
+  balance_possible_root_interval (interval);
 
   return new;
 }
 \f
+/* Return the proper position for the first character
+   described by the interval tree SOURCE.
+   This is 1 if the parent is a buffer,
+   0 if the parent is a string or if there is no parent.
+
+   Don't use this function on an interval which is the child
+   of another interval!  */
+
+int
+interval_start_pos (source)
+     INTERVAL source;
+{
+  Lisp_Object parent;
+
+  if (NULL_INTERVAL_P (source))
+    return 0;
+
+  if (! INTERVAL_HAS_OBJECT (source))
+    return 0;
+  GET_INTERVAL_OBJECT (parent, source);
+  if (BUFFERP (parent))
+    return BUF_BEG (XBUFFER (parent));
+  return 0;
+}
+
 /* Find the interval containing text position POSITION in the text
-   represented by the interval tree TREE.  POSITION is relative to
-   the beginning of that text.
+   represented by the interval tree TREE.  POSITION is a buffer
+   position (starting from 1) or a string index (starting from 0).
+   If POSITION is at the end of the buffer or string,
+   return the interval containing the last character.
 
    The `position' field, which is a cache of an interval's position,
    is updated in the interval found.  Other functions (e.g., next_interval)
-   will update this cache based on the result of find_interval. */
+   will update this cache based on the result of find_interval.  */
 
-INLINE INTERVAL
+INTERVAL
 find_interval (tree, position)
      register INTERVAL tree;
      register int position;
 {
-  register int relative_position = position;
+  /* The distance from the left edge of the subtree at TREE
+                    to POSITION.  */
+  register int relative_position;
 
   if (NULL_INTERVAL_P (tree))
     return NULL_INTERVAL;
 
-  if (position > TOTAL_LENGTH (tree))
+  relative_position = position;
+  if (INTERVAL_HAS_OBJECT (tree))
+    {
+      Lisp_Object parent;
+      GET_INTERVAL_OBJECT (parent, tree);
+      if (BUFFERP (parent))
+       relative_position -= BUF_BEG (XBUFFER (parent));
+    }
+
+  if (relative_position > TOTAL_LENGTH (tree))
     abort ();                  /* Paranoia */
-#if 0
-    position = TOTAL_LENGTH (tree);
-#endif
+
+  if (!handling_signal)
+    tree = balance_possible_root_interval (tree);
 
   while (1)
     {
-      if (relative_position <= LEFT_TOTAL_LENGTH (tree))
+      if (relative_position < LEFT_TOTAL_LENGTH (tree))
        {
          tree = tree->left;
        }
-      else if (relative_position > (TOTAL_LENGTH (tree)
-                                   - RIGHT_TOTAL_LENGTH (tree)))
+      else if (! NULL_RIGHT_CHILD (tree)
+              && relative_position >= (TOTAL_LENGTH (tree)
+                                       - RIGHT_TOTAL_LENGTH (tree)))
        {
          relative_position -= (TOTAL_LENGTH (tree)
                                - RIGHT_TOTAL_LENGTH (tree));
@@ -470,8 +653,10 @@ find_interval (tree, position)
        }
       else
        {
-         tree->position = LEFT_TOTAL_LENGTH (tree)
-                          + position - relative_position + 1;
+         tree->position
+           = (position - relative_position /* the left edge of *tree */
+              + LEFT_TOTAL_LENGTH (tree)); /* the left edge of this interval */
+
          return tree;
        }
     }
@@ -479,7 +664,7 @@ find_interval (tree, position)
 \f
 /* Find the succeeding interval (lexicographically) to INTERVAL.
    Sets the `position' field based on that of INTERVAL (see
-   find_interval). */
+   find_interval).  */
 
 INTERVAL
 next_interval (interval)
@@ -506,12 +691,12 @@ next_interval (interval)
     {
       if (AM_LEFT_CHILD (i))
        {
-         i = i->parent;
+         i = INTERVAL_PARENT (i);
          i->position = next_position;
          return i;
        }
 
-      i = i->parent;
+      i = INTERVAL_PARENT (i);
     }
 
   return NULL_INTERVAL;
@@ -519,14 +704,13 @@ next_interval (interval)
 
 /* Find the preceding interval (lexicographically) to INTERVAL.
    Sets the `position' field based on that of INTERVAL (see
-   find_interval). */
+   find_interval).  */
 
 INTERVAL
 previous_interval (interval)
      register INTERVAL interval;
 {
   register INTERVAL i;
-  register position_of_previous;
 
   if (NULL_INTERVAL_P (interval))
     return NULL_INTERVAL;
@@ -546,16 +730,67 @@ previous_interval (interval)
     {
       if (AM_RIGHT_CHILD (i))
        {
-         i = i->parent;
+         i = INTERVAL_PARENT (i);
 
          i->position = interval->position - LENGTH (i);
          return i;
        }
-      i = i->parent;
+      i = INTERVAL_PARENT (i);
     }
 
   return NULL_INTERVAL;
 }
+
+/* Find the interval containing POS given some non-NULL INTERVAL
+   in the same tree.  Note that we need to update interval->position
+   if we go down the tree.
+   To speed up the process, we assume that the ->position of
+   I and all its parents is already uptodate.  */
+INTERVAL
+update_interval (i, pos)
+     register INTERVAL i;
+     int pos;
+{
+  if (NULL_INTERVAL_P (i))
+    return NULL_INTERVAL;
+
+  while (1) 
+    {
+      if (pos < i->position) 
+       {
+         /* Move left. */
+         if (pos >= i->position - TOTAL_LENGTH (i->left)) 
+           {
+             i->left->position = i->position - TOTAL_LENGTH (i->left)
+               + LEFT_TOTAL_LENGTH (i->left);
+             i = i->left;              /* Move to the left child */
+           }
+         else if (NULL_PARENT (i)) 
+           error ("Point before start of properties");
+         else  
+             i = INTERVAL_PARENT (i);
+         continue;
+       }
+      else if (pos >= INTERVAL_LAST_POS (i))
+       {
+         /* Move right. */
+         if (pos < INTERVAL_LAST_POS (i) + TOTAL_LENGTH (i->right)) 
+           {
+             i->right->position = INTERVAL_LAST_POS (i) +
+               LEFT_TOTAL_LENGTH (i->right);
+             i = i->right;             /* Move to the right child */
+           }
+         else if (NULL_PARENT (i)) 
+           error ("Point after end of properties");
+         else 
+             i = INTERVAL_PARENT (i);
+         continue;
+       }
+      else 
+       return i;
+    }
+}
+
 \f
 #if 0
 /* Traverse a path down the interval tree TREE to the interval
@@ -567,8 +802,8 @@ previous_interval (interval)
    Modifications are needed to handle the hungry bits -- after simply
    finding the interval at position (don't add length going down),
    if it's the beginning of the interval, get the previous interval
-   and check the hugry bits of both.  Then add the length going back up
-   to the root. */
+   and check the hungry bits of both.  Then add the length going back up
+   to the root.  */
 
 static INTERVAL
 adjust_intervals_for_insertion (tree, position, length)
@@ -606,7 +841,7 @@ adjust_intervals_for_insertion (tree, position, length)
       else
        {
          /* If we are to use zero-length intervals as buffer pointers,
-            then this code will have to change. */
+            then this code will have to change.  */
          this->total_length += length;
          this->position = LEFT_TOTAL_LENGTH (this)
                           + position - relative_position + 1;
@@ -618,7 +853,7 @@ adjust_intervals_for_insertion (tree, position, length)
 
 /* Effect an adjustment corresponding to the addition of LENGTH characters
    of text.  Do this by finding the interval containing POSITION in the
-   interval tree TREE, and then adjusting all of it's ancestors by adding
+   interval tree TREE, and then adjusting all of its ancestors by adding
    LENGTH to them.
 
    If POSITION is the first character of an interval, meaning that point
@@ -627,7 +862,7 @@ adjust_intervals_for_insertion (tree, position, length)
 
    If both intervals are "sticky", then make them belong to the left-most
    interval.  Another possibility would be to create a new interval for
-   this text, and make it have the merged properties of both ends. */
+   this text, and make it have the merged properties of both ends.  */
 
 static INTERVAL
 adjust_intervals_for_insertion (tree, position, length)
@@ -635,42 +870,361 @@ adjust_intervals_for_insertion (tree, position, length)
      int position, length;
 {
   register INTERVAL i;
-
+  register INTERVAL temp;
+  int eobp = 0;
+  Lisp_Object parent;
+  int offset;
+  
   if (TOTAL_LENGTH (tree) == 0)        /* Paranoia */
     abort ();
 
-  /* If inserting at point-max of a buffer, that position
-     will be out of range. */
-  if (position > TOTAL_LENGTH (tree))
-    position = TOTAL_LENGTH (tree);
+  GET_INTERVAL_OBJECT (parent, tree);
+  offset = (BUFFERP (parent) ? BUF_BEG (XBUFFER (parent)) : 0);
+
+  /* If inserting at point-max of a buffer, that position will be out
+     of range.  Remember that buffer positions are 1-based.  */
+  if (position >= TOTAL_LENGTH (tree) + offset)
+    {
+      position = TOTAL_LENGTH (tree) + offset;
+      eobp = 1;
+    }
 
   i = find_interval (tree, position);
-  /* If we are positioned between intervals, check the stickiness of
-     both of them. */
-  if (position == i->position
-      && position != 1)
+
+  /* If in middle of an interval which is not sticky either way,
+     we must not just give its properties to the insertion.
+     So split this interval at the insertion point.
+
+     Originally, the if condition here was this:
+       (! (position == i->position || eobp)
+        && END_NONSTICKY_P (i)
+        && FRONT_NONSTICKY_P (i))
+     But, these macros are now unreliable because of introduction of
+     Vtext_property_default_nonsticky.  So, we always check properties
+     one by one if POSITION is in middle of an interval.  */
+  if (! (position == i->position || eobp))
     {
-      register INTERVAL prev = previous_interval (i);
+      Lisp_Object tail;
+      Lisp_Object front, rear;
+
+      tail = i->plist;
+
+      /* Properties font-sticky and rear-nonsticky override
+         Vtext_property_default_nonsticky.  So, if they are t, we can
+         skip one by one checking of properties.  */
+      rear = textget (i->plist, Qrear_nonsticky);
+      if (! CONSP (rear) && ! NILP (rear))
+       {
+         /* All properties are nonsticky.  We split the interval.  */
+         goto check_done;
+       }
+      front = textget (i->plist, Qfront_sticky);
+      if (! CONSP (front) && ! NILP (front))
+       {
+         /* All properties are sticky.  We don't split the interval.  */
+         tail = Qnil;
+         goto check_done;
+       }
+
+      /* Does any actual property pose an actual problem?  We break
+         the loop if we find a nonsticky property.  */
+      for (; CONSP (tail); tail = Fcdr (XCDR (tail)))
+       {
+         Lisp_Object prop, tmp;
+         prop = XCAR (tail);
+
+         /* Is this particular property front-sticky?  */
+         if (CONSP (front) && ! NILP (Fmemq (prop, front)))
+           continue;
 
-      /* If both intervals are sticky here, then default to the
-         left-most one.  But perhaps we should create a new
-        interval here instead... */
-      if (END_STICKY_P (prev) || ! FRONT_STICKY_P (i))
-       i = prev;
+         /* Is this particular property rear-nonsticky?  */
+         if (CONSP (rear) && ! NILP (Fmemq (prop, rear)))
+           break;
+
+         /* Is this particular property recorded as sticky or
+             nonsticky in Vtext_property_default_nonsticky?  */
+         tmp = Fassq (prop, Vtext_property_default_nonsticky);
+         if (CONSP (tmp))
+           {
+             if (NILP (tmp))
+               continue;
+             break;
+           }
+
+         /* By default, a text property is rear-sticky, thus we
+            continue the loop.  */
+       }
+
+    check_done:
+      /* If any property is a real problem, split the interval.  */
+      if (! NILP (tail))
+       {
+         temp = split_interval_right (i, position - i->position);
+         copy_properties (i, temp);
+         i = temp;
+       }
     }
 
-  while (! NULL_INTERVAL_P (i))
+  /* If we are positioned between intervals, check the stickiness of
+     both of them.  We have to do this too, if we are at BEG or Z.  */
+  if (position == i->position || eobp)
     {
-      i->total_length += length;
-      i = i->parent;
+      register INTERVAL prev;
+
+      if (position == BEG)
+       prev = 0;
+      else if (eobp)
+       {
+         prev = i;
+         i = 0;
+       }
+      else
+       prev = previous_interval (i);
+
+      /* Even if we are positioned between intervals, we default
+        to the left one if it exists.  We extend it now and split
+        off a part later, if stickiness demands it.  */
+      for (temp = prev ? prev : i; temp; temp = INTERVAL_PARENT_OR_NULL (temp))
+       {
+         temp->total_length += length;
+         temp = balance_possible_root_interval (temp);
+       }
+      
+      /* If at least one interval has sticky properties,
+        we check the stickiness property by property.
+
+        Originally, the if condition here was this:
+               (END_NONSTICKY_P (prev) || FRONT_STICKY_P (i))
+        But, these macros are now unreliable because of introduction
+        of Vtext_property_default_nonsticky.  So, we always have to
+        check stickiness of properties one by one.  If cache of
+        stickiness is implemented in the future, we may be able to
+        use those macros again.  */
+      if (1)
+       {
+         Lisp_Object pleft, pright;
+         struct interval newi;
+
+         pleft = NULL_INTERVAL_P (prev) ? Qnil : prev->plist;
+         pright = NULL_INTERVAL_P (i) ? Qnil : i->plist;
+         newi.plist = merge_properties_sticky (pleft, pright);
+
+         if (! prev) /* i.e. position == BEG */
+           {
+             if (! intervals_equal (i, &newi))
+               {
+                 i = split_interval_left (i, length);
+                 i->plist = newi.plist;
+               }
+           }
+         else if (! intervals_equal (prev, &newi))
+           {
+             prev = split_interval_right (prev,
+                                          position - prev->position);
+             prev->plist = newi.plist;
+             if (! NULL_INTERVAL_P (i)
+                 && intervals_equal (prev, i))
+               merge_interval_right (prev);
+           }
+
+         /* We will need to update the cache here later.  */
+       }
+      else if (! prev && ! NILP (i->plist))
+        {
+         /* Just split off a new interval at the left.
+            Since I wasn't front-sticky, the empty plist is ok.  */
+         i = split_interval_left (i, length);
+        }
     }
 
+  /* Otherwise just extend the interval.  */
+  else
+    {
+      for (temp = i; temp; temp = INTERVAL_PARENT_OR_NULL (temp))
+       {
+         temp->total_length += length;
+         temp = balance_possible_root_interval (temp);
+       }
+    }
+      
   return tree;
 }
+
+/* Any property might be front-sticky on the left, rear-sticky on the left,
+   front-sticky on the right, or rear-sticky on the right; the 16 combinations
+   can be arranged in a matrix with rows denoting the left conditions and
+   columns denoting the right conditions:
+      _  __  _
+_     FR FR FR FR
+FR__   0  1  2  3
+ _FR   4  5  6  7
+FR     8  9  A  B
+  FR   C  D  E  F
+
+   left-props  = '(front-sticky (p8 p9 pa pb pc pd pe pf)
+                  rear-nonsticky (p4 p5 p6 p7 p8 p9 pa pb)
+                  p0 L p1 L p2 L p3 L p4 L p5 L p6 L p7 L
+                  p8 L p9 L pa L pb L pc L pd L pe L pf L)
+   right-props = '(front-sticky (p2 p3 p6 p7 pa pb pe pf)
+                  rear-nonsticky (p1 p2 p5 p6 p9 pa pd pe)
+                  p0 R p1 R p2 R p3 R p4 R p5 R p6 R p7 R
+                  p8 R p9 R pa R pb R pc R pd R pe R pf R)
+
+   We inherit from whoever has a sticky side facing us.  If both sides
+   do (cases 2, 3, E, and F), then we inherit from whichever side has a
+   non-nil value for the current property.  If both sides do, then we take
+   from the left.
+
+   When we inherit a property, we get its stickiness as well as its value.
+   So, when we merge the above two lists, we expect to get this:
+
+   result      = '(front-sticky (p6 p7 pa pb pc pd pe pf)
+                  rear-nonsticky (p6 pa)
+                  p0 L p1 L p2 L p3 L p6 R p7 R
+                  pa R pb R pc L pd L pe L pf L)
+
+   The optimizable special cases are:
+       left rear-nonsticky = nil, right front-sticky = nil (inherit left)
+       left rear-nonsticky = t,   right front-sticky = t   (inherit right)
+       left rear-nonsticky = t,   right front-sticky = nil (inherit none)
+*/
+
+Lisp_Object
+merge_properties_sticky (pleft, pright)
+     Lisp_Object pleft, pright;
+{
+  register Lisp_Object props, front, rear;
+  Lisp_Object lfront, lrear, rfront, rrear;
+  register Lisp_Object tail1, tail2, sym, lval, rval, cat;
+  int use_left, use_right;
+  int lpresent;
+
+  props = Qnil;
+  front = Qnil;
+  rear  = Qnil;
+  lfront = textget (pleft, Qfront_sticky);
+  lrear  = textget (pleft, Qrear_nonsticky);
+  rfront = textget (pright, Qfront_sticky);
+  rrear  = textget (pright, Qrear_nonsticky);
+
+  /* Go through each element of PRIGHT.  */
+  for (tail1 = pright; CONSP (tail1); tail1 = Fcdr (Fcdr (tail1)))
+    {
+      Lisp_Object tmp;
+
+      sym = Fcar (tail1);
+
+      /* Sticky properties get special treatment.  */
+      if (EQ (sym, Qrear_nonsticky) || EQ (sym, Qfront_sticky))
+       continue;
+
+      rval = Fcar (Fcdr (tail1));
+      for (tail2 = pleft; CONSP (tail2); tail2 = Fcdr (Fcdr (tail2)))
+       if (EQ (sym, Fcar (tail2)))
+         break;
+
+      /* Indicate whether the property is explicitly defined on the left.
+        (We know it is defined explicitly on the right
+        because otherwise we don't get here.)  */
+      lpresent = ! NILP (tail2);
+      lval = (NILP (tail2) ? Qnil : Fcar (Fcdr (tail2)));
+
+      /* Even if lrear or rfront say nothing about the stickiness of
+        SYM, Vtext_property_default_nonsticky may give default
+        stickiness to SYM.  */
+      tmp = Fassq (sym, Vtext_property_default_nonsticky);
+      use_left = (lpresent
+                 && ! (TMEM (sym, lrear)
+                       || (CONSP (tmp) && ! NILP (XCDR (tmp)))));
+      use_right = (TMEM (sym, rfront)
+                  || (CONSP (tmp) && NILP (XCDR (tmp))));
+      if (use_left && use_right)
+       {
+         if (NILP (lval))
+           use_left = 0;
+         else if (NILP (rval))
+           use_right = 0;
+       }
+      if (use_left)
+       {
+         /* We build props as (value sym ...) rather than (sym value ...)
+            because we plan to nreverse it when we're done.  */
+         props = Fcons (lval, Fcons (sym, props));
+         if (TMEM (sym, lfront))
+           front = Fcons (sym, front);
+         if (TMEM (sym, lrear))
+           rear = Fcons (sym, rear);
+       }
+      else if (use_right)
+       {
+         props = Fcons (rval, Fcons (sym, props));
+         if (TMEM (sym, rfront))
+           front = Fcons (sym, front);
+         if (TMEM (sym, rrear))
+           rear = Fcons (sym, rear);
+       }
+    }
+
+  /* Now go through each element of PLEFT.  */
+  for (tail2 = pleft; CONSP (tail2); tail2 = Fcdr (Fcdr (tail2)))
+    {
+      Lisp_Object tmp;
+
+      sym = Fcar (tail2);
+
+      /* Sticky properties get special treatment.  */
+      if (EQ (sym, Qrear_nonsticky) || EQ (sym, Qfront_sticky))
+       continue;
+
+      /* If sym is in PRIGHT, we've already considered it.  */
+      for (tail1 = pright; CONSP (tail1); tail1 = Fcdr (Fcdr (tail1)))
+       if (EQ (sym, Fcar (tail1)))
+         break;
+      if (! NILP (tail1))
+       continue;
+
+      lval = Fcar (Fcdr (tail2));
+
+      /* Even if lrear or rfront say nothing about the stickiness of
+        SYM, Vtext_property_default_nonsticky may give default
+        stickiness to SYM.  */
+      tmp = Fassq (sym, Vtext_property_default_nonsticky);
+
+      /* Since rval is known to be nil in this loop, the test simplifies.  */
+      if (! (TMEM (sym, lrear) || (CONSP (tmp) && ! NILP (XCDR (tmp)))))
+       {
+         props = Fcons (lval, Fcons (sym, props));
+         if (TMEM (sym, lfront))
+           front = Fcons (sym, front);
+       }
+      else if (TMEM (sym, rfront) || (CONSP (tmp) && NILP (XCDR (tmp))))
+       {
+         /* The value is nil, but we still inherit the stickiness
+            from the right.  */
+         front = Fcons (sym, front);
+         if (TMEM (sym, rrear))
+           rear = Fcons (sym, rear);
+       }
+    }
+  props = Fnreverse (props);
+  if (! NILP (rear))
+    props = Fcons (Qrear_nonsticky, Fcons (Fnreverse (rear), props));
+
+  cat = textget (props, Qcategory);
+  if (! NILP (front)
+      && 
+      /* If we have inherited a front-stick category property that is t,
+        we don't need to set up a detailed one.  */
+      ! (! NILP (cat) && SYMBOLP (cat)
+        && EQ (Fget (cat, Qfront_sticky), Qt)))
+    props = Fcons (Qfront_sticky, Fcons (Fnreverse (front), props));
+  return props;
+}
+
 \f
 /* Delete an node I from its interval tree by merging its subtrees
    into one subtree which is then returned.  Caller is responsible for
-   storing the resulting subtree into its parent. */
+   storing the resulting subtree into its parent.  */
 
 static INTERVAL
 delete_node (i)
@@ -694,7 +1248,7 @@ delete_node (i)
       this->total_length += migrate_amt;
     }
   this->left = migrate;
-  migrate->parent = this;
+  SET_INTERVAL_PARENT (migrate, this);
 
   return i->right;
 }
@@ -703,7 +1257,7 @@ delete_node (i)
    and properly connecting the resultant subtree.
 
    I is presumed to be empty; that is, no adjustments are made
-   for the length of I. */
+   for the length of I.  */
 
 void
 delete_interval (i)
@@ -712,19 +1266,20 @@ delete_interval (i)
   register INTERVAL parent;
   int amt = LENGTH (i);
 
-  if (amt > 0)                 /* Only used on zero-length intervals now. */
+  if (amt > 0)                 /* Only used on zero-length intervals now.  */
     abort ();
 
   if (ROOT_INTERVAL_P (i))
     {
-      Lisp_Object owner = (Lisp_Object) i->parent;
+      Lisp_Object owner;
+      GET_INTERVAL_OBJECT (owner, i);
       parent = delete_node (i);
       if (! NULL_INTERVAL_P (parent))
-       parent->parent = (INTERVAL) owner;
+       SET_INTERVAL_OBJECT (parent, owner);
 
-      if (XTYPE (owner) == Lisp_Buffer)
-       XBUFFER (owner)->intervals = parent;
-      else if (XTYPE (owner) == Lisp_String)
+      if (BUFFERP (owner))
+       BUF_INTERVALS (XBUFFER (owner)) = parent;
+      else if (STRINGP (owner))
        XSTRING (owner)->intervals = parent;
       else
        abort ();
@@ -732,29 +1287,32 @@ delete_interval (i)
       return;
     }
 
-  parent = i->parent;
+  parent = INTERVAL_PARENT (i);
   if (AM_LEFT_CHILD (i))
     {
       parent->left = delete_node (i);
       if (! NULL_INTERVAL_P (parent->left))
-       parent->left->parent = parent;
+       SET_INTERVAL_PARENT (parent->left, parent);
     }
   else
     {
       parent->right = delete_node (i);
       if (! NULL_INTERVAL_P (parent->right))
-       parent->right->parent = parent;
+       SET_INTERVAL_PARENT (parent->right, parent);
     }
 }
 \f
-/* Find the interval in TREE corresponding to the character position FROM
-   and delete as much as possible of AMOUNT from that interval, starting
-   after the relative position of FROM within it.  Return the amount
-   actually deleted, and if the interval was zeroed-out, delete that
-   interval node from the tree.
+/* Find the interval in TREE corresponding to the relative position
+   FROM and delete as much as possible of AMOUNT from that interval.
+   Return the amount actually deleted, and if the interval was
+   zeroed-out, delete that interval node from the tree.
+
+   Note that FROM is actually origin zero, aka relative to the
+   leftmost edge of tree.  This is appropriate since we call ourselves
+   recursively on subtrees.
 
    Do this by recursing down TREE to the interval in question, and
-   deleting the appropriate amount of text. */
+   deleting the appropriate amount of text.  */
 
 static int
 interval_deletion_adjustment (tree, from, amount)
@@ -767,7 +1325,7 @@ interval_deletion_adjustment (tree, from, amount)
     return 0;
 
   /* Left branch */
-  if (relative_position <= LEFT_TOTAL_LENGTH (tree))
+  if (relative_position < LEFT_TOTAL_LENGTH (tree))
     {
       int subtract = interval_deletion_adjustment (tree->left,
                                                   relative_position,
@@ -776,8 +1334,8 @@ interval_deletion_adjustment (tree, from, amount)
       return subtract;
     }
   /* Right branch */
-  else if (relative_position > (TOTAL_LENGTH (tree)
-                               - RIGHT_TOTAL_LENGTH (tree)))
+  else if (relative_position >= (TOTAL_LENGTH (tree)
+                                - RIGHT_TOTAL_LENGTH (tree)))
     {
       int subtract;
 
@@ -789,58 +1347,31 @@ interval_deletion_adjustment (tree, from, amount)
       tree->total_length -= subtract;
       return subtract;
     }
-  /* Here -- this node */
+  /* Here -- this node */
   else
     {
-      /* If this is a zero-length, marker interval, then
-        we must skip it. */
-
-      if (relative_position == LEFT_TOTAL_LENGTH (tree) + 1)
-       {
-         /* This means we're deleting from the beginning of this interval. */
-         register int my_amount = LENGTH (tree);
-
-         if (amount < my_amount)
-           {
-             tree->total_length -= amount;
-             return amount;
-           }
-         else
-           {
-             tree->total_length -= my_amount;
-             if (LENGTH (tree) != 0)
-               abort ();       /* Paranoia */
-
-             delete_interval (tree);
-             return my_amount;
-           }
-       }
-      else                     /* Deleting starting in the middle. */
-       {
-         register int my_amount = ((tree->total_length
-                                    - RIGHT_TOTAL_LENGTH (tree))
-                                   - relative_position + 1);
-
-         if (amount <= my_amount)
-           {
-             tree->total_length -= amount;
-             return amount;
-           }
-         else
-           {
-             tree->total_length -= my_amount;
-             return my_amount;
-           }
-       }
+      /* How much can we delete from this interval?  */
+      int my_amount = ((tree->total_length 
+                       - RIGHT_TOTAL_LENGTH (tree))
+                      - relative_position);
+
+      if (amount > my_amount)
+       amount = my_amount;
+
+      tree->total_length -= amount;
+      if (LENGTH (tree) == 0)
+       delete_interval (tree);
+      
+      return amount;
     }
 
-  /* Never reach here */
+  /* Never reach here */
 }
 
-/* Effect the adjustments necessary to the interval tree of BUFFER
-   to correspond to the deletion of LENGTH characters from that buffer
-   text.  The deletion is effected at position START (relative to the
-   buffer). */
+/* Effect the adjustments necessary to the interval tree of BUFFER to
+   correspond to the deletion of LENGTH characters from that buffer
+   text.  The deletion is effected at position START (which is a
+   buffer position, i.e. origin 1).  */
 
 static void
 adjust_intervals_for_deletion (buffer, start, length)
@@ -848,15 +1379,23 @@ adjust_intervals_for_deletion (buffer, start, length)
      int start, length;
 {
   register int left_to_delete = length;
-  register INTERVAL tree = buffer->intervals;
-  register int deleted;
+  register INTERVAL tree = BUF_INTERVALS (buffer);
+  Lisp_Object parent;
+  int offset;
+
+  GET_INTERVAL_OBJECT (parent, tree);
+  offset = (BUFFERP (parent) ? BUF_BEG (XBUFFER (parent)) : 0);
 
   if (NULL_INTERVAL_P (tree))
     return;
 
+  if (start > offset + TOTAL_LENGTH (tree)
+      || start + length > offset + TOTAL_LENGTH (tree))
+    abort ();
+
   if (length == TOTAL_LENGTH (tree))
     {
-      buffer->intervals = NULL_INTERVAL;
+      BUF_INTERVALS (buffer) = NULL_INTERVAL;
       return;
     }
 
@@ -866,16 +1405,16 @@ adjust_intervals_for_deletion (buffer, start, length)
       return;
     }
 
-  if (start > TOTAL_LENGTH (tree))
-    start = TOTAL_LENGTH (tree);
+  if (start > offset + TOTAL_LENGTH (tree))
+    start = offset + TOTAL_LENGTH (tree);
   while (left_to_delete > 0)
     {
-      left_to_delete -= interval_deletion_adjustment (tree, start,
+      left_to_delete -= interval_deletion_adjustment (tree, start - offset,
                                                      left_to_delete);
-      tree = buffer->intervals;
+      tree = BUF_INTERVALS (buffer);
       if (left_to_delete == tree->total_length)
        {
-         buffer->intervals = NULL_INTERVAL;
+         BUF_INTERVALS (buffer) = NULL_INTERVAL;
          return;
        }
     }
@@ -884,18 +1423,18 @@ adjust_intervals_for_deletion (buffer, start, length)
 /* Make the adjustments necessary to the interval tree of BUFFER to
    represent an addition or deletion of LENGTH characters starting
    at position START.  Addition or deletion is indicated by the sign
-   of LENGTH. */
+   of LENGTH.  */
 
 INLINE void
 offset_intervals (buffer, start, length)
      struct buffer *buffer;
      int start, length;
 {
-  if (NULL_INTERVAL_P (buffer->intervals) || length == 0)
+  if (NULL_INTERVAL_P (BUF_INTERVALS (buffer)) || length == 0)
     return;
 
   if (length > 0)
-    adjust_intervals_for_insertion (buffer->intervals, start, length);
+    adjust_intervals_for_insertion (BUF_INTERVALS (buffer), start, length);
   else
     adjust_intervals_for_deletion (buffer, start, -length);
 }
@@ -907,7 +1446,7 @@ offset_intervals (buffer, start, length)
 
    IMPORTANT:
    The caller must verify that this is not the last (rightmost)
-   interval. */
+   interval.  */
 
 INTERVAL
 merge_interval_right (i)
@@ -916,12 +1455,12 @@ merge_interval_right (i)
   register int absorb = LENGTH (i);
   register INTERVAL successor;
 
-  /* Zero out this interval. */
+  /* Zero out this interval.  */
   i->total_length -= absorb;
 
-  /* Find the succeeding interval. */
+  /* Find the succeeding interval.  */
   if (! NULL_RIGHT_CHILD (i))      /* It's below us.  Add absorb
-                                     as we descend. */
+                                     as we descend.  */
     {
       successor = i->right;
       while (! NULL_LEFT_CHILD (successor))
@@ -937,21 +1476,21 @@ merge_interval_right (i)
 
   successor = i;
   while (! NULL_PARENT (successor))       /* It's above us.  Subtract as
-                                             we ascend. */
+                                             we ascend.  */
     {
       if (AM_LEFT_CHILD (successor))
        {
-         successor = successor->parent;
+         successor = INTERVAL_PARENT (successor);
          delete_interval (i);
          return successor;
        }
 
-      successor = successor->parent;
+      successor = INTERVAL_PARENT (successor);
       successor->total_length -= absorb;
     }
 
   /* This must be the rightmost or last interval and cannot
-     be merged right.  The caller should have known. */
+     be merged right.  The caller should have known.  */
   abort ();
 }
 \f
@@ -960,7 +1499,7 @@ merge_interval_right (i)
    The properties of I are lost.  Interval node I is removed from the tree.
 
    IMPORTANT:
-   The caller must verify that this is not the first (leftmost) interval. */
+   The caller must verify that this is not the first (leftmost) interval.  */
 
 INTERVAL
 merge_interval_left (i)
@@ -969,12 +1508,12 @@ merge_interval_left (i)
   register int absorb = LENGTH (i);
   register INTERVAL predecessor;
 
-  /* Zero out this interval. */
+  /* Zero out this interval.  */
   i->total_length -= absorb;
 
-  /* Find the preceding interval. */
+  /* Find the preceding interval.  */
   if (! NULL_LEFT_CHILD (i))   /* It's below us. Go down,
-                                  adding ABSORB as we go. */
+                                  adding ABSORB as we go.  */
     {
       predecessor = i->left;
       while (! NULL_RIGHT_CHILD (predecessor))
@@ -990,28 +1529,28 @@ merge_interval_left (i)
 
   predecessor = i;
   while (! NULL_PARENT (predecessor))  /* It's above us.  Go up,
-                                  subtracting ABSORB. */
+                                  subtracting ABSORB.  */
     {
       if (AM_RIGHT_CHILD (predecessor))
        {
-         predecessor = predecessor->parent;
+         predecessor = INTERVAL_PARENT (predecessor);
          delete_interval (i);
          return predecessor;
        }
 
-      predecessor = predecessor->parent;
+      predecessor = INTERVAL_PARENT (predecessor);
       predecessor->total_length -= absorb;
     }
 
   /* This must be the leftmost or first interval and cannot
-     be merged left.  The caller should have known. */
+     be merged left.  The caller should have known.  */
   abort ();
 }
 \f
 /* Make an exact copy of interval tree SOURCE which descends from
    PARENT.  This is done by recursing through SOURCE, copying
    the current interval and its properties, and then adjusting
-   the pointers of the copy. */
+   the pointers of the copy.  */
 
 static INTERVAL
 reproduce_tree (source, parent)
@@ -1021,7 +1560,25 @@ reproduce_tree (source, parent)
 
   bcopy (source, t, INTERVAL_SIZE);
   copy_properties (source, t);
-  t->parent = parent;
+  SET_INTERVAL_PARENT (t, parent);
+  if (! NULL_LEFT_CHILD (source))
+    t->left = reproduce_tree (source->left, t);
+  if (! NULL_RIGHT_CHILD (source))
+    t->right = reproduce_tree (source->right, t);
+
+  return t;
+}
+
+static INTERVAL
+reproduce_tree_obj (source, parent)
+     INTERVAL source;
+     Lisp_Object parent;
+{
+  register INTERVAL t = make_interval ();
+
+  bcopy (source, t, INTERVAL_SIZE);
+  copy_properties (source, t);
+  SET_INTERVAL_OBJECT (t, parent);
   if (! NULL_LEFT_CHILD (source))
     t->left = reproduce_tree (source->left, t);
   if (! NULL_RIGHT_CHILD (source))
@@ -1030,12 +1587,15 @@ reproduce_tree (source, parent)
   return t;
 }
 
+#if 0
+/* Nobody calls this.  Perhaps it's a vestige of an earlier design.  */
+
 /* Make a new interval of length LENGTH starting at START in the
    group of intervals INTERVALS, which is actually an interval tree.
    Returns the new interval.
 
    Generate an error if the new positions would overlap an existing
-   interval. */
+   interval.  */
 
 static INTERVAL
 make_new_interval (intervals, start, length)
@@ -1053,32 +1613,41 @@ make_new_interval (intervals, start, length)
 
   if (slot->position == start)
     {
-      /* New right node. */
-      split_interval_right (slot, length + 1);
+      /* New right node.  */
+      split_interval_right (slot, length);
       return slot;
     }
 
   if (slot->position + LENGTH (slot) == start + length)
     {
-      /* New left node. */
-      split_interval_left (slot, LENGTH (slot) - length + 1);
+      /* New left node.  */
+      split_interval_left (slot, LENGTH (slot) - length);
       return slot;
     }
 
-  /* Convert interval SLOT into three intervals. */
-  split_interval_left (slot, start - slot->position + 1);
-  split_interval_right (slot, length + 1);
+  /* Convert interval SLOT into three intervals.  */
+  split_interval_left (slot, start - slot->position);
+  split_interval_right (slot, length);
   return slot;
 }
+#endif
 \f
 /* Insert the intervals of SOURCE into BUFFER at POSITION.
+   LENGTH is the length of the text in SOURCE.
 
-   This is used in insdel.c when inserting Lisp_Strings into
-   the buffer.  The text corresponding to SOURCE is already in
-   the buffer when this is called.  The intervals of new tree are
-   those belonging to the string being inserted;  a copy is not made.
+   The `position' field of the SOURCE intervals is assumed to be
+   consistent with its parent; therefore, SOURCE must be an
+   interval tree made with copy_interval or must be the whole
+   tree of a buffer or a string.
 
-   If the inserted text had no intervals associated, this function
+   This is used in insdel.c when inserting Lisp_Strings into the
+   buffer.  The text corresponding to SOURCE is already in the buffer
+   when this is called.  The intervals of new tree are a copy of those
+   belonging to the string being inserted; intervals are never
+   shared.
+
+   If the inserted text had no intervals associated, and we don't
+   want to inherit the surrounding text's properties, this function
    simply returns -- offset_intervals should handle placing the
    text in the correct interval, depending on the sticky bits.
 
@@ -1100,65 +1669,84 @@ make_new_interval (intervals, start, length)
    intervals to the new text are "sticky", then the new text retains
    only its properties, as if neither sticky property were set.  Perhaps
    we should consider merging all three sets of properties onto the new
-   text... */
+   text...  */
 
 void
-graft_intervals_into_buffer (source, position, buffer)
+graft_intervals_into_buffer (source, position, length, buffer, inherit)
      INTERVAL source;
-     int position;
+     int position, length;
      struct buffer *buffer;
+     int inherit;
 {
   register INTERVAL under, over, this, prev;
-  register INTERVAL tree = buffer->intervals;
-  int middle;
+  register INTERVAL tree;
+
+  tree = BUF_INTERVALS (buffer);
 
-  /* If the new text has no properties, it becomes part of whatever
-     interval it was inserted into. */
+  /* If the new text has no properties, then with inheritance it
+     becomes part of whatever interval it was inserted into.
+     To prevent inheritance, we must clear out the properties
+     of the newly inserted text.  */
   if (NULL_INTERVAL_P (source))
-    return;
+    {
+      Lisp_Object buf;
+      if (!inherit && !NULL_INTERVAL_P (tree) && length > 0)
+       {
+         XSETBUFFER (buf, buffer);
+         set_text_properties_1 (make_number (position),
+                                make_number (position + length),
+                                Qnil, buf, 0);
+       }
+      if (! NULL_INTERVAL_P (BUF_INTERVALS (buffer)))
+       BUF_INTERVALS (buffer) = balance_an_interval (BUF_INTERVALS (buffer));
+      return;
+    }
 
   if (NULL_INTERVAL_P (tree))
     {
       /* The inserted text constitutes the whole buffer, so
-        simply copy over the interval structure. */
-      if (BUF_Z (buffer) == TOTAL_LENGTH (source))
+        simply copy over the interval structure.  */
+      if ((BUF_Z (buffer) - BUF_BEG (buffer)) == TOTAL_LENGTH (source))
        {
-         buffer->intervals = reproduce_tree (source, tree->parent);
-         /* Explicitly free the old tree here. */
+         Lisp_Object buf;
+         XSETBUFFER (buf, buffer);
+         BUF_INTERVALS (buffer) = reproduce_tree_obj (source, buf);
+         BUF_INTERVALS (buffer)->position = 1;
+
+         /* Explicitly free the old tree here?  */
 
          return;
        }
 
       /* Create an interval tree in which to place a copy
-        of the intervals of the inserted string. */
+        of the intervals of the inserted string.  */
       {
        Lisp_Object buf;
-       XSET (buf, Lisp_Buffer, buffer);
+       XSETBUFFER (buf, buffer);
        tree = create_root_interval (buf);
       }
     }
-  else
-    if (TOTAL_LENGTH (tree) == TOTAL_LENGTH (source))
-      /* If the buffer contains only the new string, but
-        there was already some interval tree there, then it may be
-        some zero length intervals.  Eventually, do something clever
-        about inserting properly.  For now, just waste the old intervals. */
-      {
-       buffer->intervals = reproduce_tree (source, tree->parent);
-       /* Explicitly free the old tree here. */
+  else if (TOTAL_LENGTH (tree) == TOTAL_LENGTH (source))
+    /* If the buffer contains only the new string, but
+       there was already some interval tree there, then it may be
+       some zero length intervals.  Eventually, do something clever
+       about inserting properly.  For now, just waste the old intervals.  */
+    {
+      BUF_INTERVALS (buffer) = reproduce_tree (source, INTERVAL_PARENT (tree));
+      BUF_INTERVALS (buffer)->position = 1;
+      /* Explicitly free the old tree here.  */
 
-       return;
-      }
-    else
-      /* Paranoia -- the text has already been added, so this buffer
-        should be of non-zero length. */
-      if (TOTAL_LENGTH (tree) == 0)
-       abort ();
+      return;
+    }
+  /* Paranoia -- the text has already been added, so this buffer
+     should be of non-zero length.  */
+  else if (TOTAL_LENGTH (tree) == 0)
+    abort ();
 
   this = under = find_interval (tree, position);
   if (NULL_INTERVAL_P (under)) /* Paranoia */
     abort ();
-  over = find_interval (source, 1);
+  over = find_interval (source, interval_start_pos (source));
 
   /* Here for insertion in the middle of an interval.
      Split off an equivalent interval to the right,
@@ -1167,51 +1755,58 @@ graft_intervals_into_buffer (source, position, buffer)
   if (position > under->position)
     {
       INTERVAL end_unchanged
-       = split_interval_left (this, position - under->position + 1);
+       = split_interval_left (this, position - under->position);
       copy_properties (under, end_unchanged);
       under->position = position;
-      prev = 0;
-      middle = 1;
     }
   else
     {
+      /* This call may have some effect because previous_interval may
+         update `position' fields of intervals.  Thus, don't ignore it
+         for the moment.  Someone please tell me the truth (K.Handa).  */
       prev = previous_interval (under);
-      if (prev && !END_STICKY_P (prev))
+#if 0
+      /* But, this code surely has no effect.  And, anyway,
+         END_NONSTICKY_P is unreliable now.  */
+      if (prev && !END_NONSTICKY_P (prev))
        prev = 0;
+#endif /* 0 */
     }
 
   /* Insertion is now at beginning of UNDER.  */
 
   /* The inserted text "sticks" to the interval `under',
-     which means it gets those properties. */
+     which means it gets those properties.
+     The properties of under are the result of
+     adjust_intervals_for_insertion, so stickiness has
+     already been taken care of.  */
+     
   while (! NULL_INTERVAL_P (over))
     {
-      position = LENGTH (over) + 1;
-      if (position < LENGTH (under))
-       this = split_interval_left (under, position);
+      if (LENGTH (over) < LENGTH (under))
+       {
+         this = split_interval_left (under, LENGTH (over));
+         copy_properties (under, this);
+       }
       else
        this = under;
       copy_properties (over, this);
-      /* Insertion at the end of an interval, PREV,
-        inherits from PREV if PREV is sticky at the end.  */
-      if (prev && ! FRONT_STICKY_P (under)
-         && MERGE_INSERTIONS (prev))
-       merge_properties (prev, this);
-      /* Maybe it inherits from the following interval
-        if that is sticky at the front.  */
-      else if ((FRONT_STICKY_P (under) || middle)
-              && MERGE_INSERTIONS (under))
-       merge_properties (under, this);
+      if (inherit)
+       merge_properties (over, this);
+      else
+       copy_properties (over, this);
       over = next_interval (over);
     }
 
-  buffer->intervals = balance_intervals (buffer->intervals);
+  if (! NULL_INTERVAL_P (BUF_INTERVALS (buffer)))
+    BUF_INTERVALS (buffer) = balance_an_interval (BUF_INTERVALS (buffer));
   return;
 }
 
 /* Get the value of property PROP from PLIST,
    which is the plist of an interval.
-   We check for direct properties and for categories with property PROP.  */
+   We check for direct properties, for categories with property PROP, 
+   and for PROP appearing on the default-text-properties list.  */
 
 Lisp_Object
 textget (plist, prop)
@@ -1228,57 +1823,163 @@ textget (plist, prop)
       if (EQ (prop, tem))
        return Fcar (Fcdr (tail));
       if (EQ (tem, Qcategory))
-       fallback = Fget (Fcar (Fcdr (tail)), prop);
+       {
+         tem = Fcar (Fcdr (tail));
+         if (SYMBOLP (tem))
+           fallback = Fget (tem, prop);
+       }
     }
 
-  return fallback;
+  if (! NILP (fallback))
+    return fallback;
+  if (CONSP (Vdefault_text_properties))
+    return Fplist_get (Vdefault_text_properties, prop);
+  return Qnil;
 }
+
 \f
-/* Set point in BUFFER to POSITION.  If the target position is 
-   before an invisible character which is not displayed with a special glyph,
-   move back to an ok place to display.  */
+/* Set point "temporarily", without checking any text properties.  */
+
+INLINE void
+temp_set_point (buffer, charpos)
+     struct buffer *buffer;
+     int charpos;
+{
+  temp_set_point_both (buffer, charpos,
+                      buf_charpos_to_bytepos (buffer, charpos));
+}
+
+/* Set point in BUFFER "temporarily" to CHARPOS, which corresponds to
+   byte position BYTEPOS.  */
+
+INLINE void
+temp_set_point_both (buffer, charpos, bytepos)
+     int charpos, bytepos;
+     struct buffer *buffer;
+{
+  /* In a single-byte buffer, the two positions must be equal.  */
+  if (BUF_ZV (buffer) == BUF_ZV_BYTE (buffer)
+      && charpos != bytepos)
+    abort ();
+
+  if (charpos > bytepos)
+    abort ();
+
+  if (charpos > BUF_ZV (buffer) || charpos < BUF_BEGV (buffer))
+    abort ();
+
+  BUF_PT_BYTE (buffer) = bytepos;
+  BUF_PT (buffer) = charpos;
+}
+
+/* Set point in BUFFER to CHARPOS.  If the target position is 
+   before an intangible character, move to an ok place.  */
 
 void
-set_point (position, buffer)
-     register int position;
+set_point (buffer, charpos)
      register struct buffer *buffer;
+     register int charpos;
+{
+  set_point_both (buffer, charpos, buf_charpos_to_bytepos (buffer, charpos));
+}
+
+/* If there's an invisible character at position POS + TEST_OFFS in the
+   current buffer, and the invisible property has a `stickiness' such that
+   inserting a character at position POS would inherit the property it,
+   return POS + ADJ, otherwise return POS.  If TEST_INTANG is non-zero,
+   then intangibility is required as well as invisibleness.
+
+   TEST_OFFS should be either 0 or -1, and ADJ should be either 1 or -1.
+
+   Note that `stickiness' is determined by overlay marker insertion types,
+   if the invisible property comes from an overlay.  */   
+
+static int
+adjust_for_invis_intang (pos, test_offs, adj, test_intang)
+     int pos, test_offs, adj, test_intang;
 {
-  register INTERVAL to, from, toprev, fromprev, target;
-  register int iposition = position;
+  Lisp_Object invis_propval, invis_overlay;
+  Lisp_Object test_pos;
+
+  if ((adj < 0 && pos + adj < BEGV) || (adj > 0 && pos + adj > ZV))
+    /* POS + ADJ would be beyond the buffer bounds, so do no adjustment.  */
+    return pos;
+
+  test_pos = make_number (pos + test_offs);
+
+  invis_propval
+    = get_char_property_and_overlay (test_pos, Qinvisible, Qnil,
+                                    &invis_overlay);
+
+  if ((!test_intang
+       || ! NILP (Fget_char_property (test_pos, Qintangible, Qnil)))
+      && TEXT_PROP_MEANS_INVISIBLE (invis_propval)
+      /* This next test is true if the invisible property has a stickiness
+        such that an insertion at POS would inherit it.  */
+      && (NILP (invis_overlay)
+         /* Invisible property is from a text-property.  */
+         ? (text_property_stickiness (Qinvisible, make_number (pos))
+            == (test_offs == 0 ? 1 : -1))
+         /* Invisible property is from an overlay.  */
+         : (test_offs == 0
+            ? XMARKER (OVERLAY_START (invis_overlay))->insertion_type == 0
+            : XMARKER (OVERLAY_END (invis_overlay))->insertion_type == 1)))
+    pos += adj;
+
+  return pos;
+}
+
+/* Set point in BUFFER to CHARPOS, which corresponds to byte
+   position BYTEPOS.  If the target position is 
+   before an intangible character, move to an ok place.  */
+
+void
+set_point_both (buffer, charpos, bytepos)
+     register struct buffer *buffer;
+     register int charpos, bytepos;
+{
+  register INTERVAL to, from, toprev, fromprev;
   int buffer_point;
-  register Lisp_Object obj;
-  int backwards = (position < BUF_PT (buffer)) ? 1 : 0;
-  int old_position = buffer->text.pt;
+  int old_position = BUF_PT (buffer);
+  int backwards = (charpos < old_position ? 1 : 0);
+  int have_overlays;
+  int original_position;
+
+  buffer->point_before_scroll = Qnil;
 
-  if (position == buffer->text.pt)
+  if (charpos == BUF_PT (buffer))
     return;
 
+  /* In a single-byte buffer, the two positions must be equal.  */
+  if (BUF_ZV (buffer) == BUF_ZV_BYTE (buffer)
+      && charpos != bytepos)
+    abort ();
+
   /* Check this now, before checking if the buffer has any intervals.
      That way, we can catch conditions which break this sanity check
      whether or not there are intervals in the buffer.  */
-  if (position > BUF_Z (buffer) || position < BUF_BEG (buffer))
+  if (charpos > BUF_ZV (buffer) || charpos < BUF_BEGV (buffer))
     abort ();
 
-  if (NULL_INTERVAL_P (buffer->intervals))
+  have_overlays = (! NILP (buffer->overlays_before)
+                  || ! NILP (buffer->overlays_after));
+
+  /* If we have no text properties and overlays,
+     then we can do it quickly.  */
+  if (NULL_INTERVAL_P (BUF_INTERVALS (buffer)) && ! have_overlays)
     {
-      buffer->text.pt = position;
+      temp_set_point_both (buffer, charpos, bytepos);
       return;
     }
 
-  /* Position Z is really one past the last char in the buffer.  */
-  if (position == BUF_ZV (buffer))
-    iposition = position - 1;
-
-  /* Set TO to the interval containing the char after POSITION,
-     and TOPREV to the interval containing the char before POSITION.
+  /* Set TO to the interval containing the char after CHARPOS,
+     and TOPREV to the interval containing the char before CHARPOS.
      Either one may be null.  They may be equal.  */
-  to = find_interval (buffer->intervals, iposition);
-  if (position == BUF_BEGV (buffer))
+  to = find_interval (BUF_INTERVALS (buffer), charpos);
+  if (charpos == BUF_BEGV (buffer))
     toprev = 0;
-  else if (to->position == position)
+  else if (to && to->position == charpos)
     toprev = previous_interval (to);
-  else if (iposition != position)
-    toprev = to, to = 0;
   else
     toprev = to;
 
@@ -1289,45 +1990,134 @@ set_point (position, buffer)
   /* Set FROM to the interval containing the char after PT,
      and FROMPREV to the interval containing the char before PT.
      Either one may be null.  They may be equal.  */
-  /* We could cache this and save time. */
-  from = find_interval (buffer->intervals, buffer_point);
-  if (from->position == BUF_BEGV (buffer))
+  /* We could cache this and save time.  */
+  from = find_interval (BUF_INTERVALS (buffer), buffer_point);
+  if (buffer_point == BUF_BEGV (buffer))
     fromprev = 0;
-  else if (from->position == BUF_PT (buffer))
+  else if (from && from->position == BUF_PT (buffer))
     fromprev = previous_interval (from);
   else if (buffer_point != BUF_PT (buffer))
     fromprev = from, from = 0;
   else
     fromprev = from;
 
-  /* Moving within an interval */
-  if (to == from && toprev == fromprev && INTERVAL_VISIBLE_P (to))
+  /* Moving within an interval.  */
+  if (to == from && toprev == fromprev && INTERVAL_VISIBLE_P (to)
+      && ! have_overlays)
     {
-      buffer->text.pt = position;
+      temp_set_point_both (buffer, charpos, bytepos);
       return;
     }
 
-  /* If the new position is before an invisible character,
-     move forward over all such.  */
-  while (! NULL_INTERVAL_P (to)
-        && ! INTERVAL_VISIBLE_P (to)
-        && ! DISPLAY_INVISIBLE_GLYPH (to))
+  original_position = charpos;
+
+  /* If the new position is between two intangible characters
+     with the same intangible property value,
+     move forward or backward until a change in that property.  */
+  if (NILP (Vinhibit_point_motion_hooks)
+      && ((! NULL_INTERVAL_P (to) && ! NULL_INTERVAL_P (toprev))
+         || have_overlays)
+      /* Intangibility never stops us from positioning at the beginning
+        or end of the buffer, so don't bother checking in that case.  */
+      && charpos != BEGV && charpos != ZV)
+    {
+      Lisp_Object pos;
+      Lisp_Object intangible_propval;
+
+      if (backwards)
+       {
+         /* If the preceeding character is both intangible and invisible,
+            and the invisible property is `rear-sticky', perturb it so
+            that the search starts one character earlier -- this ensures
+            that point can never move to the end of an invisible/
+            intangible/rear-sticky region.  */
+         charpos = adjust_for_invis_intang (charpos, -1, -1, 1);
+
+         XSETINT (pos, charpos);
+
+         /* If following char is intangible,
+            skip back over all chars with matching intangible property.  */
+
+         intangible_propval = Fget_char_property (pos, Qintangible, Qnil);
+
+         if (! NILP (intangible_propval))
+           {
+             while (XINT (pos) > BUF_BEGV (buffer)
+                    && EQ (Fget_char_property (make_number (XINT (pos) - 1),
+                                               Qintangible, Qnil),
+                           intangible_propval))
+               pos = Fprevious_char_property_change (pos, Qnil);
+
+             /* Set CHARPOS from POS, and if the final intangible character
+                that we skipped over is also invisible, and the invisible
+                property is `front-sticky', perturb it to be one character
+                earlier -- this ensures that point can never move to the
+                beginning of an invisible/intangible/front-sticky region.  */
+             charpos = adjust_for_invis_intang (XINT (pos), 0, -1, 0);
+           }
+       }
+      else
+       {
+         /* If the following character is both intangible and invisible,
+            and the invisible property is `front-sticky', perturb it so
+            that the search starts one character later -- this ensures
+            that point can never move to the beginning of an
+            invisible/intangible/front-sticky region.  */
+         charpos = adjust_for_invis_intang (charpos, 0, 1, 1);
+
+         XSETINT (pos, charpos);
+
+         /* If preceding char is intangible,
+            skip forward over all chars with matching intangible property.  */
+
+         intangible_propval = Fget_char_property (make_number (charpos - 1),
+                                                  Qintangible, Qnil);
+
+         if (! NILP (intangible_propval))
+           {
+             while (XINT (pos) < BUF_ZV (buffer)
+                    && EQ (Fget_char_property (pos, Qintangible, Qnil),
+                           intangible_propval))
+               pos = Fnext_char_property_change (pos, Qnil);
+
+             /* Set CHARPOS from POS, and if the final intangible character
+                that we skipped over is also invisible, and the invisible
+                property is `rear-sticky', perturb it to be one character
+                later -- this ensures that point can never move to the
+                end of an invisible/intangible/rear-sticky region.  */
+             charpos = adjust_for_invis_intang (XINT (pos), -1, 1, 0);
+           }
+       }
+
+      bytepos = buf_charpos_to_bytepos (buffer, charpos);
+    }
+
+  if (charpos != original_position)
     {
-      toprev = to;
-      to = next_interval (to);
-      if (NULL_INTERVAL_P (to))
-       position = BUF_ZV (buffer);
+      /* Set TO to the interval containing the char after CHARPOS,
+        and TOPREV to the interval containing the char before CHARPOS.
+        Either one may be null.  They may be equal.  */
+      to = find_interval (BUF_INTERVALS (buffer), charpos);
+      if (charpos == BUF_BEGV (buffer))
+       toprev = 0;
+      else if (to && to->position == charpos)
+       toprev = previous_interval (to);
       else
-       position = to->position;
+       toprev = to;
     }
 
-  buffer->text.pt = position;
+  /* Here TO is the interval after the stopping point
+     and TOPREV is the interval before the stopping point.
+     One or the other may be null.  */
+
+  temp_set_point_both (buffer, charpos, bytepos);
 
   /* We run point-left and point-entered hooks here, iff the
      two intervals are not equivalent.  These hooks take
      (old_point, new_point) as arguments.  */
-  if (! intervals_equal (from, to)
-      || ! intervals_equal (fromprev, toprev))
+  if (NILP (Vinhibit_point_motion_hooks)
+      && (! intervals_equal (from, to)
+         || ! intervals_equal (fromprev, toprev)))
     {
       Lisp_Object leave_after, leave_before, enter_after, enter_before;
 
@@ -1350,268 +2140,180 @@ set_point (position, buffer)
        enter_before = Qnil;
 
       if (! EQ (leave_before, enter_before) && !NILP (leave_before))
-       call2 (leave_before, old_position, position);
+       call2 (leave_before, make_number (old_position),
+              make_number (charpos));
       if (! EQ (leave_after, enter_after) && !NILP (leave_after))
-       call2 (leave_after, old_position, position);
+       call2 (leave_after, make_number (old_position),
+              make_number (charpos));
 
       if (! EQ (enter_before, leave_before) && !NILP (enter_before))
-       call2 (enter_before, old_position, position);
+       call2 (enter_before, make_number (old_position),
+              make_number (charpos));
       if (! EQ (enter_after, leave_after) && !NILP (enter_after))
-       call2 (enter_after, old_position, position);
+       call2 (enter_after, make_number (old_position),
+              make_number (charpos));
     }
 }
-
-/* Set point temporarily, without checking any text properties. */
-
-INLINE void
-temp_set_point (position, buffer)
-     int position;
-     struct buffer *buffer;
-{
-  buffer->text.pt = position;
-}
 \f
-/* Return the proper local map for position POSITION in BUFFER.
-   Use the map specified by the local-map property, if any.
-   Otherwise, use BUFFER's local map.  */
-
-Lisp_Object
-get_local_map (position, buffer)
-     register int position;
-     register struct buffer *buffer;
-{
-  register INTERVAL interval;
-  Lisp_Object prop, tem;
-
-  if (NULL_INTERVAL_P (buffer->intervals))
-    return current_buffer->keymap;
-
-  /* Perhaps we should just change `position' to the limit. */
-  if (position > BUF_Z (buffer) || position < BUF_BEG (buffer))
-    abort ();
-
-  /* Position Z is really one past the last char in the buffer.  */
-  if (position == BUF_ZV (buffer))
-    return current_buffer->keymap;
-
-  interval = find_interval (buffer->intervals, position);
-  prop = textget (interval->plist, Qlocal_map);
-  if (NILP (prop))
-    return current_buffer->keymap;
-
-  /* Use the local map only if it is valid.  */
-  tem = Fkeymapp (prop);
-  if (!NILP (tem))
-    return prop;
-
-  return current_buffer->keymap;
-}
-\f
-/* Call the modification hook functions in LIST, each with START and END.  */
-
-static void
-call_mod_hooks (list, start, end)
-     Lisp_Object list, start, end;
-{
-  struct gcpro gcpro1;
-  GCPRO1 (list);
-  while (!NILP (list))
-    {
-      call2 (Fcar (list), start, end);
-      list = Fcdr (list);
-    }
-  UNGCPRO;
-}
-
-/* Check for read-only intervals and signal an error if we find one.
-   Then check for any modification hooks in the range START up to
-   (but not including) TO.  Create a list of all these hooks in
-   lexicographic order, eliminating consecutive extra copies of the
-   same hook.  Then call those hooks in order, with START and END - 1
-   as arguments. */
+/* Move point to POSITION, unless POSITION is inside an intangible
+   segment that reaches all the way to point.  */
 
 void
-verify_interval_modification (buf, start, end)
-     struct buffer *buf;
-     int start, end;
+move_if_not_intangible (position)
+     int position;
 {
-  register INTERVAL intervals = buf->intervals;
-  register INTERVAL i, prev;
-  Lisp_Object hooks;
-  register Lisp_Object prev_mod_hooks;
-  Lisp_Object mod_hooks;
-  struct gcpro gcpro1;
-
-  hooks = Qnil;
-  prev_mod_hooks = Qnil;
-  mod_hooks = Qnil;
-
-  if (NULL_INTERVAL_P (intervals))
-    return;
+  Lisp_Object pos;
+  Lisp_Object intangible_propval;
 
-  if (start > end)
+  XSETINT (pos, position);
+
+  if (! NILP (Vinhibit_point_motion_hooks))
+    /* If intangible is inhibited, always move point to POSITION.  */
+    ;
+  else if (PT < position && XINT (pos) < ZV)
     {
-      int temp = start;
-      start = end;
-      end = temp;
+      /* We want to move forward, so check the text before POSITION.  */
+
+      intangible_propval = Fget_char_property (pos,
+                                              Qintangible, Qnil);
+
+      /* If following char is intangible,
+        skip back over all chars with matching intangible property.  */
+      if (! NILP (intangible_propval))
+       while (XINT (pos) > BEGV
+              && EQ (Fget_char_property (make_number (XINT (pos) - 1),
+                                         Qintangible, Qnil),
+                     intangible_propval))
+         pos = Fprevious_char_property_change (pos, Qnil);
     }
-
-  /* For an insert operation, check the two chars around the position.  */
-  if (start == end)
+  else if (XINT (pos) > BEGV)
     {
-      INTERVAL prev;
-      Lisp_Object before, after;
+      /* We want to move backward, so check the text after POSITION.  */
 
-      /* Set I to the interval containing the char after START,
-        and PREV to the interval containing the char before START.
-        Either one may be null.  They may be equal.  */
-      i = find_interval (intervals,
-                        (start == BUF_ZV (buf) ? start - 1 : start));
-
-      if (start == BUF_BEGV (buf))
-       prev = 0;
-      if (i->position == start)
-       prev = previous_interval (i);
-      else if (i->position < start)
-       prev = i;
-      if (start == BUF_ZV (buf))
-       i = 0;
+      intangible_propval = Fget_char_property (make_number (XINT (pos) - 1),
+                                              Qintangible, Qnil);
 
-      if (NULL_INTERVAL_P (prev))
-       {
-         if (! INTERVAL_WRITABLE_P (i))
-           error ("Attempt to insert within read-only text");
-       }
-      else if (NULL_INTERVAL_P (i))
-       {
-         if (! INTERVAL_WRITABLE_P (prev))
-           error ("Attempt to insert within read-only text");
-       }
-      else
-       {
-         before = textget (prev->plist, Qread_only);
-         after = textget (i->plist, Qread_only);
-         if (! NILP (before) && EQ (before, after)
-             /* This checks Vinhibit_read_only properly
-                for the common value of the read-only property.  */
-             && ! INTERVAL_WRITABLE_P (i))
-           error ("Attempt to insert within read-only text");
-       }
+      /* If following char is intangible,
+        skip forward over all chars with matching intangible property.  */
+      if (! NILP (intangible_propval))
+       while (XINT (pos) < ZV
+              && EQ (Fget_char_property (pos, Qintangible, Qnil),
+                     intangible_propval))
+         pos = Fnext_char_property_change (pos, Qnil);
 
-      /* Run both mod hooks (just once if they're the same).  */
-      if (!NULL_INTERVAL_P (prev))
-       prev_mod_hooks = textget (prev->plist, Qmodification_hooks);
-      if (!NULL_INTERVAL_P (i))
-       mod_hooks = textget (i->plist, Qmodification_hooks);
-      GCPRO1 (mod_hooks);
-      if (! NILP (prev_mod_hooks))
-       call_mod_hooks (prev_mod_hooks, make_number (start),
-                       make_number (end));
-      UNGCPRO;
-      if (! NILP (mod_hooks) && ! EQ (mod_hooks, prev_mod_hooks))
-       call_mod_hooks (mod_hooks, make_number (start), make_number (end));
     }
-  else
-    {
-      /* Loop over intervals on or next to START...END,
-        collecting their hooks.  */
-
-      i = find_interval (intervals, start);
-      do
-       {
-         if (! INTERVAL_WRITABLE_P (i))
-           error ("Attempt to modify read-only text");
 
-         mod_hooks = textget (i->plist, Qmodification_hooks);
-         if (! NILP (mod_hooks) && ! EQ (mod_hooks, prev_mod_hooks))
-           {
-             hooks = Fcons (mod_hooks, hooks);
-             prev_mod_hooks = mod_hooks;
-           }
-
-         i = next_interval (i);
-       }
-      /* Keep going thru the interval containing the char before END.  */
-      while (! NULL_INTERVAL_P (i) && i->position < end);
+  /* If the whole stretch between PT and POSITION isn't intangible, 
+     try moving to POSITION (which means we actually move farther
+     if POSITION is inside of intangible text).  */
 
-      GCPRO1 (hooks);
-      hooks = Fnreverse (hooks);
-      while (! EQ (hooks, Qnil))
-       {
-         call_mod_hooks (Fcar (hooks), make_number (start),
-                         make_number (end));
-         hooks = Fcdr (hooks);
-       }
-      UNGCPRO;
-    }
+  if (XINT (pos) != PT)
+    SET_PT (position);
 }
+\f
+/* If text at position POS has property PROP, set *VAL to the property
+   value, *START and *END to the beginning and end of a region that
+   has the same property, and return 1.  Otherwise return 0.
 
-/* Balance an interval node if the amount of text in its left and right
-   subtrees differs by more than the percentage specified by
-   `interval-balance-threshold'. */
+   OBJECT is the string or buffer to look for the property in;
+   nil means the current buffer. */
 
-static INTERVAL
-balance_an_interval (i)
-     INTERVAL i;
+int
+get_property_and_range (pos, prop, val, start, end, object)
+     int pos;
+     Lisp_Object prop, *val;
+     int *start, *end;
+     Lisp_Object object;
 {
-  register int total_children_size = (LEFT_TOTAL_LENGTH (i)
-                                     + RIGHT_TOTAL_LENGTH (i));
-  register int threshold = (XFASTINT (interval_balance_threshold)
-                           * (total_children_size / 100));
+  INTERVAL i, prev, next;
+
+  if (NILP (object))
+    i = find_interval (BUF_INTERVALS (current_buffer), pos);
+  else if (BUFFERP (object))
+    i = find_interval (BUF_INTERVALS (XBUFFER (object)), pos);
+  else if (STRINGP (object))
+    i = find_interval (XSTRING (object)->intervals, pos);
+  else
+    abort ();
 
-  /* Balance within each side.  */
-  balance_intervals (i->left);
-  balance_intervals (i->right);
+  if (NULL_INTERVAL_P (i) || (i->position + LENGTH (i) <= pos))
+    return 0;
+  *val = textget (i->plist, prop);
+  if (NILP (*val))
+    return 0;
 
-  if (LEFT_TOTAL_LENGTH (i) > RIGHT_TOTAL_LENGTH (i)
-      && (LEFT_TOTAL_LENGTH (i) - RIGHT_TOTAL_LENGTH (i)) > threshold)
-    {
-      i = rotate_right (i);
-      /* If that made it unbalanced the other way, take it back.  */
-      if (RIGHT_TOTAL_LENGTH (i) > LEFT_TOTAL_LENGTH (i)
-         && (RIGHT_TOTAL_LENGTH (i) - LEFT_TOTAL_LENGTH (i)) > threshold)
-       return rotate_left (i);
-      return i;
-    }
+  next = i;                    /* remember it in advance */
+  prev = previous_interval (i);
+  while (! NULL_INTERVAL_P (prev)
+        && EQ (*val, textget (prev->plist, prop)))
+    i = prev, prev = previous_interval (prev);
+  *start = i->position;
 
-  if (RIGHT_TOTAL_LENGTH (i) > LEFT_TOTAL_LENGTH (i)
-      && (RIGHT_TOTAL_LENGTH (i) - LEFT_TOTAL_LENGTH (i)) > threshold)
-    {
-      i = rotate_left (i);
-      if (LEFT_TOTAL_LENGTH (i) > RIGHT_TOTAL_LENGTH (i)
-         && (LEFT_TOTAL_LENGTH (i) - RIGHT_TOTAL_LENGTH (i)) > threshold)
-       return rotate_right (i);
-      return i;
-    }
+  next = next_interval (i);
+  while (! NULL_INTERVAL_P (next) 
+        && EQ (*val, textget (next->plist, prop)))
+    i = next, next = next_interval (next);
+  *end = i->position + LENGTH (i);
 
-  return i;
+  return 1;
 }
+\f
+/* Return the proper local keymap TYPE for position POSITION in
+   BUFFER; TYPE should be one of `keymap' or `local-map'.  Use the map
+   specified by the PROP property, if any.  Otherwise, if TYPE is
+   `local-map' use BUFFER's local map.  */
 
-/* Balance the interval tree TREE.  Balancing is by weight
-   (the amount of text). */
-
-INTERVAL
-balance_intervals (tree)
-     register INTERVAL tree;
+Lisp_Object
+get_local_map (position, buffer, type)
+     register int position;
+     register struct buffer *buffer;
+     Lisp_Object type;
 {
-  register INTERVAL new_tree;
+  Lisp_Object prop, lispy_position, lispy_buffer;
+  int old_begv, old_zv, old_begv_byte, old_zv_byte;
 
-  if (NULL_INTERVAL_P (tree))
-    return NULL_INTERVAL;
+  /* Perhaps we should just change `position' to the limit.  */
+  if (position > BUF_Z (buffer) || position < BUF_BEG (buffer))
+    abort ();
 
-  new_tree = tree;
-  do
-    {
-      tree = new_tree;
-      new_tree = balance_an_interval (new_tree);
-    }
-  while (new_tree != tree);
+  /* Ignore narrowing, so that a local map continues to be valid even if
+     the visible region contains no characters and hence no properties.  */
+  old_begv = BUF_BEGV (buffer);
+  old_zv = BUF_ZV (buffer);
+  old_begv_byte = BUF_BEGV_BYTE (buffer);
+  old_zv_byte = BUF_ZV_BYTE (buffer);
+  BUF_BEGV (buffer) = BUF_BEG (buffer);
+  BUF_ZV (buffer) = BUF_Z (buffer);
+  BUF_BEGV_BYTE (buffer) = BUF_BEG_BYTE (buffer);
+  BUF_ZV_BYTE (buffer) = BUF_Z_BYTE (buffer);
+
+  /* There are no properties at the end of the buffer, so in that case
+     check for a local map on the last character of the buffer instead.  */
+  if (position == BUF_Z (buffer) && BUF_Z (buffer) > BUF_BEG (buffer))
+    --position;
+  XSETFASTINT (lispy_position, position);
+  XSETBUFFER (lispy_buffer, buffer);
+  prop = Fget_char_property (lispy_position, type, lispy_buffer);
+
+  BUF_BEGV (buffer) = old_begv;
+  BUF_ZV (buffer) = old_zv;
+  BUF_BEGV_BYTE (buffer) = old_begv_byte;
+  BUF_ZV_BYTE (buffer) = old_zv_byte;
 
-  return new_tree;
-}
+  /* Use the local map only if it is valid.  */
+  prop = get_keymap (prop, 0, 0);
+  if (CONSP (prop))
+    return prop;
 
+  if (EQ (type, Qkeymap))
+    return Qnil;
+  else
+    return buffer->keymap;
+}
+\f
 /* Produce an interval tree reflecting the intervals in
-   TREE from START to START + LENGTH. */
+   TREE from START to START + LENGTH.
+   The new interval tree has no parent and has a starting-position of 0.  */
 
 INTERVAL
 copy_intervals (tree, start, length)
@@ -1628,13 +2330,13 @@ copy_intervals (tree, start, length)
   if (NULL_INTERVAL_P (i) || LENGTH (i) == 0)
     abort ();
 
-  /* If there is only one interval and it's the default, return nil. */
+  /* If there is only one interval and it's the default, return nil.  */
   if ((start - i->position + 1 + length) < LENGTH (i)
       && DEFAULT_INTERVAL_P (i))
     return NULL_INTERVAL;
 
   new = make_interval ();
-  new->position = 1;
+  new->position = 0;
   got = (LENGTH (i) - (start - i->position));
   new->total_length = length;
   copy_properties (i, new);
@@ -1644,29 +2346,135 @@ copy_intervals (tree, start, length)
   while (got < length)
     {
       i = next_interval (i);
-      t = split_interval_right (t, prevlen + 1);
+      t = split_interval_right (t, prevlen);
       copy_properties (i, t);
       prevlen = LENGTH (i);
       got += prevlen;
     }
 
-  return balance_intervals (new);
+  return balance_an_interval (new);
 }
 
-/* Give STRING the properties of BUFFER from POSITION to LENGTH. */
+/* Give STRING the properties of BUFFER from POSITION to LENGTH.  */
 
 INLINE void
 copy_intervals_to_string (string, buffer, position, length)
-     Lisp_Object string, buffer;
+     Lisp_Object string;
+     struct buffer *buffer;
      int position, length;
 {
-  INTERVAL interval_copy = copy_intervals (XBUFFER (buffer)->intervals,
+  INTERVAL interval_copy = copy_intervals (BUF_INTERVALS (buffer),
                                           position, length);
   if (NULL_INTERVAL_P (interval_copy))
     return;
 
-  interval_copy->parent = (INTERVAL) string;
+  SET_INTERVAL_OBJECT (interval_copy, string);
   XSTRING (string)->intervals = interval_copy;
 }
+\f
+/* Return 1 if strings S1 and S2 have identical properties; 0 otherwise.
+   Assume they have identical characters.  */
+
+int
+compare_string_intervals (s1, s2)
+     Lisp_Object s1, s2;
+{
+  INTERVAL i1, i2;
+  int pos = 0;
+  int end = XSTRING (s1)->size;
+
+  i1 = find_interval (XSTRING (s1)->intervals, 0);
+  i2 = find_interval (XSTRING (s2)->intervals, 0);
+
+  while (pos < end)
+    {
+      /* Determine how far we can go before we reach the end of I1 or I2.  */
+      int len1 = (i1 != 0 ? INTERVAL_LAST_POS (i1) : end) - pos;
+      int len2 = (i2 != 0 ? INTERVAL_LAST_POS (i2) : end) - pos;
+      int distance = min (len1, len2);
+
+      /* If we ever find a mismatch between the strings,
+        they differ.  */
+      if (! intervals_equal (i1, i2))
+       return 0;
+
+      /* Advance POS till the end of the shorter interval,
+        and advance one or both interval pointers for the new position.  */
+      pos += distance;
+      if (len1 == distance)
+       i1 = next_interval (i1);
+      if (len2 == distance)
+       i2 = next_interval (i2);
+    }
+  return 1;
+}
+\f
+/* Recursively adjust interval I in the current buffer
+   for setting enable_multibyte_characters to MULTI_FLAG.
+   The range of interval I is START ... END in characters,
+   START_BYTE ... END_BYTE in bytes.  */
+
+static void
+set_intervals_multibyte_1 (i, multi_flag, start, start_byte, end, end_byte)
+     INTERVAL i;
+     int multi_flag;
+     int start, start_byte, end, end_byte;
+{
+  /* Fix the length of this interval.  */
+  if (multi_flag)
+    i->total_length = end - start;
+  else
+    i->total_length = end_byte - start_byte;
+
+  /* Recursively fix the length of the subintervals.  */
+  if (i->left)
+    {
+      int left_end, left_end_byte;
+
+      if (multi_flag)
+       {
+         left_end_byte = start_byte + LEFT_TOTAL_LENGTH (i);
+         left_end = BYTE_TO_CHAR (left_end_byte);
+       }
+      else
+       {
+         left_end = start + LEFT_TOTAL_LENGTH (i);
+         left_end_byte = CHAR_TO_BYTE (left_end);
+       }
 
-#endif /* USE_TEXT_PROPERTIES */
+      set_intervals_multibyte_1 (i->left, multi_flag, start, start_byte,
+                                left_end, left_end_byte);
+    }
+  if (i->right)
+    {
+      int right_start_byte, right_start;
+
+      if (multi_flag)
+       {
+         right_start_byte = end_byte - RIGHT_TOTAL_LENGTH (i);
+         right_start = BYTE_TO_CHAR (right_start_byte);
+       }
+      else
+       {
+         right_start = end - RIGHT_TOTAL_LENGTH (i);
+         right_start_byte = CHAR_TO_BYTE (right_start);
+       }
+
+      set_intervals_multibyte_1 (i->right, multi_flag,
+                                right_start, right_start_byte,
+                                end, end_byte);
+    }
+}
+
+/* Update the intervals of the current buffer
+   to fit the contents as multibyte (if MULTI_FLAG is 1)
+   or to fit them as non-multibyte (if MULTI_FLAG is 0).  */
+
+void
+set_intervals_multibyte (multi_flag)
+     int multi_flag;
+{
+  if (BUF_INTERVALS (current_buffer))
+    set_intervals_multibyte_1 (BUF_INTERVALS (current_buffer), multi_flag,
+                              BEG, BEG_BYTE, Z, Z_BYTE);
+}