diff mbox

[FFmpeg-devel,RFC] Changelog: Compress slightly

Message ID 201609300008.47044.cehoyos@ag.or.at
State Accepted
Headers show

Commit Message

Carl Eugen Hoyos Sept. 29, 2016, 10:08 p.m. UTC
Hi!

I consider the Changelog more readable with attached patch.

I wonder if the ogv line should be dropped...

Please comment, Carl Eugen
From 01b62bd9f7da4c991144dfa4039f168d20612f06 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <cehoyos@ag.or.at>
Date: Fri, 30 Sep 2016 00:06:12 +0200
Subject: [PATCH] Changelog: Compress slightly to improve readability.

---
 Changelog |   12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

Comments

James Almer Sept. 30, 2016, 12:13 a.m. UTC | #1
On 9/29/2016 7:08 PM, Carl Eugen Hoyos wrote:
> Hi!
> 
> I consider the Changelog more readable with attached patch.
> 
> I wonder if the ogv line should be dropped...

No, the alias muxer reports a different mimetype than when .ogv is mapped to
the Ogg Muxer and has different defaults (VP8 when libtheora isn't installed).
It's worth mentioning.

> 
> Please comment, Carl Eugen
> 
> 
> 0001-Changelog-Compress-slightly-to-improve-readability.patch
> 
> 
> From 01b62bd9f7da4c991144dfa4039f168d20612f06 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <cehoyos@ag.or.at>
> Date: Fri, 30 Sep 2016 00:06:12 +0200
> Subject: [PATCH] Changelog: Compress slightly to improve readability.
> 
> ---
>  Changelog |   12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
> 
> diff --git a/Changelog b/Changelog
> index d7f5dc9..ce186fc 100644
> --- a/Changelog
> +++ b/Changelog
> @@ -8,10 +8,9 @@ version <next>:
>  - Alias muxer for Ogg Video (.ogv)
>  - VP8 in Ogg muxing
>  - curves filter doesn't automatically insert points at x=0 and x=1 anymore
> -- 16-bit support in curves filter
> -- 16-bit support in selectivecolor filter
> +- 16-bit support in curves filter and selectivecolor filter
>  - OpenH264 decoder wrapper
> -- MediaCodec hwaccel
> +- MediaCodec H.264 and HEVC hwaccel
>  - True Audio (TTA) muxer
>  - crystalizer audio filter
>  - acrusher audio filter
> @@ -28,12 +27,9 @@ version <next>:
>  - gblur filter
>  - avgblur filter
>  - sobel and prewitt filter
> -- MediaCodec HEVC decoding

This release added MediaCodec h264 and hevc hwaccels as mentioned above, but
also a hevc decoder, so removing this line is wrong.

> -- TrueHD encoder
> -- Meridian Lossless Packing (MLP) encoder
> +- Meridian Lossless Packing (MLP) / TrueHD encoder
>  - Non-Local Means (nlmeans) denoising filter
> -- sdl2 output device
> -- sdl2 support for ffplay
> +- sdl2 output device and ffplay support
>  - sdl1 output device and sdl1 support removed
>  - extended mov edit list support
>  

LGTM otherwise.
Carl Eugen Hoyos Oct. 2, 2016, 1:30 a.m. UTC | #2
2016-09-30 2:13 GMT+02:00 James Almer <jamrial@gmail.com>:

> LGTM otherwise.

Applied with the requested change.

Thank you, Carl Eugen
diff mbox

Patch

diff --git a/Changelog b/Changelog
index d7f5dc9..ce186fc 100644
--- a/Changelog
+++ b/Changelog
@@ -8,10 +8,9 @@  version <next>:
 - Alias muxer for Ogg Video (.ogv)
 - VP8 in Ogg muxing
 - curves filter doesn't automatically insert points at x=0 and x=1 anymore
-- 16-bit support in curves filter
-- 16-bit support in selectivecolor filter
+- 16-bit support in curves filter and selectivecolor filter
 - OpenH264 decoder wrapper
-- MediaCodec hwaccel
+- MediaCodec H.264 and HEVC hwaccel
 - True Audio (TTA) muxer
 - crystalizer audio filter
 - acrusher audio filter
@@ -28,12 +27,9 @@  version <next>:
 - gblur filter
 - avgblur filter
 - sobel and prewitt filter
-- MediaCodec HEVC decoding
-- TrueHD encoder
-- Meridian Lossless Packing (MLP) encoder
+- Meridian Lossless Packing (MLP) / TrueHD encoder
 - Non-Local Means (nlmeans) denoising filter
-- sdl2 output device
-- sdl2 support for ffplay
+- sdl2 output device and ffplay support
 - sdl1 output device and sdl1 support removed
 - extended mov edit list support