Re: [PATCH stable-2.10] Make network_vlan an optional OpParam

['Klaus Aehlig' via ganeti-devel]

Hi Dimara,

> [...]
> To bypass a sphinx warning (i.e. Inline literal start-string without
> end-string), substitute "" with "EmptyString" in the generated html.

I'm a bit conused about that work around. Why not avoid bad generated
html by using a different wording? I.e., replace

>        if has_default:
> +        if default == "":
> +          default = "EmptyString"
>          buf.write("defaults to ``%s``" % (default,))

by

  if has_default:
    if default == "":
      buf.write("defaults to the empty string")
    else:
      buf.write("defaults to ``%s``" % (default,))

?

Thanks, Klaus

-- 
Klaus Aehlig
Google Germany GmbH, Dienerstr. 12, 80331 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Graham Law, Christine Elizabeth Flores