[FFmpeg-cvslog] rtpenc_amr: Use s->num_frames instead of s->buf_ptr - s->buf

Martin Storsjö git at videolan.org
Sun Mar 1 01:00:59 CET 2015


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Fri Feb 27 12:30:54 2015 +0200| [12b3459979f5ea6481660cd2c99a0381e2b5ba37] | committer: Martin Storsjö

rtpenc_amr: Use s->num_frames instead of s->buf_ptr - s->buf

This doesn't fix any bug, but makes the code simpler for later
patches, and more straightforward to read as is.

Signed-off-by: Martin Storsjö <martin at martin.st>

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

 libavformat/rtpenc_amr.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/rtpenc_amr.c b/libavformat/rtpenc_amr.c
index 73da8c8..9f4f21a 100644
--- a/libavformat/rtpenc_amr.c
+++ b/libavformat/rtpenc_amr.c
@@ -36,7 +36,7 @@ void ff_rtp_send_amr(AVFormatContext *s1, const uint8_t *buff, int size)
 
     /* Test if the packet must be sent. */
     len = s->buf_ptr - s->buf;
-    if (s->num_frames == s->max_frames_per_packet || (len && len + size - 1 > s->max_payload_size)) {
+    if (s->num_frames == s->max_frames_per_packet || (s->num_frames && len + size - 1 > s->max_payload_size)) {
         int header_size = s->num_frames + 1;
         p = s->buf + max_header_toc_size - header_size;
         if (p != s->buf)



More information about the ffmpeg-cvslog mailing list