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

Re: [PATCH v2 04/13] tools/xenstore: let hashtable_insert() return 0 on success



Hi,

On 30/03/2023 09:50, Juergen Gross wrote:
Today hashtable_insert() returns 0 in case of an error. Change that to
let it return an errno value in the error case and 0 in case of success.
I usually find such change risky because it makes the backport more 
complex if we introduce a new call to hashtable_insert() and it is also 
quite difficult to review (the compiler would not help to confirm all 
the callers have changed).
So can you provide a compelling reason for doing the change? 
(consistency would not be one IMO)
Cheers,

--
Julien Grall



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.