Feature request: Raising OnErrorThrownEventArgs instead of an exception when an error occurs in FFmpeg about mediatoolkit HOT 3 OPEN

The issue is that FFmpeg writes to STDERR, so we can only read the output from ErrorDataReceived. This is how we raise progress events and completion events by reading the output and attempting to serialise it,

If FFmpeg exits with errors, it will return an ExitCode of something other than 0 or 1, in such situations the an exception is composed and thrown. You can find the code below in Engine.cs at the end,

if ((this.FFmpegProcess.ExitCode != 0 && this.FFmpegProcess.ExitCode != 1) || caughtException != null)
{
    throw new Exception(
    this.FFmpegProcess.ExitCode + ": " + receivedMessagesLog[1] + receivedMessagesLog[0],
    caughtException);
}

from mediatoolkit.

That is all fine but instead of throwing a new exception would it not be great if the OnError event is invoked? Ofcourse, if no OnError event handler is attached an exception is thrown.

from mediatoolkit.

Ah my bad, I misunderstood you. I like that idea!

If I'm honest... I want to bulldoze the entire thing and rebuild from scratch, sometimes I wonder if I was drunk the entire time I was writing the damn thing.

from mediatoolkit.