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

auto_unbox error due to Uint8 deprecation #13204

Closed
timholy opened this issue Sep 18, 2015 · 6 comments
Closed

auto_unbox error due to Uint8 deprecation #13204

timholy opened this issue Sep 18, 2015 · 6 comments
Milestone

Comments

@timholy
Copy link
Sponsor Member

timholy commented Sep 18, 2015

Mentioned in 1b6efc7 and #13107 (comment), but since there's no issue open:

julia> function bar(lib)
           soname = ccall(:jl_lookup_soname, Ptr{Uint8}, (Ptr{Uint8}, Csize_t), lib, sizeof(lib))
           if soname != C_NULL
               println("not null")
           else
               println("null")
           end
       end
bar (generic function with 1 method)

julia> bar("libfoo")
WARNING: Base.Uint8 is deprecated, use UInt8 instead.
in bar at none:2
WARNING: Base.Uint8 is deprecated, use UInt8 instead.
WARNING: Base.Uint8 is deprecated, use UInt8 instead.
WARNING: Base.Uint8 is deprecated, use UInt8 instead.
in bar at none:2
ERROR: auto_unbox: unable to determine argument type
 in bar at none:3

but it works fine if you use the uppercase UInt8 instead.

@timholy timholy added this to the 0.4.0 milestone Sep 18, 2015
@timholy
Copy link
Sponsor Member Author

timholy commented Sep 18, 2015

This is quite breaking, so I added the tag.

@yuyichao
Copy link
Contributor

Isn't this already fixed by #13180 ?

@yuyichao
Copy link
Contributor

IIUC Tony have already confirmed with PkgEval that the fixed the issue so I'm going to close this. Feel free to reopen if it is still a problem.

@timholy
Copy link
Sponsor Member Author

timholy commented Sep 18, 2015

Not in release-0.4 yet. I can't keep up with all the email, so I'll just ask: already slated, or does something more need to be done?

@mbauman
Copy link
Sponsor Member

mbauman commented Sep 18, 2015

It's due to be merged in #13182

@yuyichao
Copy link
Contributor

It should work on master already and I think it's in Tony's latest queue of 0.4 backport.

Edit: Yeah that one ^^^

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

No branches or pull requests

3 participants