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.

Sign up for GitHub

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

Jump to bottom

ECC-1948: Conversion of geo-iterators to C++ #256

Merged
merged 24 commits into from
Oct 21, 2024
Merged

ECC-1948: Conversion of geo-iterators to C++ #256

merged 24 commits into from
Oct 21, 2024

Conversation

joobog
Copy link
Collaborator

@joobog joobog commented Oct 21, 2024 •
edited by shahramn
Loading

@joobog
Modernisation: C++ iterators
1afda19
@joobog
Modernisation: Introduce namespaces and change class names
2122cd6
@joobog
Modernisation: Iterators (all ctests pass)
3940d7d
@joobog
Merge branch 'develop' into feature/modernisation_iterators
fff9e47
@joobog
Modernisation: Fix error code propagation in iterators destroy()
3038878
@joobog
Modernisation: Fix memory leak and minor changes
423521b
@joobog
Modernisation: Protect iterator's init() with a mutex
b490cfc
@joobog
Modernisation: has_next() returns bool instead of long
2bd5482
@joobog
Modernisation: Fix grib_iterator_delete()
a8f43e9
@joobog
Merge branch 'develop' into feature/modernisation_iterators
c41dd80
@joobog
Modernisation: clang-format
c761089
@joobog
Modernisation: Make init_sphere() and init_oblate() member functions
408066a
@joobog
Modernisation: Make members private
e4df2db
@joobog
Modernisation: Make next(), previous() and has_next() const members
f63f70f
@joobog
Merge branch 'develop' into feature/modernisation_iterators
fe7ff3a
@joobog
Merge branch 'develop' into feature/modernisation_iterators
80cc5a4
@joobog
Modernisation: Change namespace and move lats_ and lons_ to base class
f2c1a83
@shahramn shahramn self-assigned this Oct 21, 2024
@shahramn shahramn added the approved-for-ci Approved to run CI on ECMWF machines label Oct 21, 2024
@shahramn shahramn changed the title Modernisation: Conversion of geo-iterators to C++ ECC-1948 Modernisation: Conversion of geo-iterators to C++ Oct 21, 2024
@shahramn shahramn changed the title ECC-1948 Modernisation: Conversion of geo-iterators to C++ ECC-1948: Conversion of geo-iterators to C++ Oct 21, 2024
@shahramn shahramn merged commit bc034fc into develop Oct 21, 2024
175 of 176 checks passed
@shahramn shahramn deleted the feature/modernisation_iterators branch October 21, 2024 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@shahramn shahramn

Labels
approved-for-ci Approved to run CI on ECMWF machines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joobog @shahramn