On 4 Jan 2019, at 20:42, Stephen Hemminger wrote:

On Fri,  4 Jan 2019 13:59:42 +0000
Eelco Chaudron <echau...@redhat.com> wrote:

This patch adds support for RFC4115 trTCM meters.

Signed-off-by: Eelco Chaudron <echau...@redhat.com>



Fix comment formatting.

Hi Stephen, I left these warnings in on purpose, to match the existing comment style in the file.

These specific warnings are from a structure copied from one right above which also has the same style and line lengths.

As they are warning I though I should leave it as is, Christian what are your thoughts on this?


### [dpdk-dev] [PATCH v4 1/2] lib/librte_meter: add RFC4115 trTCM meter support

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#172: FILE: lib/librte_meter/rte_meter.h:59:
+/** trTCM parameters per metered traffic flow. The CIR, EIR, CBS and EBS +parameters only count bytes of IP packets and do not include link specific

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#174: FILE: lib/librte_meter/rte_meter.h:61:
+none-zero respectively.*/

WARNING:LONG_LINE_COMMENT: line over 80 characters
#176: FILE: lib/librte_meter/rte_meter.h:63:
+ uint64_t cir; /**< Committed Information Rate (CIR). Measured in bytes per second. */

WARNING:LONG_LINE_COMMENT: line over 80 characters
#177: FILE: lib/librte_meter/rte_meter.h:64:
+ uint64_t eir; /**< Excess Information Rate (EIR). Measured in bytes per second. */

WARNING:LONG_LINE_COMMENT: line over 80 characters
#355: FILE: lib/librte_meter/rte_meter.h:406:
+ /**< Number of bytes currently available in the excess(E) token bucket */

total: 0 errors, 5 warnings, 356 lines checked

Reply via email to