[dpdk-dev] coding style question - c99 comments

Lance Richardson lance.richardson at broadcom.com
Fri Jan 24 22:37:52 CET 2020


Thanks, a patch adding C99_COMMENT_TOLERANCE has been posted.

   Lance

On Fri, Jan 24, 2020 at 12:10 PM Ferruh Yigit <ferruh.yigit at intel.com>
wrote:

> On 1/23/2020 4:43 PM, Lance Richardson wrote:
> > On Thu, Jan 23, 2020 at 11:15 AM Stephen Hemminger
> > <stephen at networkplumber.org> wrote:
> >>
> >> On Thu, 23 Jan 2020 11:05:58 -0500
> >> Lance Richardson <lance.richardson at broadcom.com> wrote:
> >>
> >>> The DPDK Coding Style document doesn't mention the use of c99 comments
> >>> using '//' as a comment delimiter, but it does say that traditional
> >>> '/*  */' comment
> >>> delimiters should be used and therefore seems to imply that c99-style
> comments
> >>> are not allowed.
> >>>
> >>> Is this correct?
> >>>
> >>>    Lance
> >>
> >> Don't use c99 comments.
> >> These will get flagged by checkpatch
> >
> > Hi Stephen,
> >
> > Actually, the reason I was asking was they aren't being flagged by
> checkpatch.
> >
> > It seems checkpatch.pl now has this:
> > my $allow_c99_comments = 1; # Can be overridden by --ignore
> > C99_COMMENT_TOLERANCE
> >
> > If c99 comments are should not be allowed, it looks like
> > C99_COMMENT_TOLERANCE should
> > be added to the list of checkers to ignore in checkpatches.sh. I was
> > thinking of rolling a patch,
> > but wanted to first confirm that they are actually not allowed.
> >
>
> Hi Lance,
>
> That is what I understand from coding style document that c99 comments are
> not
> allowed and agree to not use them.
>
> +1 to add C99_COMMENT_TOLERANCE
>
>


More information about the dev mailing list