-
-
Notifications
You must be signed in to change notification settings - Fork 366
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
C types reorganization and cleanup #2103
C types reorganization and cleanup #2103
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Btw, is creating types like IID_t_rt
a good idea? Although this generalizes the types, but simultaneously, it decreases the readability to someone who is not aware of what I and D denote.
Anyway, if we want to generalize, then that's a good way to do so, so it's okay.
Yeah, needs documentation. |
This is a follow up from #1893
IID_t_rt
II_t_rt
I_rt`pgr_
pgr_
historically comes from v1.x@pgRouting/admins