[FFmpeg-devel] GSoC Weely report (libswscale)
Michael Niedermayer
michael at niedermayer.cc
Sun Jul 12 04:51:38 CEST 2015
On Sat, Jul 11, 2015 at 04:57:22PM -0300, Pedro Arthur wrote:
> Here is the full patch rebased including all previous changes.
if you build ffmpeg with --assert-level=2
fails fate
for example in fate-vsynth1-cinepak
its probably best to always build with --assert-level=2
so you wont miss such failures
--- ./tests/ref/vsynth/vsynth1-cinepak 2015-07-11 12:25:01.268257903 +0200
+++ tests/data/fate/vsynth1-cinepak 2015-07-12 04:11:06.041453781 +0200
@@ -1,4 +0,0 @@
-546c7c1069f9e418aa787f469b693b94 *tests/data/fate/vsynth1-cinepak.mov
-99465 tests/data/fate/vsynth1-cinepak.mov
-bee091c200262be3427a233a2812388c *tests/data/fate/vsynth1-cinepak.out.rawvideo
-stddev: 8.46 PSNR: 29.58 MAXDIFF: 105 bytes: 7603200/ 456192
Assertion lumSrcPtr + vLumFilterSize - 1 < (const int16_t **)lumPixBuf + vLumBufSize * 2 failed at libswscale/swscale.c:694
there are also many other fate tests which fail with --assert-level=2
also it might make more sense to stash all these commits together
for submission,
if you like you could keep them seperate in your branch (could make
sense for debug/bisect)
but the commits showing the step wise development are rather hard to
review and would be impossible to push to master git
commits for master git should not introduce known issues that are
fixed in later commits and should be split in selfcontained changesets
a single stashed together patch should be fine and easy to generate
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150712/4fe73a68/attachment.sig>
More information about the ffmpeg-devel
mailing list