From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id 826B84A63; Fri, 25 Nov 2016 07:31:40 +0100 (CET) In-Reply-To: <1480055491-137021-1-git-send-email-chunguang.yang@windriver.com> References: <1480055491-137021-1-git-send-email-chunguang.yang@windriver.com> To: test-report@dpdk.org Cc: chunguang yang Message-Id: <20161125063140.826B84A63@dpdk.org> Date: Fri, 25 Nov 2016 07:31:40 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| [PATCH] lpm: rte_lpm_iterate() - iterate through the routes X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Nov 2016 06:31:40 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/17257 _coding style issues_ ERROR:GERRIT_CHANGE_ID: Remove Gerrit Change-Id's before submitting upstream. #56: Change-Id: I28ea3d7d92f318988444553ee2bb30b709bcb3b6 WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #138: FILE: lib/librte_lpm/rte_lpm_iterate.c:49: +rte_lpm_iterate(struct rte_lpm_route* const buf, unsigned buflen, ERROR:POINTER_LOCATION: "foo* const bar" should be "foo * const bar" #138: FILE: lib/librte_lpm/rte_lpm_iterate.c:49: +rte_lpm_iterate(struct rte_lpm_route* const buf, unsigned buflen, ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar" #139: FILE: lib/librte_lpm/rte_lpm_iterate.c:50: + const struct rte_lpm* lpm, ERROR:POINTER_LOCATION: "foo* const bar" should be "foo * const bar" #140: FILE: lib/librte_lpm/rte_lpm_iterate.c:51: + struct rte_lpm_cursor* const cursor) ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar" #142: FILE: lib/librte_lpm/rte_lpm_iterate.c:53: + struct rte_lpm_route* p = buf; ERROR:POINTER_LOCATION: "foo* const bar" should be "foo * const bar" #143: FILE: lib/librte_lpm/rte_lpm_iterate.c:54: + struct rte_lpm_route* const end = p + buflen; ERROR:POINTER_LOCATION: "foo* const bar" should be "foo * const bar" #145: FILE: lib/librte_lpm/rte_lpm_iterate.c:56: + const struct rte_lpm_rule_info* const rinfo = lpm->rule_info; ERROR:POINTER_LOCATION: "foo* const bar" should be "foo * const bar" #146: FILE: lib/librte_lpm/rte_lpm_iterate.c:57: + const struct rte_lpm_rule* const rtbl = lpm->rules_tbl; WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #148: FILE: lib/librte_lpm/rte_lpm_iterate.c:59: + unsigned d = cursor->d; WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #149: FILE: lib/librte_lpm/rte_lpm_iterate.c:60: + unsigned n = cursor->n; ERROR:SPACING: spaces required around that '!=' (ctx:VxV) #151: FILE: lib/librte_lpm/rte_lpm_iterate.c:62: + while(p!=end) { ^ ERROR:SPACING: space required before the open parenthesis '(' #151: FILE: lib/librte_lpm/rte_lpm_iterate.c:62: + while(p!=end) { ERROR:SPACING: spaces required around that '==' (ctx:VxV) #152: FILE: lib/librte_lpm/rte_lpm_iterate.c:63: + if(d==32) break; ^ ERROR:SPACING: space required before the open parenthesis '(' #152: FILE: lib/librte_lpm/rte_lpm_iterate.c:63: + if(d==32) break; ERROR:TRAILING_STATEMENTS: trailing statements should be on next line #152: FILE: lib/librte_lpm/rte_lpm_iterate.c:63: + if(d==32) break; ERROR:SPACING: spaces required around that '>=' (ctx:VxV) #153: FILE: lib/librte_lpm/rte_lpm_iterate.c:64: + if(n>=rinfo[d].used_rules) { ^ ERROR:SPACING: space required before the open parenthesis '(' #153: FILE: lib/librte_lpm/rte_lpm_iterate.c:64: + if(n>=rinfo[d].used_rules) { WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #218: FILE: lib/librte_lpm/rte_lpm_iterate.h:42: + unsigned d; WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #219: FILE: lib/librte_lpm/rte_lpm_iterate.h:43: + unsigned n; WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #228: FILE: lib/librte_lpm/rte_lpm_iterate.h:52: +unsigned rte_lpm_iterate(struct rte_lpm_route* buf, unsigned buflen, WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #228: FILE: lib/librte_lpm/rte_lpm_iterate.h:52: +unsigned rte_lpm_iterate(struct rte_lpm_route* buf, unsigned buflen, ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar" #228: FILE: lib/librte_lpm/rte_lpm_iterate.h:52: +unsigned rte_lpm_iterate(struct rte_lpm_route* buf, unsigned buflen, ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar" #229: FILE: lib/librte_lpm/rte_lpm_iterate.h:53: + const struct rte_lpm* lpm, ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar" #230: FILE: lib/librte_lpm/rte_lpm_iterate.h:54: + struct rte_lpm_cursor* cursor); total: 18 errors, 7 warnings, 149 lines checked