Merge commit '1fc64e2e07787bbca82a72c146588e850e6d098a'
authorMichael Niedermayer <michaelni@gmx.at>
Sun, 1 Mar 2015 00:00:57 +0000 (01:00 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sun, 1 Mar 2015 00:01:01 +0000 (01:01 +0100)
* commit '1fc64e2e07787bbca82a72c146588e850e6d098a':
  rtpenc: Write conditional statements on separate lines

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavformat/rtpenc.c
libavformat/rtpenc_xiph.c

diff --combined libavformat/rtpenc.c
@@@ -2,20 -2,20 +2,20 @@@
   * RTP output format
   * Copyright (c) 2002 Fabrice Bellard
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -98,7 -98,7 +98,7 @@@ static int rtp_write_header(AVFormatCon
      }
      st = s1->streams[0];
      if (!is_supported(st->codec->codec_id)) {
 -        av_log(s1, AV_LOG_ERROR, "Unsupported codec %x\n", st->codec->codec_id);
 +        av_log(s1, AV_LOG_ERROR, "Unsupported codec %s\n", avcodec_get_name(st->codec->codec_id));
  
          return -1;
      }
          s->ssrc = av_get_random_seed();
      s->first_packet = 1;
      s->first_rtcp_ntp_time = ff_ntp_time();
 -    if (s1->start_time_realtime)
 +    if (s1->start_time_realtime != 0  &&  s1->start_time_realtime != AV_NOPTS_VALUE)
          /* Round the NTP time to whole milliseconds. */
          s->first_rtcp_ntp_time = (s1->start_time_realtime / 1000) * 1000 +
                                   NTP_OFFSET_US;
      // Pick a random sequence start number, but in the lower end of the
      // available range, so that any wraparound doesn't happen immediately.
      // (Immediate wraparound would be an issue for SRTP.)
 -    if (s->seq < 0)
 -        s->seq = av_get_random_seed() & 0x0fff;
 -    else
 +    if (s->seq < 0) {
 +        if (s1->flags & AVFMT_FLAG_BITEXACT) {
 +            s->seq = 0;
 +        } else
 +            s->seq = av_get_random_seed() & 0x0fff;
 +    } else
          s->seq &= 0xffff; // Use the given parameter, wrapped to the right interval
  
      if (s1->packet_size) {
          break;
      case AV_CODEC_ID_VORBIS:
      case AV_CODEC_ID_THEORA:
-         if (!s->max_frames_per_packet) s->max_frames_per_packet = 15;
+         if (!s->max_frames_per_packet)
+             s->max_frames_per_packet = 15;
          s->max_frames_per_packet = av_clip(s->max_frames_per_packet, 1, 15);
          s->max_payload_size -= 6; // ident+frag+tdt/vdt+pkt_num+pkt_length
          s->num_frames = 0;
@@@ -593,10 -591,6 +594,10 @@@ static int rtp_write_packet(AVFormatCon
              const uint8_t *mb_info =
                  av_packet_get_side_data(pkt, AV_PKT_DATA_H263_MB_INFO,
                                          &mb_info_size);
 +            if (!mb_info) {
 +                av_log(s1, AV_LOG_ERROR, "failed to allocate side data\n");
 +                return AVERROR(ENOMEM);
 +            }
              ff_rtp_send_h263_rfc2190(s1, pkt->data, size, mb_info, mb_info_size);
              break;
          }
@@@ -2,24 -2,23 +2,24 @@@
   * RTP packetization for Xiph audio and video
   * Copyright (c) 2010 Josh Allmann
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
 +#include "libavutil/avassert.h"
  #include "avformat.h"
  #include "rtpenc.h"
  
@@@ -73,7 -72,7 +73,7 @@@ void ff_rtp_send_xiph(AVFormatContext *
          uint8_t *ptr     = s->buf_ptr + 2 + size; // what we're going to write
          int remaining    = end_ptr - ptr;
  
 -        assert(s->num_frames <= s->max_frames_per_packet);
 +        av_assert1(s->num_frames <= s->max_frames_per_packet);
          if ((s->num_frames > 0 && remaining < 0) ||
              s->num_frames == s->max_frames_per_packet) {
              // send previous packets now; no room for new data
          }
  
          // buffer current frame to send later
-         if (0 == s->num_frames) s->timestamp = s->cur_timestamp;
+         if (0 == s->num_frames)
+             s->timestamp = s->cur_timestamp;
          s->num_frames++;
  
          // Set packet header. Normally, this is OR'd with frag and xdt,
          // but those are zero, so omitted here
          *q++ = s->num_frames;
  
-         if (s->num_frames > 1) q = s->buf_ptr; // jump ahead if needed
+         if (s->num_frames > 1)
+             q = s->buf_ptr; // jump ahead if needed
          *q++ = (size >> 8) & 0xff;
          *q++ = size & 0xff;
          memcpy(q, buff, size);