[PATCH] No nested functions in dwarf_get{srclines,scopevar}

Chih-hung Hsieh chh at google.com
Thu Oct 15 23:40:47 UTC 2015


On Thu, Oct 15, 2015 at 1:18 PM, Roland McGrath <roland at hack.frob.com>
wrote:

> > +inline static int
>
> static inline int


Fixed.


>
> > +file_matches (Dwarf_Files * files, size_t idx,
>
> Gratuitous space after * there.
>
>
I found most other places do not have space after "*".
So I removed the extra space here to be consistent. :-)



> > +              const char *lastfile, size_t match_file_len,
> > +              const char *match_file, bool *lastfile_matches)
>
> When what should be a local function has several parameters like this to
> replace what should be the closed-over variables, I think it's clearer to
> put all those parameters first, before the real parameters to the function.
>
>
Changed, to have input read-only parameter first, followed by old
parameters and then output parameters.

Please review the new attached 0002*patch file again.
Thanks.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/elfutils-devel/attachments/20151015/a6a79ec4/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-No-nested-functions-in-dwarf_get-srclines-scopevar.patch
Type: application/octet-stream
Size: 16883 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/elfutils-devel/attachments/20151015/a6a79ec4/attachment-0001.obj>


More information about the elfutils-devel mailing list