Looks good to me. Next time, it would be even better to leave the removal of the (now) redundant casting for another patch. That way, the commit message would more properly reflect the patch.
Applied to cdk-1.4.x.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Yes, looks good to me. Can you convert that into a git patch? (We'll chat about that on IRC...)
Commit in Git:
https://github.com/dmak/cdk/commit/53ee4b64aa802db381d32a0ddfceaf02e87e9c00
Looks good to me. Next time, it would be even better to leave the removal of the (now) redundant casting for another patch. That way, the commit message would more properly reflect the patch.
Applied to cdk-1.4.x.