On 08.06.23 16:56, Eric Blake wrote:
Peform several minor refactorings of how the list of negotiated meta
contexts is managed, to make upcoming patches easier: Promote the
internal type NBDExportMetaContexts to the public opaque type
NBDMetaContexts, and mark exp const.  Use a shorter member name in
NBDClient.  Hoist calls to nbd_check_meta_context() earlier in their
callers, as the number of negotiated contexts may impact the flags
exposed in regards to an export, which in turn requires a new
parameter.

Hmm, a bit of semantic change: we drop context in nbd_check_meta_export() even 
when we report error. Pre-patch we call nbd_check_meta_export() only on success 
path.. Still, it seems even more safe.

Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@yandex-team.ru>

  Drop a redundant parameter to nbd_negotiate_meta_queries.
No semantic change intended.

Signed-off-by: Eric Blake <ebl...@redhat.com>


--
Best regards,
Vladimir

_______________________________________________
Libguestfs mailing list
Libguestfs@redhat.com
https://listman.redhat.com/mailman/listinfo/libguestfs

Reply via email to