On Fri, Nov 13, 2020 at 12:47:59PM +0300, Serge Petrenko wrote: > > AFAIK we use `return 0/-1` for such functions. bool is only used for > something that > really returns true or false, like `field_mp_type_is_compatible()`. > > Other than that the patch looks good. OK, thanks!