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

Change for i=1:length(A) to for i in eachindex(A) #10858

Merged
merged 2 commits into from
Apr 20, 2015
Merged

Change for i=1:length(A) to for i in eachindex(A) #10858

merged 2 commits into from
Apr 20, 2015

Commits on Apr 19, 2015

  1. Change i=1:length(A) to i in eachindex(A) 

    This fixes performance problems for many SubArray operations
    @timholy
    timholy committed Apr 19, 2015
    20 Configuration menu
    Copy the full SHA
    4297135 View commit details
    Browse the repository at this point in the history

  2. Fix iteration for empty CartesianRanges 

    Since `done` is called on each iteration but `start` is called
only once, it makes more sense to put this logic in `start`.
    @timholy
    timholy committed Apr 19, 2015
    Configuration menu
    Copy the full SHA
    4216ae5 View commit details
    Browse the repository at this point in the history