Re: [FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-06 Thread Andrew Sayers
On Sat, Jul 06, 2024 at 06:49:51PM +0200, Michael Niedermayer wrote: > On Sat, Jul 06, 2024 at 12:40:07PM +0200, Paul B Mahol wrote: > [...] > > > its much more text to type too. > > shorter options: > > AV_OPT_FLAG_ALTERABLE_PARAM > > AV_OPT_FLAG_MUTABLE_PARAM > > (just in case consensus ends

Re: [FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-06 Thread Michael Niedermayer
On Sat, Jul 06, 2024 at 12:40:07PM +0200, Paul B Mahol wrote: [...] > its much more text to type too. shorter options: AV_OPT_FLAG_ALTERABLE_PARAM AV_OPT_FLAG_MUTABLE_PARAM (just in case consensus ends on a rename, i am not sure a rename is a good idea here) thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-06 Thread Andrew Sayers
On Sat, Jul 06, 2024 at 11:37:19AM +0200, Stefano Sabatini wrote: > On date Tuesday 2024-07-02 10:08:37 +0100, Andrew Sayers wrote: [...] > While I agree with Anton that we should avoid duplication, for the > usual arguments that a reference should avoid duplication of content > as much as possible

Re: [FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-06 Thread Paul B Mahol
On Sat, Jul 6, 2024 at 11:44 AM Stefano Sabatini wrote: > On date Tuesday 2024-07-02 10:08:37 +0100, Andrew Sayers wrote: > > Some notes about this version: > > > > As previously mentioned, I think this is better with all three patches, > > but can live without #2. > > > > I think I've followed t

Re: [FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-06 Thread Stefano Sabatini
On date Tuesday 2024-07-02 10:08:37 +0100, Andrew Sayers wrote: > Some notes about this version: > > As previously mentioned, I think this is better with all three patches, > but can live without #2. > > I think I've followed the deprecation instructions, but editing APIchanges > seems to imply

[FFmpeg-devel] [PATCH v4 0/3] s/RUNTIME/POST_INIT_SETTABLE/

2024-07-02 Thread Andrew Sayers
Some notes about this version: As previously mentioned, I think this is better with all three patches, but can live without #2. I think I've followed the deprecation instructions, but editing APIchanges seems to imply needing a minor version bump? This patch includes said bump. Zhao Zhili arg