On 2018-04-05 01:00, Mattias Amnefelt wrote:
On 2018-04-04 09:22, Mattias Amnefelt wrote:
On 2018-04-04 03:42, James Almer wrote:
On 4/3/2018 10:40 PM, Carl Eugen Hoyos wrote:
2018-04-04 3:38 GMT+02:00, James Almer <jamr...@gmail.com>:
On 4/3/2018 10:33 PM, Carl Eugen Hoyos wrote:
The "-f aac" looks like a bad idea to me.
It's also true for the tests above, but that's still not reason to
add more.
Please avoid top-posting here, Carl Eugen
At least in one of them it was added because the sample had too few
frames and probing was detecting it with a score of 1, which
seemed too
fragile.
I believe that it is good to have a sample that is detected with
a small score as part of fate.
Carl Eugen
When i asked it was suggested to just force the demuxer. I have no
opinion one way or another, so feel free to change it.
I have to admit I just copy-n-pasted the test above. I just
double-checked and all the id3 tag tests pass without -f aac now. I'm
not sure if anything has changed since the test was added or not. Do
you want a patch which removes it for all of them?
Here's an updated version of the patch without -f aac
/mattiasa
Did anyone have any options on the updated patch?
/mattiasa
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
