JuliaIO / LibExpat.jl

Julia interface to the Expat XML parser library
Other
9 stars 32 forks source link

use Csize_t instead of size_t #73

Closed tkelman closed 7 years ago

SimonDanisch commented 7 years ago

this was simply wrong and not working before, no?

tkelman commented 7 years ago

It was wrong, and accidentally working because ccall wasn't so picky if it wasn't getting called. Jameson's recently merged "eliminate intrinsics" PR to base has made ccall more picky about errors like this

SimonDanisch commented 7 years ago

Ah that makes sense!

tkelman commented 7 years ago

looks like the callbacks are probably 265-broken, not sure how to fix that

vtjnash commented 7 years ago

I'll fix that on Julia master fairly soon