diff mbox

[dpdk-dev,4/5] cmdline: make parse_set_list() use size_t instead of int for low/high parameter

Message ID 1424700600-1765-5-git-send-email-pawelx.wodkowski@intel.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Wodkowski, PawelX Feb. 23, 2015, 2:09 p.m. UTC 
  Fix warning reported by klocwork about size_t to int cast when passing
parameters to parse_set_list().

This patch fix code formating errors that give checkpatch.pl errors
after generating patch.

Signed-off-by: Pawel Wodkowski <pawelx.wodkowski@intel.com>
---
 lib/librte_cmdline/cmdline_parse_portlist.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Olivier Matz Feb. 24, 2015, 11:05 a.m. UTC | #1 
Hi Pawel,

On 02/23/2015 03:09 PM, Pawel Wodkowski wrote:
> Fix warning reported by klocwork about size_t to int cast when passing
> parameters to parse_set_list().
>
> This patch fix code formating errors that give checkpatch.pl errors
> after generating patch.
>
> Signed-off-by: Pawel Wodkowski <pawelx.wodkowski@intel.com>

Acked-by: Olivier Matz <olivier.matz@6wind.com>
diff mbox

Patch

diff --git a/lib/librte_cmdline/cmdline_parse_portlist.c b/lib/librte_cmdline/cmdline_parse_portlist.c
index fc6c14e..9c1fe3e 100644
--- a/lib/librte_cmdline/cmdline_parse_portlist.c
+++ b/lib/librte_cmdline/cmdline_parse_portlist.c
@@ -78,7 +78,7 @@  struct cmdline_token_ops cmdline_token_portlist_ops = {
 };
 
 static void
-parse_set_list(cmdline_portlist_t * pl, int low, int high)
+parse_set_list(cmdline_portlist_t *pl, size_t low, size_t high)
 {
 	do {
 		pl->map |= (1 << low++);
@@ -86,7 +86,7 @@  parse_set_list(cmdline_portlist_t * pl, int low, int high)
 }
 
 static int
-parse_ports(cmdline_portlist_t * pl, const char * str)
+parse_ports(cmdline_portlist_t *pl, const char *str)
 {
 	size_t ps, pe;
 	const char *first, *last;