Skip to content

Commit

Permalink
Merge pull request #79 from digitalocean/jphines/storage/doctl-pagina…
Browse files Browse the repository at this point in the history
…tion

doctl: pagination is 1 indexed, not 0 indexed
  • Loading branch information
jphines committed May 23, 2016
2 parents 3be90e4 + 735a754 commit 4ab6c43
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions do/pagination.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ func PaginateResp(gen Generator) ([]interface{}, error) {
fetchChan := make(chan int, maxFetchPages)

var wg sync.WaitGroup
for i := 0; i < 4; i++ {
for i := 0; i < maxFetchPages-1; i++ {
wg.Add(1)
go func() {
for page := range fetchChan {
Expand Down Expand Up @@ -80,8 +80,9 @@ func PaginateResp(gen Generator) ([]interface{}, error) {
}

// start with second page
for i := 1; i < lp; i++ {
fetchChan <- i
opt.Page++
for ; opt.Page <= lp; opt.Page++ {
fetchChan <- opt.Page
}
close(fetchChan)

Expand Down

0 comments on commit 4ab6c43

Please sign in to comment.