mantarimay/Aegisub
1
0
Fork 0
forked from mia/Aegisub
Code Pull requests Activity

Don't try other providers after FileNotFound error

Browse source 
Otherwise this can cause confusing error messages from unrelated
providers when simply opening an invalid path.
This commit is contained in:
arch1t3cht 2023-02-08 17:37:44 +01:00
parent e4f0a9a510
commit d96fc1f70d
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/video_provider_manager.cpp
View file

@ -72,11 +72,6 @@ std::unique_ptr<VideoProvider> VideoProviderFactory::GetProvider(agi::fs::path c
LOG_I("manager/video/provider") << factory->name << ": opened " << filename; LOG_I("manager/video/provider") << factory->name << ": opened " << filename;
return provider->WantsCaching() ? CreateCacheVideoProvider(std::move(provider)) : std::move(provider); return provider->WantsCaching() ? CreateCacheVideoProvider(std::move(provider)) : std::move(provider);
} }
catch (agi::fs::FileNotFound const&) {
err = "file not found.";
// Keep trying other providers as this one may just not be able to
// open a valid path
}
catch (VideoNotSupported const&) { catch (VideoNotSupported const&) {
found = true; found = true;
err = "video is not in a supported format."; err = "video is not in a supported format.";