On Wed, May 30, 2018 at 02:17:55PM +0200, Max Reitz wrote:
> On 2018-04-26 18:19, Roman Kagan wrote:
> > @@ -550,13 +561,26 @@ def verify_cache_mode(supported_cache_modes=[]):
> > if supported_cache_modes and (cachemode not in supported_cache_modes):
> > notrun('not suitable for this cache mode: %s' % cachemode)
> >
> > +rw_formats = None
> > +
> > +def supports_format(format_name):
> > + format_message = qemu_pipe('-drive', 'format=?')
> > + global rw_formats
> > + if rw_formats is None:
> > + rw_formats = format_message.splitlines()[0].split(':')[1].split()
>
> Isn't it sufficient to call qemu_pipe() only if rw_formats is None?
Sure it is, and this was exactly my intention with this global, but I
managed to forget moving the most relevant part under 'if' :$
Thanks for catching,
Roman.
