On 22-05-2019 02:29 PM, Hendrik Leppkes wrote:
On Wed, May 22, 2019 at 10:53 AM Gyan <ffm...@gyani.pro> wrote:


On 22-05-2019 01:41 PM, Liu Steven wrote:
在 2019年5月22日,下午3:10,Gyan <ffm...@gyani.pro> 写道:


Helps users to identify temp files for a given instance.

In the longer term, we should aim to clean up all temp files.

Gyan
<0001-avutil-file_open-log-temp-filename.patch>_______________________________________________
Why don’t use AV_LOG_DEBUG or AV_LOG_TRACE?

DEBUG and TRACE are meant to record micro-ops for debugging purposes.
This message will be present a handful of times in the log and is
essential for end-users so they can delete these files, which ffmpeg
won't do, but should since these are meant to be temporary files.

Why don't we work on fixing this instead, as users will not know what
implications this message has in any case and just consider it spam.

1) A grep of the tree shows only two components make use of this function. The cache protocol, which needs to be expressly invoked by the user, and the xvid encoder in 2-pass mode, which also has to be expressly invoked. No unsuspecting user will be presented with this message.

2) Fixing this is the ideal solution. The above two components already unlink the file after use, but with cache:, the file remains available after ffmpeg has exited. So this has to be looked at by someone acquainted and able to test with all the environments that ffmpeg can be run on. I don't have that knowledge or access. Till then, let's not the perfect be the enemy of the good.

Gyan
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to