Skip to content

Conversation

@steve-chavez
Copy link
Member

@steve-chavez steve-chavez commented Sep 10, 2025

Closes #10. This option is included by default on GCC 15 with -Wextra.

Not really a bug in the codebase as we don't use the null terminator, we use the exact size of the BOM string for appending it. See:

if (state->options->bom) appendBinaryStringInfo(&state->accum_buf, BOM, sizeof(BOM));

This now changes compilation to always use GCC 15.

@coveralls
Copy link

coveralls commented Sep 10, 2025

Pull Request Test Coverage Report for Build 17623090584

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.099%

Totals Coverage Status
Change from base Build 17591197101: 0.0%
Covered Lines: 110
Relevant Lines: 111

💛 - Coveralls

This options is included by default on GCC 15 with -Wextra.

Not really a bug in the codebase as we don't use the null terminator,
we use the exact size of the BOM string for appending it.

.
@steve-chavez steve-chavez marked this pull request as ready for review September 10, 2025 18:21
@steve-chavez steve-chavez merged commit d21a1da into PostgREST:master Sep 10, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants