Re: [libosinfo v5 06/12] Remove now redundant code in InstallConfigParam

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

 



On Tue, Nov 20, 2012 at 12:17:15AM +0200, Zeeshan Ali (Khattak) wrote:
> On Mon, Nov 19, 2012 at 7:07 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > I guess I would have kept the empty 'priv' struct in case it's ever needed
> > in the future, which would make _init non empty.
> 
> *If* we ever need it, its not so hard to add it (even from scratch) so
> not worth keeping redundant code/allocations.

For what it's worth, my understanding is that there is no extra allocation
involved. And removing this code and readding it is still more costly
time-wise than just keeping it ;) But I don't feel very strongly about
that.

Christophe

Attachment: pgpe8cqrxNONC.pgp
Description: PGP signature


[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux