mirror of
https://github.com/rosenbjerg/FFMpegCore.git
synced 2025-12-14 10:05:44 +00:00
Update the ThrowIfExitCodeNotZero() to check the exit code before handling cancellation.
This preserves the original semantics and contract (throw only if the ffprobe exits with a non-zero code).
This commit is contained in:
parent
8f9930ad2a
commit
65cbc5552c
1 changed files with 3 additions and 3 deletions
|
|
@ -214,11 +214,11 @@ public static class FFProbe
|
|||
|
||||
private static void ThrowIfExitCodeNotZero(IProcessResult result, CancellationToken cancellationToken = default)
|
||||
{
|
||||
// if cancellation requested, then we are not interested in the exit code, just throw the cancellation exception
|
||||
// to get consistent and expected behavior.
|
||||
cancellationToken.ThrowIfCancellationRequested();
|
||||
if (result.ExitCode != 0)
|
||||
{
|
||||
// if cancellation requested, then we are not interested in the exit code, just throw the cancellation exception
|
||||
// to get consistent and expected behavior.
|
||||
cancellationToken.ThrowIfCancellationRequested();
|
||||
var message = $"ffprobe exited with non-zero exit-code ({result.ExitCode} - {string.Join("\n", result.ErrorData)})";
|
||||
throw new FFMpegException(FFMpegExceptionType.Process, message, null, string.Join("\n", result.ErrorData));
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue