[FFmpeg-devel] [PATCH 01/21] avutil/avassert: Add av_unreachable and av_assume() macros

[James Almer]

On 5/7/2025 2:57 PM, softworkz . wrote:
> 
> 
>> -----Original Message-----
>> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Andreas
>> Rheinhardt
>> Sent: Mittwoch, 7. Mai 2025 13:59
>> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
>> Subject: [FFmpeg-devel] [PATCH 01/21] avutil/avassert: Add av_unreachable and
>> av_assume() macros
>>
>> Patches attached.
>>
>> - Andreas
> 
> Hi Andreas,
> 
> from https://ffmpeg.org/developer.html#Submitting-patches-1
> 
> "..ensure the correct mime type is used (text/x-diff or text/x-patch or at least text/plain) and that only one patch is inline or attached per mail."
> 
> I'm saying that because the way you are submitting, Patchwork only ever recognizes the first attached patch, so - in this case - the remaining 20 patches are not checked by the CI runs.
> It's also difficult to review patches that are submitted in this way, even more difficult to respond, as that requires to copy the patch into a text editor first, in order to add some synthetic quote marks ("> ") and when replying, it isn't clear to which of the patches the reply is corresponding to.
> 
> As long as there's no migration to a better way, why not try out making PRs to https://github.com/ffstaging/FFmpeg?

I doubt "ffstaging" is affiliated with the project in any capacity.
He can make a MR to the Forgejo instance once we enable it for that purpose.

-------------- 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/20250507/edcc4b1d/attachment.sig>