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

IntSet initialization mistake #156

Closed
abyrd opened this issue Dec 22, 2014 · 1 comment
Closed

IntSet initialization mistake #156

abyrd opened this issue Dec 22, 2014 · 1 comment
Labels

Comments

@abyrd
Copy link
Contributor

abyrd commented Dec 22, 2014

IntSet_new calls IntSet_clear, which assumes that all the list head pointers are already initialized to NULL. Freeing the collision-lists and initializing should be in separate functions. This is fixed in some code I have locally, where IntSet becomes a map from int64_t to uint32_t, i.e. a map from OSM IDs to positive integer indexes.

Really we should pull all this code out into a libbliksem, along with my PBF loader code. See #155.

@abyrd abyrd added the bug label Dec 22, 2014
@koch-t
Copy link
Contributor

koch-t commented Mar 16, 2015

It was moved to https://github.com/conveyal/vanilla-extract

@koch-t koch-t closed this as completed Mar 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants