[Tarantool-patches] [PATCH 10/13] popen: add missed diag_set() in popen IO functions
Cyrill Gorcunov
gorcunov at gmail.com
Fri Apr 10 11:28:22 MSK 2020
On Fri, Apr 10, 2020 at 05:50:48AM +0300, Alexander Turenko wrote:
> Our usual convention for C code is to return a negative value at failure
> and set an entry to the diagnostics area.
>
> When code uses this convention consistently, it is much easier to handle
> failures when using it: you always know where to find an error type and
> message and how to pass the error to a C or Lua caller.
>
> See also the previous commit ('popen: add missed diag_set in
> popen_signal/delete').
>
> Part of #4031
Acked-by: Cyrill Gorcunov <gorcunov at gmail.com>
More information about the Tarantool-patches
mailing list