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

ranges: Add size() and empty() functions as well as additional constructors #122

Merged
merged 1 commit into from
Mar 25, 2020

Conversation

stotko
Copy link
Owner

@stotko stotko commented Mar 25, 2020

The API of the ranges module, which aims to model the C++20 ranges library in a significantly simplified yet still useful manner, is still rather limited. Add additional constructors to host_range, device_range and transform_range as well as size() and empty() functions and some typedefs. Furthermore, replace the non-const begin() and end() functions with equivalent const versions and implicitly deprecate the non-const ones. This is a small step towards a more useful ranges module.

@stotko stotko added this to the 1.3.0 milestone Mar 25, 2020
@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #122 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   97.87%   97.92%   +0.04%     
==========================================
  Files          27       28       +1     
  Lines        1648     1685      +37     
==========================================
+ Hits         1613     1650      +37     
  Misses         35       35              
Impacted Files Coverage Δ
src/stdgpu/impl/ranges_detail.h 100.00% <100.00%> (ø)
src/stdgpu/ranges.h 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 01a7703...61b1a9c. Read the comment docs.

@stotko stotko merged commit 5b6b90a into master Mar 25, 2020
@stotko stotko deleted the ranges_interface branch March 25, 2020 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant