[FFmpeg-devel] [PATCH v5 1/2] avformat/riffdec: change declaration of ff_get_wav_header()

James Almer jamrial at gmail.com
Tue Mar 4 17:29:25 EET 2025


On 3/4/2025 12:28 PM, Michael Niedermayer wrote:
> On Tue, Mar 04, 2025 at 01:47:10PM +0100, Andreas Rheinhardt wrote:
>> Michael Niedermayer:
>>> On Mon, Feb 03, 2025 at 09:54:57AM +0530, Viraaj Raulgaonkar wrote:
>>>> Change the type of logctx from void* to AVFormatContext*. This is in
>>>> preparation for the next commit.
>>>> ---
>>>>   libavformat/riff.h    |  2 +-
>>>>   libavformat/riffdec.c | 14 +++++++-------
>>>>   2 files changed, 8 insertions(+), 8 deletions(-)
>>>
>>> will apply
>>>
>>
>> You applied only patch 1/2. This means the "next commit" mentioned in
>> the commit message is "swscale/aarch64: dotprod implementation of
>> rgba32_to_Y".
> 
> I didnt notice that
> 
> I guess, it would be better if patches dont refer to "next commit"
> but rather to what the commit does so the exact ordering of commits
> dont affect the commit messages
> 
> thx

"In a following commit" should be enough to make it valid regardless of 
how far in the future the referenced commit is.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250304/b48bed2c/attachment.sig>


More information about the ffmpeg-devel mailing list