[dpdk-dev] rte_kvargs:fix comments to use commas not semicolons

Message ID 20180310162428.25111-1-keith.wiles@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Wiles, Keith March 10, 2018, 4:24 p.m. UTC
  Signed-off-by: Keith Wiles <keith.wiles@intel.com>
---
 lib/librte_kvargs/rte_kvargs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Ferruh Yigit March 12, 2018, 11:16 a.m. UTC | #1
On 3/10/2018 4:24 PM, Keith Wiles wrote:
> Signed-off-by: Keith Wiles <keith.wiles@intel.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
  
Thomas Monjalon March 27, 2018, 10:42 p.m. UTC | #2
12/03/2018 12:16, Ferruh Yigit:
> On 3/10/2018 4:24 PM, Keith Wiles wrote:
> > Signed-off-by: Keith Wiles <keith.wiles@intel.com>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Fixes: a8b97e3a1db0 ("devargs: use a comma instead of semicolon to separate key/values")

Applied, thanks
  

Patch

diff --git a/lib/librte_kvargs/rte_kvargs.c b/lib/librte_kvargs/rte_kvargs.c
index 9662375e8..d92a5f9dc 100644
--- a/lib/librte_kvargs/rte_kvargs.c
+++ b/lib/librte_kvargs/rte_kvargs.c
@@ -13,7 +13,7 @@ 
 
 /*
  * Receive a string with a list of arguments following the pattern
- * key=value;key=value;... and insert them into the list.
+ * key=value,key=value,... and insert them into the list.
  * strtok() is used so the params string will be copied to be modified.
  */
 static int
@@ -154,7 +154,7 @@  rte_kvargs_free(struct rte_kvargs *kvlist)
 }
 
 /*
- * Parse the arguments "key=value;key=value;..." string and return
+ * Parse the arguments "key=value,key=value,..." string and return
  * an allocated structure that contains a key/value list. Also
  * check if only valid keys were used.
  */