Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-19 Thread Moritz Barsnick
On Mon, Jun 19, 2017 at 14:32:38 +0200, Hein-Pieter van Braam wrote: > > > +if (! opcode) { > > > > Style: You should drop the space. > > You mean the space between the ! and opcode? The space between if and > opening parenthesis should stay, right? Correct. The handling of paranthes

Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-19 Thread Hein-Pieter van Braam
I've resubmitted the patch in a more reviewable format here: http://ffmpeg.org/pipermail/ffmpeg-devel/2017-June/212532.html having said that: On Mon, 2017-06-19 at 14:22 +0200, Moritz Barsnick wrote: > On Sat, Jun 17, 2017 at 19:13:52 +0200, Hein-Pieter van Braam wrote: > > +s->pixel_

Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-19 Thread Moritz Barsnick
On Sat, Jun 17, 2017 at 19:13:52 +0200, Hein-Pieter van Braam wrote: > +s->pixel_ptr = frame->data[0] + x + y*frame->linesize[0]; Style: You might want to add spaces around '*'. > +if (! opcode) { Style: You should drop the space. > +if (opcode < 0) { > +

Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-17 Thread Hein-Pieter van Braam
On Sat, 2017-06-17 at 18:45 +0200, Paul B Mahol wrote: > Have you made sure that new code does not cause overreads? > > Use bytestream2* for that. > I have made that change, thanks, the code actually looks better now too! Attached is a new version of the patch. Thanks! - HPFrom e8a45ce4748e39

Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-17 Thread Paul B Mahol
On 6/17/17, Hein-Pieter van Braam wrote: > Hi all, > > I seemed to have done something wrong when fixing the patchcheck > problems originally. Apparently I didn't build between making my > 'fixes' and creating the patch. The previous patch doesn't compile. > > I fixed this in this version, my apol

Re: [FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-17 Thread Hein-Pieter van Braam
Hi all, I seemed to have done something wrong when fixing the patchcheck problems originally. Apparently I didn't build between making my 'fixes' and creating the patch. The previous patch doesn't compile. I fixed this in this version, my apologies for the noise. I've also fixed reading garbage

[FFmpeg-devel] [RFC] Implement support for interplay MVE 0x06, 0x0F and 0x10

2017-06-17 Thread Hein-Pieter van Braam
Hi all, This patch implements support for 3 previously unknown Interplay MVE opcodes. These opcodes together implement support for 2 additional video frame formats. This is my first time trying to contribute to ffmpeg so I expect this code to not be entirely up to snuff, I'm interested in getting