[FFmpeg-devel] Pretty printing patch

Roman V. Shaposhnik rvs
Sun Oct 12 00:42:40 CEST 2008


On Wed, 2008-10-08 at 20:04 +0200, Michael Niedermayer wrote:
> > -                run=   new_dv_vlc_run[code] + 1;
> > -                level= new_dv_vlc_level[code];
> > +                run   = new_dv_vlc_run[code] + 1;
> > +                level = new_dv_vlc_level[code];
> 
> i think 
> run   = new_dv_vlc_run  [code] + 1;
> level = new_dv_vlc_level[code];
> 
> would be even better

Good point.

> > -            s->dv_zigzag[1][i] = dsp.idct_permutation[(j&7) + (j&8)*4 + (j&48)/2];
> > +            s->dv_zigzag[1][i] = dsp.idct_permutation[(j & 7) + (j & 8) * 4 + (j & 48) / 2];
> 
> IMHO the previous was more readable, but of course its your code and
> your decission on how you want it to look.

To tell you the truth -- as far as these changes go I'm on the fence
myself. But it sure helps to know your opinion.

Diego, what's your take? It is your patch after all -- I was merely
reapplying it on top of my changes and submitting it for review ;-)

Thanks,
Roman.





More information about the ffmpeg-devel mailing list