Hi,
On Wed, Dec 21, 2011 at 11:59 AM, Ronald S. Bultje <[email protected]> wrote:
> On Wed, Dec 21, 2011 at 11:48 AM, Ronald S. Bultje <[email protected]> wrote:
> [review for line 537-1037].
[ and the rest]
> @@ -1125,27 +1246,34 @@ void ff_er_frame_end(MpegEncContext *s){
[..]
> - if(s->pict_type!=AV_PICTURE_TYPE_B &&
> (error&(ER_DC_ERROR|ER_MV_ERROR|ER_AC_ERROR))){
> - s->mbskip_table[mb_xy]=0;
> + if (s->pict_type != AV_PICTURE_TYPE_B &&
> + (error & (ER_DC_ERROR | ER_MV_ERROR | ER_AC_ERROR))) {
> + s->mbskip_table[mb_xy] = 0;
Make sure the "(" on the second line aligns with the "s" vertically of
the first line, i.e.:
if (s->... &&
(error & (...))) {
...
}
Rest looks OK.
Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
