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