Add a global set to hold references to the handles we create. Need to…

[drnlm]

… fix a bug exposed by later cffi versions

Fixes the current breakage on travis.

[hodgestar]

Looks good to me. Just wanted to check whether, in the cases where we don't set rwops.hidden.unknown.data1 ourselves (i.e. we get rwops from SDL_RWFromFP), data1 is still guaranteed to be set to something sensible for discarding from __localhandles?

[drnlm]

We only add the obj_close method (where discard is called) onto objects where we've explicitly set data1 - if we get rwops fro SDL_RWFromFP, then the SDL functions are used, no handle is created and this code isn't involved.

There is a potential concern that this approach can leak references - destroying a RWOps object without explicitly call close will leave a non longer useful handle still in the set - I don't see that this is particularly worth worrying about, since we already leak some memory because SDL_FreeRW isn't called in that case either (as does pygame).

[hodgestar]

Woot.

👍