[FFmpeg-devel] [PATCH][VAAPI][2/6] Add common data structures and helpers (take 9)
Diego Biurrun
diego
Thu Mar 12 12:31:34 CET 2009
On Thu, Mar 12, 2009 at 06:09:08AM +0100, Gwenole Beauchesne wrote:
> Le 12 mars 09 ? 01:27, Michael Niedermayer a ?crit :
>
> >> + slice_param->slice_data_size = size;
> >> + slice_param->slice_data_offset = p->slice_data_size;
> >> + slice_param->slice_data_flag = VA_SLICE_DATA_FLAG_ALL;
> >
> > extra useless whitespace
>
> Oh, it's great, you are now focusing on cosmetics, so this means the
> rest is correct!
There is a pattern emerging here: You get a multitude of comments, some
of them minor in nature, you use the minor comments as a pretext to
flame. However, the time you waste flaming exceeds the time it takes
to adjust those minor nits.
To sum it up: We've heard it all before, we already didn't care the
first time around.
Diego
More information about the ffmpeg-devel
mailing list