Revision: miles@gnu.org--gnu-2004/emacs--cvs-trunk--0--patch-385

src/xfaces.c (push_named_merge_point): Return 0 if a cycle is detected

diff --git a/src/ChangeLog b/src/ChangeLog
index d4fc708db9d4f57a85226d9caed3d7f35ca1e92b..d6750648fd537a2d679eee493497ed9d31aeaeb3 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2004-06-08  Miles Bader  <miles@gnu.org>
+
+       * xfaces.c (push_named_merge_point): Return 0 when a cycle is detected.
+
 2004-06-07  Juanma Barranquero  <lektu@terra.es>
 
        * editfns.c (Fuser_login_name, Ffloat_time, Fencode_time)

diff --git a/src/xfaces.c b/src/xfaces.c
index afe352267d3638cdf7bc84f9d76fe3171cc1f435..b6b4140e729b180d69d82e8f2514232dc4bfe742 100644 (file)
--- a/src/xfaces.c
+++ b/src/xfaces.c
@@ -3169,8 +3169,8 @@ struct named_merge_point
 
 /* If a face merging cycle is detected for FACE_NAME, return 0,
    otherwise add NEW_NAMED_MERGE_POINT, which is initialized using
-   FACE_NAME, as the head of the linked list
-   pointed to by NAMED_MERGE_POINTS, and return 1.  */
+   FACE_NAME, as the head of the linked list pointed to by
+   NAMED_MERGE_POINTS, and return 1.  */
 
 static INLINE int
 push_named_merge_point (struct named_merge_point *new_named_merge_point,
@@ -3181,7 +3181,7 @@ push_named_merge_point (struct named_merge_point *new_named_merge_point,
 
   for (prev = *named_merge_points; prev; prev = prev->prev)
     if (EQ (face_name, prev->face_name))
-       break;
+      return 0;
 
   new_named_merge_point->face_name = face_name;
   new_named_merge_point->prev = *named_merge_points;