diff mbox

[dpdk-dev] lib/librte_cryptodev: Fix a typo: rte_cyptodev_names -> rte_cryptodev_names

Message ID 1510300576-17000-1-git-send-email-pavel.shirshov@gmail.com (mailing list archive)
State Rejected, archived
Headers

Checks

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

Commit Message

Pavel Shirshov Nov. 10, 2017, 7:56 a.m. UTC 
  Signed-off-by: Pavel Shirshov <pavel.shirshov@gmail.com>
---
 lib/librte_cryptodev/rte_cryptodev.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

De Lara Guarch, Pablo Nov. 10, 2017, 8:30 a.m. UTC | #1 
Hi Pavel,

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Pavel Shirshov
> Sent: Friday, November 10, 2017 7:56 AM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] lib/librte_cryptodev: Fix a typo:
> rte_cyptodev_names -> rte_cryptodev_names
> 

Actually, this variable is not used anywhere, so it can be deleted.

By the way, for future times.
In the commit title, do not use "lib/librte...", but directly the library name.
Also, start with lowercase, and try to be as short as possible, but still meaningful.
For this case, it is not necessary to describe the typo in the title.

So, this would be: "cryptodev: fix typo"

Thanks,
Pablo
diff mbox

Patch

diff --git a/lib/librte_cryptodev/rte_cryptodev.h b/lib/librte_cryptodev/rte_cryptodev.h
index dade554..5b6a254 100644
--- a/lib/librte_cryptodev/rte_cryptodev.h
+++ b/lib/librte_cryptodev/rte_cryptodev.h
@@ -50,7 +50,7 @@  extern "C" {
 #include "rte_dev.h"
 #include <rte_common.h>
 
-extern const char **rte_cyptodev_names;
+extern const char **rte_cryptodev_names;
 
 /* Logging Macros */