[aravis] gvcp: post-increment in a return statement is not a good idea.



commit 28d6d35c980e663ebd1228077c61d25d29fd7dd4
Author: Emmanuel Pacaud <emmanuel gnome org>
Date:   Mon Jul 4 17:24:59 2011 +0200

    gvcp: post-increment in a return statement is not a good idea.

 src/arvgvcp.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
---
diff --git a/src/arvgvcp.h b/src/arvgvcp.h
index 1467cb8..54a3127 100644
--- a/src/arvgvcp.h
+++ b/src/arvgvcp.h
@@ -319,7 +319,7 @@ arv_gvcp_next_packet_count (guint16 packet_count)
 	/* packet_count == 0 is an error value */
 	if (packet_count == 0xffff)
 		return 1;
-	return packet_count++;
+	return packet_count + 1;
 }
 
 G_END_DECLS



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]