Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature remove variant array view #144

Merged
merged 4 commits into from
Mar 26, 2018

Conversation

acki-m
Copy link
Contributor

@acki-m acki-m commented Mar 25, 2018

!API CHANGE!
removed the class 'variant_array_view'
removed the method property::is_array() use following instead:

property::get_type()::is_sequential_container()
property::get_type()::get_raw_type()::is_sequential_container()
property::get_type()::get_wrapped_type()::is_sequential_container()

acki-m added 2 commits March 25, 2018 12:53
!REMARK API CHANGE!
"variant_array_view" class will be removed.
For iterating over array use instead the class "variant_sequential_view"
use instead following methods:
 property::get_type()::is_sequential_container()
 property::get_type()::get_raw_type()::is_sequential_container()
 property::get_type()::get_wrapped_type()::is_sequential_container()

 An sequential_container is always also an raw C-style array
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 93.149% when pulling d76bcf1 on acki-m:feature-remove-variant-array-view into b06257b on rttrorg:master.

@coveralls
Copy link

coveralls commented Mar 25, 2018

Coverage Status

Coverage increased (+0.7%) to 93.149% when pulling 7388fcf on acki-m:feature-remove-variant-array-view into b06257b on rttrorg:master.

@acki-m acki-m merged commit ca67552 into rttrorg:master Mar 26, 2018
@acki-m acki-m deleted the feature-remove-variant-array-view branch March 26, 2018 17:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants