search

pcroland / deew

Dolby Encoding Engine Wrapper
https://deew.dev
MIT License
202 stars 21 forks source link

[BUG] Issues with preexisting temp files #52

Closed GrzeWier closed 1 year ago

GrzeWier commented 1 year ago

Description of the the bug Hi deew has some issues if temp file(s) matching current file exist in the temp folder.

Scenario: I start encoding a file and abort after a few minutes while the measure phase is still in progress. Then I start encoding the same file again: "Intermediate already exists for the following file(s): " is displayed. After this warning I observed two different scenarios: Sometimes encoding fails with "Cannot create a file when that file already exists" Sometimes encoding will succeed only to the point where preexisting temp file ends. If original analysis was interrupted after processing 30 minutes the second encode will produce a 30 minute file.

There should an option (probably a default) to ignore/delete/overwrite any preexisting temp files. If there is no such option (or user does not use it) encoder should stop with an error/warning.

Version 5.2.1

OS Windows 10

pcroland commented 1 year ago

Yeah, deew has no way to tell if the existing temp file is full or not, I could add an option that forces temp file generation even if there's already one. But I won't really touch the code until DEE 5.6.0 gets released because I'll probably have to rewrite most of deew.