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

checkbounds for NamedRange #12

Closed
todo bot opened this issue Mar 28, 2019 · 1 comment
Closed

checkbounds for NamedRange #12

todo bot opened this issue Mar 28, 2019 · 1 comment
Assignees
@Tokazama
Labels
todo 🗒️

Comments

@todo
Copy link

todo bot commented Mar 28, 2019

StaticRanges.jl/src/IndirectRange.jl

Lines 27 to 32 in 79a6439

@boundscheck checkbounds(x, i) # TODO: checkbounds for NamedRange
@inbounds x.index[i]
end
@inline @generated function __getindex(x::NamedRange{T,names}, ::Val{s}) where {T,names,s}
idx = findfirst(y -> y==s, names)

This issue was generated by todo based on a TODO comment in 79a6439. It's been assigned to @Tokazama because they committed the code.
@todo todo bot assigned Tokazama Mar 28, 2019
@todo todo bot added the todo 🗒️ label Mar 28, 2019
@Tokazama
Copy link
Owner

This work has moved to https://github.com/Tokazama/AbstractIndices.jl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tokazama Tokazama

Labels
todo 🗒️
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Tokazama