emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master dad452552b: Pacify -Wnull-dereference


From: Po Lu
Subject: master dad452552b: Pacify -Wnull-dereference
Date: Sun, 6 Nov 2022 06:43:08 -0500 (EST)

branch: master
commit dad452552bebf832e737ebb946b5eea6cd57f63f
Author: Po Lu <luangruo@yahoo.com>
Commit: Po Lu <luangruo@yahoo.com>

    Pacify -Wnull-dereference
    
    * src/xdisp.c (extend_face_to_end_of_line): Use FACE_FROM_ID,
    not FACE_FROM_ID_OR_NULL, when the next line proceeds to
    immediately dereference the face.
---
 src/xdisp.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/xdisp.c b/src/xdisp.c
index 054adb9d9d..ea193601b5 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -23154,14 +23154,14 @@ extend_face_to_end_of_line (struct it *it)
      point temporarily moved to window-point.  */
   specbind (Qinhibit_quit, Qt);
   /* The default face, possibly remapped. */
-  struct face *default_face =
-    FACE_FROM_ID_OR_NULL (f, lookup_basic_face (it->w, f, DEFAULT_FACE_ID));
-  const int extend_face_id =
-    (it->face_id == default_face->id || it->s != NULL)
-    ? it->face_id
-    : (it->glyph_row->ends_at_zv_p
-       ? default_face->id
-       : face_at_pos (it, LFACE_EXTEND_INDEX));
+  struct face *default_face
+    = FACE_FROM_ID (f, lookup_basic_face (it->w, f, DEFAULT_FACE_ID));
+  const int extend_face_id
+    = ((it->face_id == default_face->id || it->s != NULL)
+       ? it->face_id
+       : (it->glyph_row->ends_at_zv_p
+         ? default_face->id
+         : face_at_pos (it, LFACE_EXTEND_INDEX)));
   unbind_to (count, Qnil);
 
   /* Face extension extends the background and box of IT->extend_face_id



reply via email to

[Prev in Thread] Current Thread [Next in Thread]