[Libguestfs] RHashtable and the python bindings

[Richard W.M. Jones]

On Wed, Apr 03, 2013 at 05:29:57PM +0100, Matthew Booth wrote:
> I've discovered that libguestfs apis which return RHashtable are
> currently returned as a list of tuples in the python bindings rather
> than the obvious dict. I propose fixing this, whilst also maintaining
> API compatibility for a period of time.
> 
> The following apis return RHashtable:
> 
> inspect_get_mountpoints
> list_filesystems
> inspect_get_drive_mappings
> tune2fs_l
> mountpoints
> md_detail
> blkid
> list_disk_labels
> 
> I propose the following:
> 
> h = guestfs.GuestFS(return_dict=True)
> 
> If return_dict is set, any RHashtable api would return a dict rather
> than a list of tuples. If not, the behaviour would be unchanged. The
> default would be False.

Although it's not very likely, it is better to avoid a conflict with a
future guestfs_create_flags flag by calling the flag
'python_return_dict'.

Apart from that, it looks fine to me.

Rich.

> We would advertise immediately that this parameter is recommended for
> all new code, and that old code should be updated. At some point in the
> future the default would be changed to True, and at some point after
> that the old code could be removed.
> 
> Matt
> 
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at redhat.com
> https://www.redhat.com/mailman/listinfo/libguestfs

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming blog: http://rwmj.wordpress.com
Fedora now supports 80 OCaml packages (the OPEN alternative to F#)