[FFmpeg-devel] [PATCH 1/2] libavcodec/pgxdec: Add PGX decoder

[Gautam Ramakrishnan]

On Sat, Jun 27, 2020 at 7:58 PM Nicolas George <george at nsup.org> wrote:
>
> Michael Niedermayer (12020-06-27):
> > this could overflow
>
> The original code had a test to limit the size of numbers to 9 digits,
> guaranteeing no overflow. I wonder where it went.
>
I used 9 digits because I was using a fixed size string to store the number
and then used strol. Now as I removed the usage of strol, it wasnt necessary.
Do you thing using a loop of fixed iterations is a good idea?  Is it better if
I use the for loop again?



-- 
-------------
Gautam |