Re: [PATCHv4] datamap support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 17, 2012 at 11:07 PM, Christophe Fergeau
<cfergeau@xxxxxxxxxx> wrote:
> Hey,
>
> Here is another iteration of the datamap support patches. There are not
> many big changes before patch 07/11. I took a different approach from the
> previous series in that the values are only transformed when calling
> the appropriate getter. This has the side effect of making the code
> less complex, of making the implementation of the cloning function in
> patch 10/11 easier, ... This also makes it easier to support modifications
> of the OsinfoInstallConfig:config-params properties during the life
> time of the OsinfoInstallConfig objects. And in turn, this makes
> it easier to not force applications to use a new constructor, which would
> be an ABI break.

Apart from the minor issues I pointed out, it looks pretty good
already. Thanks for the hard work on this and taking my
suggestions/criticism very seriously.

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux