[x264-devel] unnecessary code or bug in x264_slicetype_analyse?

[chen]

Hello,


I found there some funny code in the function x264_slicetype_analyse.
For example,


...
    else if( num_frames == 0 )
    {
        frames[1]->i_type = X264_TYPE_I;
        return;
    }
...
    /* Restore frametypes for all frames that haven't actually been decided yet. */
    for( int j = reset_start; j <= num_frames; j++ )
        frames[j]->i_type = frames[j]->i_forced_type;
...


In here, the frames[] is temporary Stack allocated array variant, why we setting it's value before return?


Regards,
Min Chen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x264-devel/attachments/20211117/d5bc4db3/attachment.html>