Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-08 Thread u-9iep
On Tue, Feb 07, 2017 at 10:50:42PM +0100, Michael Niedermayer wrote: > There is no "Rl" in there > > git removes that when applying with git am > git am > ... > From: Rl > ... > git show > ... > Author: addr-see-the-webs...@aetey.se

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-07 Thread Michael Niedermayer
On Tue, Feb 07, 2017 at 12:54:02PM +0100, u-9...@aetey.se wrote: > Hello Michael, > > On Tue, Feb 07, 2017 at 03:09:56AM +0100, Michael Niedermayer wrote: > > > What about the other patches from that series? > > > > > The second one is purely cosmetic (looks useful to me but not much > > > of

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-07 Thread u-9iep
Hello Michael, On Tue, Feb 07, 2017 at 03:09:56AM +0100, Michael Niedermayer wrote: > > What about the other patches from that series? > > > The second one is purely cosmetic (looks useful to me but not much > > of concern) > > I have no oppinion about // vs /**/ comments but in the absence of

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-06 Thread Michael Niedermayer
On Mon, Feb 06, 2017 at 10:51:30AM +0100, u-9...@aetey.se wrote: > On Thu, Feb 02, 2017 at 05:19:16PM +0100, Michael Niedermayer wrote: > > On Thu, Feb 02, 2017 at 04:22:28PM +0100, u-9...@aetey.se wrote: > > > On Thu, Feb 02, 2017 at 01:31:00PM +0100, Michael Niedermayer wrote: > > patch applied

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-06 Thread u-9iep
On Thu, Feb 02, 2017 at 05:19:16PM +0100, Michael Niedermayer wrote: > On Thu, Feb 02, 2017 at 04:22:28PM +0100, u-9...@aetey.se wrote: > > On Thu, Feb 02, 2017 at 01:31:00PM +0100, Michael Niedermayer wrote: > patch applied > > thx Thanks Michael. What about the other patches from that series?

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-02 Thread Michael Niedermayer
On Thu, Feb 02, 2017 at 04:22:28PM +0100, u-9...@aetey.se wrote: > On Thu, Feb 02, 2017 at 01:31:00PM +0100, Michael Niedermayer wrote: > > > From: Rl > > > > Is it intended that your name is not in the Author field for git ? > > This is coherent with my earlier

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-02 Thread u-9iep
On Thu, Feb 02, 2017 at 01:31:00PM +0100, Michael Niedermayer wrote: > > From: Rl > > Is it intended that your name is not in the Author field for git ? This is coherent with my earlier patches (from 2014). Yes. Regards, Rune

Re: [FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-02-02 Thread Michael Niedermayer
On Sun, Jan 29, 2017 at 06:42:05PM +0100, u-9...@aetey.se wrote: > Attaching the new version. > > Regards, > Rune > cinepakenc.c |9 +++-- > 1 file changed, 3 insertions(+), 6 deletions(-) > 4be80f9f1f893468da6109b3ece89f7e60343671 >

[FFmpeg-devel] [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents)

2017-01-29 Thread u-9iep
Attaching the new version. Regards, Rune >From f563e57f7609cd890b655cb9d140849f0917a6d3 Mon Sep 17 00:00:00 2001 From: Rl Date: Sun, 29 Jan 2017 16:40:27 +0100 Subject: [PATCH 1/3] libavcodec/cinepakenc.c: comments cleanup (contents) Change the encoding of the