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

Re: [lvm-devel] [PATCH 4/4] Update tests for lvseg apis.



Dne 22.10.2010 10:59, Petr Rockai napsal(a):
> Zdenek Kabelac <zkabelac redhat com> writes:
>>>> And I think using pointer would be wise decision here.
>>> Care to elaborate? What's wise about using a pointer here?
>>
>> (const struct lvm_proper_value *v)
>>
>> avoid doing local copy of the 'v' structure - currently it's not a big
>> difference, but as you pointed out in other emails - it could be extended in
>> future. And I think we are using  pointers in other calls as well - so we
>> should stay consistent also internally - and just pass pointers all the time -
>> instead of thinking when to use struct and when the pointer should be passed.
> 
> Unlike object lifetime, this is something that the compiler can check
> statically. Moreover, it may or may not be more efficient to pass a
> pointer. Usually it's not, unless the structure is big and/or the
> compiler optimisations poor.
> 
> Passing a value is prone to fewer errors. I vote for a value. (We should
> aim to pass values whenever possible, IMHO. Those vg_t/pv_t/lv_t behave
> as values as well, in this respect.)
> 

I'm not going to argue whether it's more or less efficient as in this case it
will make no difference.  But passing things by value is simply very hardly
supportable by dso libraries - and usually require complete rebuild of binary
to use updated library. Also you would need to remember where are you using
passing by value - and in case structure size grows - rewrite many functions
to switch to pointers - why not do that right from the beginning? -  I'd pass
by value only the language atomic/basic types - definitely not any structure
where even you are pointing out future extensions in other post.

BTW: I don't take vg_t/pv_t/lv_t as an an argument for passing things by value
as it's nothing else that syntactical sugar for pointers - and in fact I'd not
any objections against exactly same strategy for properties - making property
object completely private structure to lvm and give the API user only handle
lvm_property_t and set of function for this handle (C++ in C :))
(i.e.
lvm_property_is_string/get_string/is_integer/get_integer(lvm_property_t...)

Zdenek


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