The patch is fine. Notice that it is currently out of date, due to inclusion of patch #1070046.

This other patch added generic support for objects having an __dict__ attribute, marshaling them struct elements. This is in real conflict to this patch: an object both supporting a list conversion and having a __dict__ attribute could be marshaled either way.

My proposal is to resolve this in favor of this patch: If an object has a list conversion, it's most likely meant to be a list.

If you agree to this resolution, please update the code and the documentation, and apply this patch.