[PATCH v3 1/7] Make getfdproto returns integer instead of string
Fabien Siron
fabien.siron at epita.fr
Thu Jun 16 12:25:02 UTC 2016
Quoting Dmitry V. Levin (2016-06-15 17:36:35)
> On Wed, Jun 15, 2016 at 12:42:59PM +0000, Fabien Siron wrote:
> > This commit aims to avoid future string comparisons with getfdproto function
> > in changing its return type. It also adapts the functions that use it.
>
> I'm not quite sure what does the first sentence mean.
Do you prefer "This commit introduces a new type (sock_proto) which replaces
strings for socket protocols."?
>
> > * defs.h (print_sockaddr_by_inode, get_proto_by_name): Add.
>
> Only get_proto_by_name is a new prototype, print_sockaddr_by_inode was
> there before the change.
>
> > (sock_proto): New enum.
> > * socketutils.c (print_sockaddr_by_inode): Move proto to int.
>
> I'm not quite sure what does "Move proto to int" mean.
Do you prefer "Convert proto argument to sock_proto type."?
>
> --
> ldv
>
>
Regards,
--
Fabien Siron
More information about the Strace-devel
mailing list