On Thu, Jan 06, 2022 at 03:09:09PM +0100, Laszlo Ersek wrote:
> + | { q35; vio10 }, { Libosinfo.id } :: tail ->
> + let q35 = q35 || id = "http://qemu.org/chipset/x86/q35";
> + and vio10 = vio10 || id = "http://pcisig.com/pci/1af4/1041";
> + in
As a matter of style we usually put "in" on its own line only after a
function definition. In this case it's defining a non-function so I'd
put the "in" at the end of the previous line.
Patch looks fine otherwise so ACK
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages. http://libguestfs.org
_______________________________________________
Libguestfs mailing list
[email protected]
https://listman.redhat.com/mailman/listinfo/libguestfs
