[FFmpeg-devel] comma at the end of enumerator lists

2016-10-16 Thread Michael Behrisch
Hi all, first of all thanks for providing this great library. Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been rejected which tried to remove the comma at the end of enumerator lists which triggers a warning when compiling with gcc and -Wpedantic. I still think it could be

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-16 Thread Michael Behrisch
Am 16.10.2016 um 23:24 schrieb Clément Bœsch: > On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: >> Hi all, >> first of all thanks for providing this great library. >> >> Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been >> r

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread Michael Behrisch
Am 17.10.2016 um 15:29 schrieb Michael Niedermayer: > On Mon, Oct 17, 2016 at 01:34:55PM +0200, wm4 wrote: >> On Mon, 17 Oct 2016 13:09:36 +0200 >> Michael Niedermayer wrote: >> >>> On Mon, Oct 17, 2016 at 10:07:42AM +0200, Nicolas George wrote: Le sextidi 26 vendémiaire, an CCXXV, Michael Ni

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread Michael Behrisch
Hi Ronald, Am 17.10.2016 um 21:37 schrieb Ronald S. Bultje: > Hi Michael, > > On Mon, Oct 17, 2016 at 3:16 PM, Michael Behrisch > wrote: > >> Am 17.10.2016 um 15:29 schrieb Michael Niedermayer: >>> On Mon, Oct 17, 2016 at 01:34:55PM +0200, wm4 wrote: >>&

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread Michael Behrisch
Hi Ronald, Am 17.10.2016 um 23:45 schrieb Ronald S. Bultje: > Hi Michael, > > On Mon, Oct 17, 2016 at 5:23 PM, Michael Behrisch wrote: > >> Hi Ronald, >> >> Am 17.10.2016 um 21:37 schrieb Ronald S. Bultje: >>> Hi Michael, >>> >>> On

[FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-19 Thread Michael Behrisch
Signed-off-by: Michael Behrisch --- libavutil/log.h| 2 +- libavutil/pixfmt.h | 12 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/libavutil/log.h b/libavutil/log.h index 0acc1b9..f0a5738 100644 --- a/libavutil/log.h +++ b/libavutil/log.h @@ -44,7 +44,7

Re: [FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-19 Thread Michael Behrisch
Am 20.10.2016 um 04:18 schrieb Muhammad Faiz: > On 10/20/16, Michael Behrisch wrote: >> Signed-off-by: Michael Behrisch >> --- >> libavutil/log.h| 2 +- >> libavutil/pixfmt.h | 12 ++-- >> 2 files changed, 7 insertions(+), 7 deletions(-) > &

Re: [FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-21 Thread Michael Behrisch
Am 20.10.2016 um 10:00 schrieb Clément Bœsch: > On Thu, Oct 20, 2016 at 09:55:17AM +0200, Nicolas George wrote: >> Le nonidi 29 vendémiaire, an CCXXV, Clement Boesch a écrit : >>> it's really a trivial and harmless patch. >> >> Which is not enough to accept it. >> > > but the patch is perfectly fi

Re: [FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-25 Thread Michael Behrisch
Hi all, Am 21.10.2016 um 10:32 schrieb Michael Behrisch: > Am 20.10.2016 um 10:00 schrieb Clément Bœsch: >> On Thu, Oct 20, 2016 at 09:55:17AM +0200, Nicolas George wrote: >>> Le nonidi 29 vendémiaire, an CCXXV, Clement Boesch a écrit : >>>> it's