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".