Re: [libvirt PATCH] docs: Rephrase the video 'none' type to clarify the reasoning

On a Tuesday in 2020, Michal Privoznik wrote: > On 9/8/20 11:59 AM, Erik Skultety wrote: > > Signed-off-by: Erik Skultety <eskultet(a)redhat.com&gt; > > --- > > docs/formatdomain.rst | 17 ++++++++++------- > > 1 file changed, 10 insertions(+), 7 deletions(-) > > > > This is a tiny follow-up improvement resulting from [1] as I felt the paragraph > > wasn't clear enough + there were a couple of typographical issues. > > [1] https://www.redhat.com/archives/libvir-list/2020-September/msg00329.html > > > > diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst > > index 1979dfb8d3..cf3ccbecc5 100644 > > --- a/docs/formatdomain.rst > > +++ b/docs/formatdomain.rst > > @@ -5784,13 +5784,16 @@ A video device. > > value "vga", "cirrus", "vmvga", "xen", "vbox", "qxl" ( :since:`since 0.8.6` > > ), "virtio" ( :since:`since 1.3.0` ), "gop" ( :since:`since 3.2.0` ), "bochs" > > ( :since:`since 5.6.0` ), "ramfb" ( :since:`since 5.9.0` ), or "none" ( > > - :since:`since 4.6.0` , depending on the hypervisor features available. The > > - purpose of the type ``none`` is to instruct libvirt not to add a default > > - video device in the guest (see the paragraph above). This legacy behaviour > > - can be inconvenient in cases where GPU mediated devices are meant to be the > > - only rendering device within a guest and so specifying another ``video`` > > - device along with type ``none``. Refer to Host device assignment to see how > > - to add a mediated device into a guest. > > + :since:`since 4.6.0`) , depending on the hypervisor features available. > > Looks like we like an extra space before ')' in this paragraph. > That is just the result of the automatic conversion. The version without spaces in parentheses seems to render just fine for me, so we can change it.