Re: [lmi] [lmi-commits] master d810372 3/9: Trade elegance for simplicity

[Greg Chicares]

On 10/24/21 12:10 AM, Vadim Zeitlin wrote:
> On Sat, 23 Oct 2021 18:32:44 -0400 (EDT) Greg Chicares 
> <gchicares@sbcglobal.net> wrote:
> 
> GC> branch: master
> GC> commit d8103724347db1cf4945adeb5fb9dec4e0a8bf23
> GC> Author: Gregory W. Chicares <gchicares@sbcglobal.net>
> GC> Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
> GC> 
> GC>     Trade elegance for simplicity
> 
>  Sorry, I probably should have explained it better, but it can't be that
> simple because this version doesn't work, as can be seen by the errors at
> 
>       https://github.com/let-me-illustrate/lmi/runs/3986134857

[Blocked by MS unless you accept their EULA.]

> GC>     $(gcc_cxx_warnings) seems adequate for gcc version-specific warnings:
> 
>  Unfortunately it isn't. And yes, this means that the existing assignment
> to it doesn't work neither, i.e. -Wredundant-tags and -Wvolatile are never
> used.

Thanks for figuring that out. I think it's all fixed now.