Re: [FFmpeg-devel] [PATCH] avformat/aea: Add aea muxer

From: Stefano Sabatini <stefasab@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: asivery <asivery@protonmail.com>
Subject: Re: [FFmpeg-devel] [PATCH] avformat/aea: Add aea muxer
Date: Sun, 10 Mar 2024 16:10:15 +0100
Message-ID: <Ze3NVxIm1YyFdDPf@mariano> (raw)
In-Reply-To: <3gq1Ftm_y1013kbc9iq5cZU-QLIOOTvaG6SwFuPWF004eJJMe_ci7aUvyCPHlENileu1rUtuCGWC2BGouRMthXOo7QmJ_-ZBJgrFX_7tAaY=@protonmail.com>

On date Sunday 2024-03-10 14:20:12 +0000, ffmpeg-devel Mailing List wrote:
> Great, thank you very much!
> I'm attaching the (hopefully) final version of the patch.
> 

> MD studio was a piece of software created by a company called "EDL",
> which in combination with alternative firmware for the Sony MDH-10
> MiniDisc recorder
> (https://www.minidisc.wiki/equipment/sony/misc/mdh-10) let people
> download raw ATRAC1 audio from their MiniDiscs onto computers, and
> (probably) put this audio back on their discs later.  Nowadays it's
> used by software like Web Minidisc Pro (which I maintain) to store
> ATRAC1 audio, as it (until recently) was the only way to store
> ATRAC1 so that it would be picked up by VLC. Now we have matroska
> support for ATRAC1, but I wanted to write a muxer for AEA before I
> phase it out from new pieces of software, so that the people who
> would like to use AEA instead of matroska have a way to go back to
> it.

Thanks, probably part of this information could go to the doc to
provide some context (can be done as a separate patch), as this kind
of info is very difficult to retrieve otherwise (and one of the goals
of FFmpeg is digital preservation).

> From e67d2df52c65528fbbfe8d5268661c88a7ad225e Mon Sep 17 00:00:00 2001
> From: asivery <asivery@protonmail.com>
> Date: Fri, 8 Mar 2024 14:45:02 +0100
> Subject: [PATCH] avformat/aea: Add aea muxer
> 
> Signed-off-by: asivery <asivery@protonmail.com>
> ---
>  Changelog                       |   1 +
>  doc/muxers.texi                 |  10 +++
>  libavformat/Makefile            |   3 +-
>  libavformat/{aea.c => aeadec.c} |   0
>  libavformat/aeaenc.c            | 115 ++++++++++++++++++++++++++++++++
>  libavformat/allformats.c        |   1 +
>  6 files changed, 129 insertions(+), 1 deletion(-)
>  rename libavformat/{aea.c => aeadec.c} (100%)
>  create mode 100644 libavformat/aeaenc.c

Patch looks good to me, waiting for more feedback by Andreas.

Thanks.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".