Subject: lookup_basic_face: Should we add VERTICAL_BORDER_FACE_ID?
From: Keith David Bershatsky <[email-masked]>
To: Emacs Bug Reports <[email-masked]>
Date: Sun, 03 Dec 2017 17:59:34 -0800
List-Id: <debbugs-submit.debbugs.gnu.org>

I had assumed that VERTICAL_BORDER_FACE_ID was a basic face (see dispextern.h at line 1750).  However, it is not identified as such within lookup_basic_face.  Is this something that should be added?  Without it, Emacs will throw an error if someone tries to look it up and its not there.

case VERTICAL_BORDER_FACE_ID: name = Qvertical_border; break;

Thanks,

Keith