Il mer 11 mar 2020, 15:50 Markus Armbruster <
address@hidden> ha scritto:
Stefan Hajnoczi <address@hidden> writes:
> Lock guards automatically call qemu_(rec_)mutex_unlock() when returning from a
> function or leaving leaving a lexical scope. This simplifies code and
> eliminates leaks (especially in error code paths).
>
> This series adds lock guards for QemuMutex and QemuRecMutex. It does not
> convert the entire tree but includes example conversions.
I support the move towards automatic cleanup, but I'm wary of
incremental conversion. Experience tells that outdated examples
invariably get copied / imitated, with incremental conversion struggling
to keep up.
Are we resigned to having both kinds of mutex cleanup forever?
There are cases where the legibility benefits of guards are debatable, or they require more complex functionality in the guards (see my other answer to Stefan). So, yes. We don't have that many mutexes so incremental conversion should be doable without taking forever.
Paolo
If not, what's the plan to get the job finished, and until when?