[07/16] rtpdec: Remove unnecessary _if_needed suffixes on functions

Message ID 1424727122-2827-7-git-send-email-martin@martin.st
State Committed
Commit db158f0dd217cf839be8af195d66cf49a76537a8
Headers show

Commit Message

Martin Storsjö Feb. 23, 2015, 9:31 p.m.
---
 libavformat/rtpdec_jpeg.c | 8 ++++----
 libavformat/rtpdec_xiph.c | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

Comments

Luca Barbato Feb. 23, 2015, 9:36 p.m. | #1
On 23/02/15 22:31, Martin Storsjö wrote:
> ---
>   libavformat/rtpdec_jpeg.c | 8 ++++----
>   libavformat/rtpdec_xiph.c | 8 ++++----
>   2 files changed, 8 insertions(+), 8 deletions(-)
>

Ok.

Patch

diff --git a/libavformat/rtpdec_jpeg.c b/libavformat/rtpdec_jpeg.c
index ed9c86c..caa1bd2 100644
--- a/libavformat/rtpdec_jpeg.c
+++ b/libavformat/rtpdec_jpeg.c
@@ -64,7 +64,7 @@  static PayloadContext *jpeg_new_context(void)
     return av_mallocz(sizeof(PayloadContext));
 }
 
-static inline void free_frame_if_needed(PayloadContext *jpeg)
+static inline void free_frame(PayloadContext *jpeg)
 {
     if (jpeg->frame) {
         uint8_t *p;
@@ -76,7 +76,7 @@  static inline void free_frame_if_needed(PayloadContext *jpeg)
 
 static void jpeg_free_context(PayloadContext *jpeg)
 {
-    free_frame_if_needed(jpeg);
+    free_frame(jpeg);
     av_free(jpeg);
 }
 
@@ -321,7 +321,7 @@  static int jpeg_parse_packet(AVFormatContext *ctx, PayloadContext *jpeg,
 
         /* Skip the current frame in case of the end packet
          * has been lost somewhere. */
-        free_frame_if_needed(jpeg);
+        free_frame(jpeg);
 
         if ((ret = avio_open_dyn_buf(&jpeg->frame)) < 0)
             return ret;
@@ -347,7 +347,7 @@  static int jpeg_parse_packet(AVFormatContext *ctx, PayloadContext *jpeg,
     if (jpeg->timestamp != *timestamp) {
         /* Skip the current frame if timestamp is incorrect.
          * A start packet has been lost somewhere. */
-        free_frame_if_needed(jpeg);
+        free_frame(jpeg);
         av_log(ctx, AV_LOG_ERROR, "RTP timestamps don't match.\n");
         return AVERROR_INVALIDDATA;
     }
diff --git a/libavformat/rtpdec_xiph.c b/libavformat/rtpdec_xiph.c
index 8f3ba18..c3385e4 100644
--- a/libavformat/rtpdec_xiph.c
+++ b/libavformat/rtpdec_xiph.c
@@ -54,7 +54,7 @@  static PayloadContext *xiph_new_context(void)
     return av_mallocz(sizeof(PayloadContext));
 }
 
-static inline void free_fragment_if_needed(PayloadContext * data)
+static inline void free_fragment(PayloadContext * data)
 {
     if (data->fragment) {
         uint8_t* p;
@@ -66,7 +66,7 @@  static inline void free_fragment_if_needed(PayloadContext * data)
 
 static void xiph_free_context(PayloadContext * data)
 {
-    free_fragment_if_needed(data);
+    free_fragment(data);
     av_free(data->split_buf);
     av_free(data);
 }
@@ -183,7 +183,7 @@  static int xiph_handle_packet(AVFormatContext *ctx, PayloadContext *data,
         int res;
 
         // end packet has been lost somewhere, so drop buffered data
-        free_fragment_if_needed(data);
+        free_fragment(data);
 
         if((res = avio_open_dyn_buf(&data->fragment)) < 0)
             return res;
@@ -196,7 +196,7 @@  static int xiph_handle_packet(AVFormatContext *ctx, PayloadContext *data,
         if (data->timestamp != *timestamp) {
             // skip if fragmented timestamp is incorrect;
             // a start packet has been lost somewhere
-            free_fragment_if_needed(data);
+            free_fragment(data);
             av_log(ctx, AV_LOG_ERROR, "RTP timestamps don't match!\n");
             return AVERROR_INVALIDDATA;
         }