OK for a patch, but it would be better to write the casts for modern c++ as reinterpret_cast( ... ) These are cpp files, and c-style casts are really out of fashion.