[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