Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Michael Niedermayer
On Sat, May 02, 2015 at 01:15:09AM +0200, Jerome Martinez wrote: > Le 02/05/2015 01:05, Michael Niedermayer a écrit : > >this looks wrong in my lyx, quite wrong, certainly doesnt result > >in a closing bracket > > Weird, it is OK on my side and actually a pretty patch. > I rebased following your l

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Jerome Martinez
Le 02/05/2015 01:05, Michael Niedermayer a écrit : this looks wrong in my lyx, quite wrong, certainly doesnt result in a closing bracket Weird, it is OK on my side and actually a pretty patch. I rebased following your latest commits, attached is a new patch. note, that is when i apply the pat

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Michael Niedermayer
On Sat, May 02, 2015 at 12:37:19AM +0200, Jerome Martinez wrote: > Le 02/05/2015 00:21, Michael Niedermayer a écrit : > >why the extra space ? > > Because of a guy doing stupid mistakes. > (this guy may try to blame Lyx but it is difficult to be believable) > > New patch attached. > ffv1.lyx |

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Jerome Martinez
Le 02/05/2015 00:21, Michael Niedermayer a écrit : why the extra space ? Because of a guy doing stupid mistakes. (this guy may try to blame Lyx but it is difficult to be believable) New patch attached. >From 40a211a344c3f7bcb8bc627bd5edcb4cb8b1c56c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Michael Niedermayer
On Fri, May 01, 2015 at 02:27:40PM +0200, Jerome Martinez wrote: > Le 01/05/2015 14:07, Michael Niedermayer a écrit : > >>-quant_tables[i][j][128] =-quant_tables[i][j][127] > >>+quant_tables[i][j][128] = -quant_tables[i][j][127] > >unrelated > > Definitely. I apologize for such mistake (should be

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Jerome Martinez
Le 01/05/2015 14:07, Michael Niedermayer a écrit : -quant_tables[i][j][128] =-quant_tables[i][j][127] +quant_tables[i][j][128] = -quant_tables[i][j][127] unrelated Definitely. I apologize for such mistake (should be in PATCH 1/4, it is now). New patch attached. >From 7fc4e973a2fede34b8cc3ca

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Michael Niedermayer
On Fri, May 01, 2015 at 01:55:58PM +0200, Jerome Martinez wrote: > Le 01/05/2015 13:50, Michael Niedermayer a écrit : > >already applied years ago > > I don't understand: the {} are still missing in GitHub FFmpeg/FFV1 master. > I forgot to attach the patch. Attached now. yes i applied the empty p

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Jerome Martinez
Le 01/05/2015 13:50, Michael Niedermayer a écrit : already applied years ago I don't understand: the {} are still missing in GitHub FFmpeg/FFV1 master. I forgot to attach the patch. Attached now. >From ca4b5c8ee5064ee0f71586ad70d426acbb46e914 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9r=C3

Re: [FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Michael Niedermayer
On Fri, May 01, 2015 at 12:13:53PM +0200, Jerome Martinez wrote: already applied years ago [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No human being will ever know the Truth, for even if they happen to say it by chance, they would not even known they had do

[FFmpeg-devel] [PATCH 2/4] FFV1 specification: Add missing { }

2015-05-01 Thread Jerome Martinez
___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel