[FFmpeg-devel] remove int readers

Diego Biurrun diego
Fri Jul 6 11:33:06 CEST 2007


On Sat, Jun 23, 2007 at 06:16:44PM +0200, Michael Niedermayer wrote:
> 
> On Sat, Jun 23, 2007 at 09:32:12AM -0400, Ronald S. Bultje wrote:
> > 
> > On 6/23/07, Michael Niedermayer <michaelni at gmx.at> wrote:
> > >
> > >> @@ -694,15 +694,9 @@
> > >>
> > >>  static int64_t get_pts(const uint8_t *p)
> > >>  {
> > >> -    int64_t pts;
> > >> -    int val;
> > >> -
> > >> -    pts = (int64_t)((p[0] >> 1) & 0x07) << 30;
> > >> -    val = (p[1] << 8) | p[2];
> > >> -    pts |= (int64_t)(val >> 1) << 15;
> > >> -    val = (p[3] << 8) | p[4];
> > >> -    pts |= (int64_t)(val >> 1);
> > >> -    return pts;
> > >> +    return ((int64_t)((p[0] >> 1) & 0x07)   << 30) |
> > >> +                     ((AV_RB16(p + 1) >> 1) << 15) |
> > >> +                      (AV_RB16(p + 3) >> 1);
> > >>  }
> > >
> > >i see
> > 
> > I changed it to be more like above again (and removed the applied parts).
> 
> iam fine with the patch (assuming none of the maintainers of the changed files
> has objections)

No objections were heard and I don't expect any, patch applied.

Diego




More information about the ffmpeg-devel mailing list