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

2016-10-26 Thread Clément Bœsch 
On Fri, Oct 21, 2016 at 10:32:58AM +0200, Michael Behrisch wrote:
> 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 fine and semantically more correct (it
> > explicits that it's wrong to add entry afterward).
> > 
> >> I am sure there are better uses of contributors' time than that.
> >>
> > 
> > people are free to do whatever they feel like; you're just wasting yours
> > by bikeshedding about this stuff you could just ignore.
> > 
> 
> So here is the patch again, this time without the line breaks, hopefully.
> 
> Best regards,
> Michael
> 

> Date: Thu, 20 Oct 2016 08:48:25 +0200
> From: Michael Behrisch 
> To: ffmpeg-devel@ffmpeg.org
> Cc: Michael Behrisch 
> Subject: [PATCH] removing comma at final enumeration items to fix pedantic
>  warnings
> X-Mailer: git-send-email 2.6.6
> 

Reworded the commit message and applied.

thanks

-- 
Clément B.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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 really a trivial and harmless patch.
>>> 
>>> Which is not enough to accept it.
>>> 
>> 
>> but the patch is perfectly fine and semantically more correct (it 
>> explicits that it's wrong to add entry afterward).
>> 
>>> I am sure there are better uses of contributors' time than that.
>>> 
>> 
>> people are free to do whatever they feel like; you're just wasting
>> yours by bikeshedding about this stuff you could just ignore.
>> 
> 
> So here is the patch again, this time without the line breaks,
> hopefully.

does the lack of further reaction mean that the patch was rejected or
does the discussion continue elsewhere?

Best regards,
Michael




signature.asc
Description: OpenPGP digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-20 Thread Carl Eugen Hoyos 
2016-10-20 9:55 GMT+02:00 Nicolas George :

> I am sure there are better uses of contributors' time than that.

After many years on this list, I don't think this is a helpful approach.

Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-20 Thread 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 fine and semantically more correct (it
explicits that it's wrong to add entry afterward).

> I am sure there are better uses of contributors' time than that.
> 

people are free to do whatever they feel like; you're just wasting yours
by bikeshedding about this stuff you could just ignore.

-- 
Clément B.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-20 Thread Nicolas George 
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.

I am sure there are better uses of contributors' time than that.

Regards,

-- 
  Nicolas George


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-20 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(-)
> 
> If you have difficulty to send your patch directly (you should use git
> send-email, not just copy paste), you may send it as an attachment.

Sorry about that, still struggling a little with git. I did not know how
to insert the signed off when using send-email.

I hope the new patch (in the separate mail) is fine.

Best regards,
Michael




signature.asc
Description: OpenPGP digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-19 Thread 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(-)

If you have difficulty to send your patch directly (you should use git
send-email, not just copy paste), you may send it as an attachment.

Thank's
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-10-19 Thread Michael Niedermayer 
On Wed, Oct 19, 2016 at 11:11:21PM +0200, Michael Behrisch wrote:
> 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 @@ typedef enum {
>  AV_CLASS_CATEGORY_DEVICE_AUDIO_INPUT,
>  AV_CLASS_CATEGORY_DEVICE_OUTPUT,
>  AV_CLASS_CATEGORY_DEVICE_INPUT,
> -AV_CLASS_CATEGORY_NB, ///< not part of ABI/API
> +AV_CLASS_CATEGORY_NB  ///< not part of ABI/API
>  }AVClassCategory;
> 
>  #define AV_IS_INPUT_DEVICE(category) \
> diff --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h
> index b15c0ef..7a3f68b 100644
> --- a/libavutil/pixfmt.h
> +++ b/libavutil/pixfmt.h
> @@ -306,7 +306,7 @@ enum AVPixelFormat {
> 
>  AV_PIX_FMT_MEDIACODEC, ///< hardware decoding through MediaCodec
> 
> -AV_PIX_FMT_NB,///< number of pixel formats, DO NOT USE THIS
> if you want to link with shared libav* because the number of formats
> might differ between versions
> +AV_PIX_FMT_NB ///< number of pixel formats, DO NOT USE THIS
> if you want to link with shared libav* because the number of formats
> might differ between versions
>  };

patch is corrupted by newlines / word wrap

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel