2016-09-04 15:42 GMT+02:00 Hendrik Leppkes <h.lepp...@gmail.com>: > On Sun, Sep 4, 2016 at 1:18 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >> 2016-09-04 13:10 GMT+02:00 Paul B Mahol <one...@gmail.com>: >>> On 9/4/16, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >>>> 2016-09-03 15:25 GMT+02:00 Timo Rothenpieler <t...@rothenpieler.org>: >>>> >>>>> + output_pixel(tdstY++, (t | (t << 8)) & 0xFFC0); >>>> >>>> Please remove the "& 0xFFC0" here and below. >>> >>> Please explain your reasoning here. >> >> The linked specification clearly explains that the content >> of P016 and P010 is identical and that it is not necessary >> to set the lsb's to 0. > > While the specification does say that P010's memory layout > was particularly chosen to be able to simply alias to P016 if > needed, having random garbage in there is not good, so > the zeroing of those bits should remain.
Since it is not garbage (and not random), your argumentation makes no sense. Is the operation really free as you all seem to insinuate? Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
