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

C API: add proj_get_domain_count(), proj_get_area_of_use_ex() and proj_get_scope_ex() #3588

Merged
merged 1 commit into from
Jan 23, 2023

Conversation

rouault
Copy link
Member

@rouault rouault commented Jan 23, 2023

Up to now, the C API only handled one single domain/usage par object, but there are objets in the database that have several ones.

refs #3246

…j_get_scope_ex()

Up to now, the C API only handled one single domain/usage par object,
but there are objets in the database that have several ones.

refs OSGeo#3246
@kbevers
Copy link
Member

kbevers commented Jan 23, 2023

Looks alright to me. I'm not sure how the _ex should be understood. Extended? Also, remember to update docs/source/development/reference/functions.rst before merging.

@rouault
Copy link
Member Author

rouault commented Jan 23, 2023

yes, _ex is for extended. We have a proj_create_vertical_crs_ex() in proj_experimental.h

There's no need to explictly extend the documentation, as those functions below to the iso19111_functions Doxygen group that is ingested through Breathe, with the ".. doxygengroup:: iso19111_functions" directive at end of docs/source/development/reference/functions.rst

@rouault
Copy link
Member Author

rouault commented Jan 23, 2023

@kbevers
Copy link
Member

kbevers commented Jan 23, 2023

There's no need to explictly extend the documentation, a

Right, I forgot about that. Everything's good then.

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