[FFmpeg-devel] Patch

Vlad Kuznetsov vlad at spreecast.com
Tue Jul 1 19:37:12 CEST 2014


Hi,
quick question about test case: do you have any requirements for tests? Is
it necessary?
Right now I have just manual testing of this functional.


On Sun, Jun 29, 2014 at 3:06 PM, Michael Niedermayer <michaelni at gmx.at>
wrote:

> On Fri, Jun 27, 2014 at 01:01:19PM -0700, Vlad Kuznetsov wrote:
> > This patch contains changes for progressive HLS
>
> please use git format-patch or get send-email and dont
> double encode patches
>
> also add a commit message and do not mix reindentions with functional
> changes, such changes make review harder
>
> also please avoid mixing declarations and code, these are not allowed
> in C90 and cause problems with some compilers
>
> how can the new code be tested, do you have a testcase ?
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> The greatest way to live with honor in this world is to be what we pretend
> to be. -- Socrates
>
>
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>


More information about the ffmpeg-devel mailing list