Skip to content

Aiprdbms#22

Open
arm2arm wants to merge 2 commits intoakorotkov:masterfrom
arm2arm:aiprdbms
Open

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

Conversation

@arm2arm
Copy link

@arm2arm arm2arm commented Mar 30, 2022

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
Copy link

esabol commented Mar 31, 2022

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

@arm2arm
Copy link
Author

arm2arm commented Mar 31, 2022

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

@esabol
Copy link

esabol commented Mar 31, 2022

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