Remove a few if (p) av_freep(&p) forms
authorClément Bœsch <ubitux@gmail.com>
Thu, 3 Feb 2011 10:30:17 +0000 (11:30 +0100)
committerMans Rullgard <mans@mansr.com>
Fri, 4 Feb 2011 10:00:49 +0000 (10:00 +0000)
Signed-off-by: Mans Rullgard <mans@mansr.com>
libavcodec/libxvidff.c
libavutil/tree.h
libswscale/swscale-test.c

index 771fc2b..9008b84 100644 (file)
@@ -527,8 +527,7 @@ static av_cold int xvid_encode_close(AVCodecContext *avctx) {
 
     xvid_encore(x->encoder_handle, XVID_ENC_DESTROY, NULL, NULL);
 
-    if( avctx->extradata != NULL )
-        av_freep(&avctx->extradata);
+    av_freep(&avctx->extradata);
     if( x->twopassbuffer != NULL ) {
         av_free(x->twopassbuffer);
         av_free(x->old_twopassbuffer);
index bf09fd0..8c7de2f 100644 (file)
@@ -67,7 +67,7 @@ void *av_tree_find(const struct AVTreeNode *root, void *key, int (*cmp)(void *ke
  *                 return av_tree_insert(rootp, key, cmp, next);
  *             }
  *             void *tree_remove(struct AVTreeNode **rootp, void *key, int (*cmp)(void *key, const void *b, AVTreeNode **next)){
- *                 if(*next) av_freep(next);
+ *                 av_freep(next);
  *                 return av_tree_insert(rootp, key, cmp, next);
  *             }
  *             @endcode
index ae5f8b7..bd24f3e 100644 (file)
@@ -104,8 +104,7 @@ static int doTest(uint8_t *ref[4], int refStride[4], int w, int h,
         int p;
 
         for (p = 0; p < 4; p++)
-            if (src[p])
-                av_freep(&src[p]);
+            av_freep(&src[p]);
 
         av_image_fill_linesizes(srcStride, srcFormat, srcW);
         for (p = 0; p < 4; p++) {