* xfaces.c (Finternal_set_lisp_face_attribute): Added FIXME comment.

diff --git a/src/ChangeLog b/src/ChangeLog
index 63fcf1305429284bf426c307612c582171c841aa..31320ed533fba31568be8695548396dadb7ec2c5 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -2,6 +2,7 @@
 
        * xfaces.c (Finternal_set_lisp_face_attribute): Don't try to
        make a font_object from a tty frame (Bug#18573).
+       (Finternal_set_lisp_face_attribute): Added FIXME comment.
 
 2014-09-30  Eli Zaretskii  <eliz@gnu.org>
 

diff --git a/src/xfaces.c b/src/xfaces.c
index 5d158f08099b82892ca4ca5acd1e6aa68b4be26a..9579bcce5727ad6f82e49f409a860300716c639a 100644 (file)
--- a/src/xfaces.c
+++ b/src/xfaces.c
@@ -3119,6 +3119,13 @@ FRAME 0 means change the face on all frames, and change the default
                f = XFRAME (selected_frame);
              else
                f = XFRAME (frame);
+
+              /* FIXME:
+                 If frame is t, and selected frame is a tty frame, the font
+                 can't be realized.  An improvement wuld be to loop over frames
+                 for a non-tty frame and use that.  See discussion in
+                 bug#18573.
+              */
               if (f->terminal->type != output_termcap)
                 {
                   if (! FONT_OBJECT_P (value))