Skip to content

Aiprdbms - #22

Open
arm2arm wants to merge 2 commits into
akorotkov:masterfrom
arm2arm:aiprdbms
Open

Aiprdbms#22
arm2arm wants to merge 2 commits into
akorotkov:masterfrom
arm2arm:aiprdbms

Conversation

@arm2arm

@arm2arm arm2arm commented Mar 30, 2022

Copy link
Copy Markdown

looks good and working on pg14 + pg_gis

arm2arm added 2 commits March 30, 2022 13:04
using dynamic arrays to avoid  **-Werror=vla** error
moving to dynamic allocated array to avoid  -Werror=vla error
@esabol

esabol commented Mar 31, 2022

Copy link
Copy Markdown

Shouldn't you be using palloc and pfree instead of malloc and free?

@arm2arm

arm2arm commented Mar 31, 2022

Copy link
Copy Markdown
Author

what is the difference? if palloc is a postgres way to go then palloc is also good 😊

@esabol

esabol commented Mar 31, 2022

Copy link
Copy Markdown

I don't know, but I think palloc is what you're supposed to use in PostgreSQL extensions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants