On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: > On 12/28/2015 02:03 PM, Michael Niedermayer wrote: > >On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: > >>On 12/28/2015 10:34 AM, Michael Niedermayer wrote: > >>>On Mon, Dec 28, 2015 at 03:10:18AM +0000, Carl Eugen Hoyos wrote: > >>>>Michael Niedermayer <michael <at> niedermayer.cc> writes: > >>>> > >>>>>Patch splited in move and matroska part > >>>>>i removed this memcpy() for now from what i > >>>>>commited as there is clearly no consenus on it > >>>> > >>>>I would really have appreciated a real review: > >>>>Apart from the unrelated audio fix > >>> > >>>what audio fix ? > >>> > >> > >>He probably means the change from 86 to 36 as the lower limit for > >>private data in A_QUICKTIME (36 is the size of a version 0 sound > >>sample description). > > > >right, i missed that > >that one should have been a seperate patch > > > >[...] > > > > > > > >_______________________________________________ > >ffmpeg-devel mailing list > >ffmpeg-devel@ffmpeg.org > >http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > Well, the proper value, as I said, is 36 rather than 86. Should we > redo that part, or is it OK to keep it in the patchset?
its already in git master so it cannot be truly redone we could revert the single line and recommit it with a more explanatory commit message, this wouldnt remove the previous commit but would add 2 more commits. i suggest we wait for carls oppinion on this, as he was the one complaining about this. [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB DNS cache poisoning attacks, popular search engine, Google internet authority dont be evil, please
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
