Re: [PATCH 10/10] reftable: make reftable_record a tagged union
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Dec 8, 2021, 16:35 UTC
- Message-ID
- <xmqq1r2n3vhj.fsf@gitster.g>
- In-Reply-To
- <CAFQ2z_NuOy+-pfSoNAYjJhS9jZCYOfoFue10=k=iyPVsPYrB3g@mail.gmail.com>
Han-Wen Nienhuys <hanwen@google.com> writes:
Show 13 quoted lines
> On Wed, Dec 8, 2021 at 5:13 AM Junio C Hamano <gitster@pobox.com> wrote: >> >> error: ISO C99 doesn't support unnamed structs/unions [-Werror=pedantic] >> > >> > Hmm. It's interesting that the regular DEVELOPER=1 doesn't catch this. >> > It's because we don't specify -std there, and newer gcc defaults to >> > gnu17 (unnamed unions appeared in c11, I think). I wonder if it would be >> > helpful to teach config.mak.dev to pass -std=c99. >> >> FWIW, I use -std=gnu99 as our Makefile suggests. > > I understand that the default build should be lenient rather than > strict for portability reasons. However, it would be good if the CI > was strict with this.
Yeah, I agree with the above, and was a bit surprised that I found the issue via my local build after it came from GGG.