----- Original Message ----- > TS-1820 Introduce TS_UNUSED, and migrate most of the code to use > either C++ style unused parameters, or this new macro [snip] > Commit: e343dbbc60819256e8db7bc0d1f0ae7a4067c7cf > Parents: 3840df1 [snip] > http://git-wip-us.apache.org/repos/asf/trafficserver/blob/e343dbbc/example/add-header/add-header.c > ---------------------------------------------------------------------- > diff --git a/example/add-header/add-header.c > b/example/add-header/add-header.c > index c0fd6f4..a329162 100644 > --- a/example/add-header/add-header.c > +++ b/example/add-header/add-header.c > @@ -35,13 +35,15 @@ > #include <ctype.h> > #include <stdio.h> > #include <string.h> > -#include <ts/ts.h> > + > +#include "ts/ts.h" > +#include "ink_defs.h"
You don't mention this conversion in the comments, what's the motivation behind it? [snip] > @@ -232,7 +236,7 @@ check_ts_version() > } > > void > -TSPluginInit(int argc, const char *argv[]) > +TSPluginInit(int argc ATS_UNUSED, const char *argv[] ATS_UNUSED) > { > int i, cc; > TSPluginRegistrationInfo info; Your commit message says TS_UNUSED, but you use ATS_UNUSED Perhaps we should once and for all document when to use ts_ and when ats_ https://cwiki.apache.org/confluence/display/TS/Coding+Style -- i Igor Galić Tel: +43 (0) 664 886 22 883 Mail: i.ga...@brainsware.org URL: http://brainsware.org/ GPG: 6880 4155 74BD FD7C B515 2EA5 4B1D 9E08 A097 C9AE