On 20.08.25 09:30, Philippe Mathieu-Daudé wrote:
Hi Vladimir,
On 18/8/25 16:06, Vladimir Sementsov-Ogievskiy wrote:
To avoid error propagation, let's follow common recommendation to
use return value together with errp.
(looking at commit e3fe3988d78 "error: Document Error API usage rules"
again). While not return a boolean?
I thought about it, but decided to keep the common style of the file, all
functions here tend to return -1 on error and 0 on success. And making
1-2 exclusions will not make things more readable.
I think, if go this way, it should be another commit to convert the whole file
to use boolean functions where appropriate.
--
Best regards,
Vladimir
