Merge e8bebe0b03 into 21792b88b7

[postprocessor/ffmpeg] Allow options for mp3
Why? Options like `preferredquality` are ignored when postprocessing mp3s. What? Remove `mp3` from the list of `filecodec` values to check for in the "lossless" condition unless `preferredquality` is unspecified.
2024-06-01 18:48:18 +00:00 · 2024-04-02 03:35:18 +02:00 · 2023-08-17 07:48:20 -07:00
1 changed files with 4 additions and 1 deletions
--- a/youtube_dl/postprocessor/ffmpeg.py
+++ b/youtube_dl/postprocessor/ffmpeg.py
@ -270,7 +270,10 @@ class FFmpegExtractAudioPP(FFmpegPostProcessor):
                acodec = 'copy'
                extension = 'm4a'
                more_opts = ['-bsf:a', 'aac_adtstoasc']
-            elif filecodec in ['aac', 'flac', 'mp3', 'vorbis', 'opus']:
+            elif (
+                filecodec in ['aac', 'flac', 'vorbis', 'opus'] or
+                (filecodec == 'mp3' and self._preferredquality is None)
+            ):
                # Lossless if possible
                acodec = 'copy'
                extension = filecodec