[FFmpeg-devel] [PATCH 1/4] cabac-test: Return 1 if there are any errors

Michael Niedermayer michaelni at gmx.at
Fri Apr 10 21:52:04 CEST 2015


On Fri, Apr 10, 2015 at 03:36:29PM -0300, James Almer wrote:
> On 10/04/15 2:27 PM, Timothy Gu wrote:
> > ---
> >  libavcodec/cabac.c | 16 +++++++++++-----
> >  1 file changed, 11 insertions(+), 5 deletions(-)
> > 
> > diff --git a/libavcodec/cabac.c b/libavcodec/cabac.c
> > index 81a75dd..c755d7f 100644
> > --- a/libavcodec/cabac.c
> > +++ b/libavcodec/cabac.c
> > @@ -174,7 +174,7 @@ int main(void){
> >      CABACContext c;
> >      uint8_t b[9*SIZE];
> >      uint8_t r[9*SIZE];
> > -    int i;
> > +    int i, ret = 0;
> >      uint8_t state[10]= {0};
> >      AVLFG prng;
> >  
> > @@ -207,21 +207,27 @@ STOP_TIMER("put_cabac")
> >  
> >      for(i=0; i<SIZE; i++){
> >  START_TIMER
> 
> Unrelated to this patch, but why are these used here?

probably they where added for benchmarking and then forgotten

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150410/e8b0236d/attachment.asc>


More information about the ffmpeg-devel mailing list