[FFmpeg-cvslog] avutil/qsort: use the do while form for AV_QSORT, AV_MSORT

Ganesh Ajjanagadde git at videolan.org
Fri Oct 23 14:44:01 CEST 2015


ffmpeg | branch: master | Ganesh Ajjanagadde <gajjanagadde at gmail.com> | Thu Oct 22 22:52:11 2015 -0400| [8ed79c45b47f274c58180b8526f7a8cab061cbed] | committer: Ganesh Ajjanagadde

avutil/qsort: use the do while form for AV_QSORT, AV_MSORT

Reviewed-by: Clément Bœsch <u at pkh.me>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=8ed79c45b47f274c58180b8526f7a8cab061cbed
---

 libavutil/qsort.h |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libavutil/qsort.h b/libavutil/qsort.h
index 30edcc8..eda24b0 100644
--- a/libavutil/qsort.h
+++ b/libavutil/qsort.h
@@ -27,7 +27,7 @@
  * to construct input that requires O(n^2) time but this is very unlikely to
  * happen with non constructed input.
  */
-#define AV_QSORT(p, num, type, cmp) {\
+#define AV_QSORT(p, num, type, cmp) do {\
     void *stack[64][2];\
     int sp= 1;\
     stack[0][0] = p;\
@@ -89,7 +89,7 @@
             }\
         }\
     }\
-}
+} while (0)
 
 /**
  * Merge sort, this sort requires a temporary buffer and is stable, its worst
@@ -97,7 +97,7 @@
  * @param p     must be a lvalue pointer, this function may exchange it with tmp
  * @param tmp   must be a lvalue pointer, this function may exchange it with p
  */
-#define AV_MSORT(p, tmp, num, type, cmp) {\
+#define AV_MSORT(p, tmp, num, type, cmp) do {\
     unsigned i, j, step;\
     for(step=1; step<(num); step+=step){\
         for(i=0; i<(num); i+=2*step){\
@@ -114,4 +114,4 @@
         }\
         FFSWAP(type*, p, tmp);\
     }\
-}
+} while (0)



More information about the ffmpeg-cvslog mailing list