Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Suppress warning in gcc-12 #443

Merged
merged 1 commit into from
Oct 9, 2023
Merged

chore: Suppress warning in gcc-12 #443

merged 1 commit into from
Oct 9, 2023

Conversation

jeroen
Copy link
Member

@jeroen jeroen commented Oct 9, 2023

There is bug in gcc-12 that leads to the warning:

warning: pointer used after 'void operator delete(void*, std::size_t)' [-Wuse-after-free]

The bug is fixed in gcc-13. So we suppress it.

@krlmlr krlmlr changed the title Suppress warning in gcc-12 chore: Suppress warning in gcc-12 Oct 9, 2023
@krlmlr krlmlr merged commit ee7e329 into main Oct 9, 2023
16 checks passed
@krlmlr krlmlr deleted the fix-warning branch October 9, 2023 19:03
@krlmlr
Copy link
Member

krlmlr commented Oct 9, 2023

Thanks!

krlmlr added a commit that referenced this pull request Mar 31, 2024
RPostgres 1.4.6

- Breaking change: `dbListObjects()` only allows `Id()` objects as `prefix` argument (@dpprdan, #390).

- Use `NULL` in favor of `NULL::text` when quoting strings and literals, to support JSON and other text-ish types. Fixes a regression introduced in #370 (#393, #425).

- `dbQuoteLiteral()` correctly quotes 64-bit integers from the bit64 package (of class `"integer64"`) (@karawoo, #435, #436).

- Breaking change: `dbListObjects()` only allows `Id()` objects as `prefix` argument (@dpprdan, #390).

- Windows: update to libpq-15.3 (#442).

- Upgrade boost to 1.81.0-1 to fix sprintf warnings (#417).

- Suppress warning in gcc-12 (#443).

- Tweak driver docs (@dpprdan, #433).

- Relicense as MIT.

- Close result set.

- Replace Rcpp by cpp11 (@Antonov548, #419).
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants