[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v6 1/4] xen: introduce SYMBOL



On Fri, 11 Jan 2019, Jan Beulich wrote:
> > If all we really care about is making PRQA happy, I believe it does support
> > some sort of comment-based suppression. I've seen comments like
> > /* PRQA S 0487 */ or /* PRQA S 0488 */ in various codebases, I'm guessing
> > comments like this have something to do with suppressing these types of
> > warnings.
> 
> I have to admit that I'm opposed to comments: We've got some
> to please Coverity. We've got others to make certain editors to
> work for certain people. How many more are we going to gain?

You have a good point. Also, I am not a fan of tools specific tags
(Coverity, EMACS, etc.) in the code base. However, we could consider
adding our own comment labels, not to please PRQA, but to highlight
sites that we know are violating MISRA-C for one reason or another.
Something like:

/* M3CM: Rule-18.2 violation */

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.