diff mbox

[ovs-dev] dp-packet: Remove misleading comment for refill init function.

Message ID 1500019898-31190-1-git-send-email-i.maximets@samsung.com
State Accepted
Headers show

Commit Message

Ilya Maximets July 14, 2017, 8:11 a.m. UTC
Function 'dp_packet_batch_refill_init' doesn't return anything.
Looks like this comment came from one of the intermediate versions
of the API enhancement patch. Additionally comment style changed
to be consistent with other comments in the same file.

CC: Andy Zhou <azhou@ovn.org>
Fixes: 72c84bc2db23 ("dp-packet: Enhance packet batch APIs.")
Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
---
 lib/dp-packet.h | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Andy Zhou July 14, 2017, 8:11 p.m. UTC | #1
On Fri, Jul 14, 2017 at 1:11 AM, Ilya Maximets <i.maximets@samsung.com> wrote:
> Function 'dp_packet_batch_refill_init' doesn't return anything.
> Looks like this comment came from one of the intermediate versions
> of the API enhancement patch. Additionally comment style changed
> to be consistent with other comments in the same file.
>
> CC: Andy Zhou <azhou@ovn.org>
> Fixes: 72c84bc2db23 ("dp-packet: Enhance packet batch APIs.")
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>

Thanks. Applied.
diff mbox

Patch

diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index 38282bd..c5fe32e 100644
--- a/lib/dp-packet.h
+++ b/lib/dp-packet.h
@@ -678,11 +678,8 @@  dp_packet_batch_size(const struct dp_packet_batch *batch)
     return batch->count;
 }
 
-/*
- * Clear 'batch' for refill. Use dp_packet_batch_refill() to add
- * packets back into the 'batch'.
- *
- * Return the original size of the 'batch'.  */
+/* Clear 'batch' for refill. Use dp_packet_batch_refill() to add
+ * packets back into the 'batch'. */
 static inline void
 dp_packet_batch_refill_init(struct dp_packet_batch *batch)
 {