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

digits should accept values of "pad" that are unsigned #8996

Merged
merged 1 commit into from
Nov 12, 2014

Conversation

goretkin
Copy link
Contributor

otherwise, even tough pad should be non-negative
julia> digits(5,2,uint64(8))
ERROR: digits has no method matching digits(::Int64, ::Int64, ::Uint64)

otherwise, even tough `pad` should be non-negative 
julia> digits(5,2,uint64(8))
ERROR: `digits` has no method matching digits(::Int64, ::Int64, ::Uint64)
staticfloat added a commit that referenced this pull request Nov 12, 2014
digits should accept values of "pad" that are unsigned
@staticfloat staticfloat merged commit caf02ef into JuliaLang:master Nov 12, 2014
@staticfloat
Copy link
Sponsor Member

Thanks!

@vtjnash
Copy link
Sponsor Member

vtjnash commented Nov 14, 2014

there's a handful more of these (hex, oct, dec, ndigit, base, bin, etc.) which could probably also use this same update

@tkelman
Copy link
Contributor

tkelman commented Nov 18, 2014

safe to @JuliaBackports this?

@JeffBezanson
Copy link
Sponsor Member

Yes seems pretty safe.

@tkelman
Copy link
Contributor

tkelman commented Nov 19, 2014

thanks, backported in ceb9632

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.

None yet

6 participants