emacs-devel
[Top][All Lists]
Advanced

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

Re: MPS: face-cache


From: Eli Zaretskii
Subject: Re: MPS: face-cache
Date: Mon, 29 Apr 2024 14:41:44 +0300

> From: Gerd Möllmann <gerd.moellmann@gmail.com>
> Cc: Helmut Eller <eller.helmut@gmail.com>,  emacs-devel@gnu.org
> Date: Mon, 29 Apr 2024 13:18:47 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> From: Helmut Eller <eller.helmut@gmail.com>
> >> Cc: gerd.moellmann@gmail.com,  emacs-devel@gnu.org
> >> Date: Mon, 29 Apr 2024 10:54:57 +0200
> >> 
> >> > Are you both using clang or something?  The change didn't compiled for
> >> > me with GCC 9.2, because the case in a switch needed braces:
> >> 
> >> I use GCC (Debian 12.2.0-14) 12.2.0
> >> 
> >> Should I add some compiler flags?  This is the output of make V=1:
> >
> > I don't know.  Maybe GCC 12 lifts that restriction or something.  I
> > clearly remember that if a case includes declarations, it must be in
> > braces.
> >
> > This is an error, not a warning, so warning switches shouldn't matter.
> 
> Clang said it's C23.

If only C23 allows this, then we shouldn't use that because we don't
compile at that STDC level.



reply via email to

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