I have taken Eric's work and attached it as a unified diff. I made no other changes, but I thought this would be easier to apply than to cut and paste out of emails. Thanks Eric!<br><br>- Kirby Kuehl<br><br><div class="gmail_quote">

On Mon, Jan 11, 2010 at 7:27 AM, Will Metcalf <span dir="ltr"><<a href="mailto:william.metcalf@gmail.com">william.metcalf@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Eric,<br><br>Wow!  Thanks for the patches.  We will have a look and work on getting them integrated as needed into the current version of the code in the public git repo. ;-).<br><br>Regards,<br><font color="#888888"><br>

Will</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">
On Mon, Jan 11, 2010 at 4:11 AM, Eric Leblond <span dir="ltr"><<a href="mailto:eleblond@edenwall.com" target="_blank">eleblond@edenwall.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


<br>
This patchset contains mainly patches that could be called code<br>
cleaning. Some of them are just humble proposal that reflect some<br>
particular coding style I'm used to see in some places.<br>
<br>
The only fixed issue is provided by<br>
    [PATCH 1/8] Fix typo in Makefile.am<br>
which fixes 'make tag' command<br>
<br>
The two next patches are a proposal, they only suggest using 'switch'<br>
construction instead of 'else if':<br>
    [PATCH 2/8] ethernet: use switch instead of 'else if'<br>
    [PATCH 3/8] nfq: use switch instead of 'else if'<br>
<br>
Patch 4 suppresses some TODO by fixing them:<br>
    [PATCH 4/8] nfq: add sanity checking<br>
<br>
The rest of the patchset is relative to using -Wextra during compilation<br>
to benefit from all gcc warnings. I was quiet impressed that there was<br>
almost any warning in this mode !<br>
<br>
    [PATCH 5/8] configure: compile with -Wextra<br>
    [PATCH 6/8] convert action_type to enum<br>
    [PATCH 7/8] gcc warning fixes.<br>
    [PATCH 8/8] fix code file permission<br>
<br>
BR,<br>
<font color="#888888">--<br>
Eric Leblond <<a href="mailto:eleblond@edenwall.com" target="_blank">eleblond@edenwall.com</a>><br>
_______________________________________________<br>
Oisf-devel mailing list<br>
<a href="mailto:Oisf-devel@openinfosecfoundation.org" target="_blank">Oisf-devel@openinfosecfoundation.org</a><br>
<a href="http://lists.openinfosecfoundation.org/mailman/listinfo/oisf-devel" target="_blank">http://lists.openinfosecfoundation.org/mailman/listinfo/oisf-devel</a><br>
</font></blockquote></div><br>
</div></div><br>_______________________________________________<br>
Oisf-devel mailing list<br>
<a href="mailto:Oisf-devel@openinfosecfoundation.org">Oisf-devel@openinfosecfoundation.org</a><br>
<a href="http://lists.openinfosecfoundation.org/mailman/listinfo/oisf-devel" target="_blank">http://lists.openinfosecfoundation.org/mailman/listinfo/oisf-devel</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>PGP Fingerprint: 8784 EAFE E9C5 D083 3E81  727C F265 5AEF E6F8 A671<br>