[FFmpeg-devel] [PATCH v4 0/4] Fix MSVC build without optimizations
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Fri Jul 28 13:48:18 EEST 2023
> On 28 Jul 2023, at 12:40, Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:
>
>
>> On 28 Jul 2023, at 03:37, L. E. Segovia <amy at amyspark.me> wrote:
>>
>> Updated for 6.0, any constructive feedback will be appreciated.
>
> Using #if means the code will not even be compile tested if the
> feature is disabled, this makes maintenance especially of rare
> features much harder (nevermind code readability).
> Also I believe we do not support compiling without optimizations
> even on primary platforms like Linux/gcc, so I'd find it very unlikely
> we'd want that for MSVC.
> At least there would have to be a very good argument why this is so
> important to have that we'd accept the drawbacks.
Since you asked for constructive feedback:
I assume the issue is missing symbols during linking?
If you really want this, why not create a file that provides dummy
symbols for all that are missing, concentrating the #if mess in
a single place and avoiding affecting any of the regular code, and thus
having no impact at all when compiling with optimizations.
Yes, it's likely to be a good bit of maintenance effort for those who
want to use it, but at least anyone not caring about this feature can
ignore it, so at least I would not have a reason to be against it.
Best regards,
Reimar
More information about the ffmpeg-devel
mailing list