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

Fix bug in gserialized_spgist_picksplit_nd #331

Closed
wants to merge 1 commit into from

Conversation

@Algunenano
Copy link
Member

commented Nov 12, 2018

https://github.com/postgis/postgis/compare/svn-trunk...Algunenano:undef_spgist?expand=1#diff-1a55a503decf929ce4bc78784eaf5ed7R334 fixes the bug. The rest is using C99 in place declaration and clearer naming to help avoiding these kind of bugs.

Having a unit test for this would be appropriate

Solves https://trac.osgeo.org/postgis/ticket/4233

@Algunenano Algunenano force-pushed the Algunenano:undef_spgist branch from a775d75 to 8c00474 Nov 13, 2018

@Komzpa Komzpa self-requested a review Nov 18, 2018

@Komzpa
Komzpa approved these changes Nov 18, 2018

@Algunenano Algunenano force-pushed the Algunenano:undef_spgist branch from 8c00474 to 1f00b59 Nov 19, 2018

@strk strk closed this in 7770364 Nov 19, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.