On Sun, Apr 05, 2020 at 10:28:17 +0530, gautamr...@gmail.com wrote:
> +    tile->has_ppt = 1;  // this tile has a ppt marker
> +/*    Zppt = */ bytestream2_get_byte(&s->g); // Zppt is skipped and not used

I don't know what others think, but, as mentioned in my first review, I
really dislike this style, because it gives the impression that this
line is commented out, when just skimming over the code.

How about just:
    bytestream2_get_byte(&s->g); // skip Zppt
?

Thanks,
Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to