Skip to content

Conversation

@ObsessiveOrange
Copy link
Member

No description provided.

Signed-off-by: Benedict Wong <bennydictwong@gmail.com>
Signed-off-by: Benedict Wong <bennydictwong@gmail.com>
Signed-off-by: Benedict Wong <bennydictwong@gmail.com>
Signed-off-by: Benedict Wong <bennydictwong@gmail.com>
Copy link
Member

@julie-is-late julie-is-late left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the logic in line 206 should also go inside of the bigger statement at 230

if startIndex < 0 {
return nil, -1, nil, ErrVersionOutOfDate
}
// If it's negative at this point, it means startIndex was negative.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm guessing this isn't actually what you wanted this comment to say? that's obvious, lol, but why is it negative?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep. I'll edit that

@julie-is-late julie-is-late merged commit 97c2cbc into test Dec 21, 2016
@julie-is-late julie-is-late deleted the sync-hotfix branch December 21, 2016 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants