On Thu, Nov 28, 2024 at 10:24:07 +0100, Peter Krempa wrote:
> On Thu, Nov 28, 2024 at 10:22:53 +0100, Jiri Denemark wrote:
> > On Tue, Nov 26, 2024 at 16:16:17 +0100, Peter Krempa wrote:
> > > Extract the matching of the node name of a single virStorage source so
> > > that the logic can be extended in the upcoming patch.
> > 
> > This is confusing. I was expecting the logic in
> > qemuDomainVirStorageSourceFindByNodeName to be extended in the following
> > patch, but in reality you just needed to reuse the same code in another
> > place. That is the goal of moving the code to a separate function was to
> > avoid code duplication.
> 
> so s/extended/reused/ ?

Sounds good.

Jirka

Reply via email to