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

Prevent low values attempting to use representation -1 #1331

Merged
merged 1 commit into from
Apr 14, 2016

Conversation

LloydW93
Copy link
Member

I don't think this is urgent enough to require being included in any future RC3 (i.e. this PR can wait until 2.2).

@wilaw
Copy link
Member

wilaw commented Apr 14, 2016

It might be useful to log the fact that you are coercing the repIdx to 0, otherwise someone who is calling the external API with a negative value would have no idea that the substitution was taking place.

@dsparacio
Copy link
Contributor

@LloydW93 I am going to have to create RC 3 for minor fix anyway. How confident are you? I could add to 2.1 or to be safe we can add right after release. Either way, I will let you make the call.

@dsparacio dsparacio merged commit 6ad7c03 into Dash-Industry-Forum:development Apr 14, 2016
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.

3 participants