On Tue, Aug 13, 2019 at 11:17 PM net-snmp Git repository <
nore...@code.net-snmp.p.re.sourceforge.net> wrote:

> Branch: V5-8-patches
>
> treewide: Terminate netsnmp_feature_*() uses with a semicolon
>
> This patch has been generated by the following shell command:
>
> git grep -l netsnmp_feature_ |
> while read f; do sed -i 's/^netsnmp_feature.*[^;]$/&;/' "$f"; done
>
> By Bart Van Assche on 08/14/2019 02:58
>

Hi Bart,

Thank you very much for all the cleanups you've been working on recently.
I really appreciate it.

Have you tried using features with this change?  I am getting errors like:

[net-snmp] In file included from
*../include/net-snmp/net-snmp-features.h:11:0*,

[net-snmp]                  from *snmp_client.c:48*:

[net-snmp] *../include/net-snmp/agent/features.h:4:39:* *error: *ISO C99
requires whitespace after the macro name [*-Werror*]

[net-snmp]  #define NETSNMP_FEATURE_HAS_BABY_STEPS*;* 1

[net-snmp]                                        *^*

which appear likely to be due to the feature calculation infrastructure not
understanding these semicolons.  The semicolon seems to be making it into
the .ft file for *some* macros but not others:

./agent/helpers/baby_steps.ft:#define NETSNMP_FEATURE_PROVIDE_*BABY_STEPS*;
1

./agent/helpers/baby_steps.ft:#define
NETSNMP_FEATURE_*BABY_STEPS*_CHILD_OF_MIB_HELPERS;
1

./agent/helpers/baby_steps.ft:#define NETSNMP_FEATURE_PROVIDE_*BABY_STEPS* 1

./agent/helpers/baby_steps.ft:#define
NETSNMP_FEATURE_NETSNMP_*BABY_STEPS*_HANDLER_INIT_CHILD_OF_NETSNMP_UNUSED;
1

./agent/helpers/baby_steps.ft:#define NETSNMP_FEATURE_PROVIDE_NETSNMP_
*BABY_STEPS*_HANDLER_INIT 1

I'm happy to dig into this, but wanted to see if you had seen it already.
(Not asking for a revert; let's move this forward together)

Thanks,
  Bill
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to