Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-12-01 Thread Paul B Mahol
On 12/1/22, Oneric wrote: > On Sun, Nov 20, 2022 at 01:15:33 +0100, Oneric wrote: >> On Sun, Nov 13, 2022 at 20:57:15 +0100, Oneric wrote: >> > This fixes colours and font selection for files converted to ASS >> >> Ping. Will apply if nobody is against. Please ping me directly if I do not apply

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-30 Thread Oneric
On Sun, Nov 20, 2022 at 01:15:33 +0100, Oneric wrote: > On Sun, Nov 13, 2022 at 20:57:15 +0100, Oneric wrote: > > This fixes colours and font selection for files converted to ASS > > Ping. > > As a reminder v1 and v2 are equivalent and patchwork is buggy and can > thus neither apply v1 or v2 and

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-19 Thread Oneric
On Sun, Nov 13, 2022 at 20:57:15 +0100, Oneric wrote: > This fixes colours and font selection for files converted to ASS Ping. As a reminder v1 and v2 are equivalent and patchwork is buggy and can thus neither apply v1 or v2 and its results can be ignored and its patch or diff downloads should no

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-14 Thread Oneric
On Sun, Nov 13, 2022 at 22:57:22 +0100, Oneric wrote: > On Sun, Nov 13, 2022 at 21:01:34 +, Soft Works wrote: > > I am unable to apply any of those patches. Neither v1 nor v2 neither > > when downloading the series mbox or [2/3] from patchwork nor when > > saving [2/3] from the e-mail. > > Re

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 21:01:34 +, Soft Works wrote: > I am unable to apply any of those patches. Neither v1 nor v2 neither > when downloading the series mbox or [2/3] from patchwork nor when > saving [2/3] from the e-mail. Re patchwork, yes I rechecked and it appears all downloads are now br

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 10:00 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > &

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 9:27 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > >

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 21:27:11 +0100, Oneric wrote: > [...] You can easily verify emails are capable of it by > applying patch 2 yourself or using patchwork’s mbox file. A correction: it seems since last time we discussed this, some patchwork update has expanded the line-mangling to also affect

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 20:15:23 +, Soft Works wrote: > > [...] it's not a Patchwork bug and it cannot be "fixed" > at all in any other way than using binary diffs. The problem lies in > the fact that it's being sent by e-mail where the different line > endings get lost (as the spec mandates

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 8:57 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > > This fixes colours and font selection

[FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
This fixes colours and font selection for files converted to ASS v2 == On Nicolas George’s request resent with plain text diff for the updated refernce files. There is no other change and applying the diff of v1 yields equal results to v2. Those files do intentionally contain CRLF line endings,