ztepsic / ztepsic.ffmpeg Goto Github PK

Write test that will check if values for MediaFormatInfo or MediaStreamInfo are not defined, factory should return null.

Killing processes when ffmpeg is not completed within the given time does not work.

Create Exception wrapper for XML loading and parsing

• xmlDoc.LoadXml(xml); can throw a XmlException because of that create exception wrapper
• add comment to methods explaining which exception could be thrown

Add log4Net on strategic places like:

• when starting the process
• when the process is finished
• when the process is killed

Extract factory methods CreateFromFile and CreateFromXml into interface.

In that case factory can't be static class but dynamic class.

Create in house/custom Timeout ffmpeg exception. Exception which is artificially created when the time allotted for a process has expired. Exception is created like as ffmpeg/ffprobe error output which is later parsed and thrown as Timeout ffmpeg exception. Need to support different types of writers for ffprobe (XML, JSON, etc.)

Something like: The exception that is thrown when the time allotted for a process or operation has expired. http://msdn.microsoft.com/en-us/library/system.timeoutexception.aspx

Separate exceptions in subclasses of the FFmpegException class.

FFprobe errors:

• Code: -1094995529; Message: Invalid data found when processing input
• Code: -5; Message: Input/Output error
• Code: -10060; Message: Unknown error
• Code: -1; Message: Operation not permited.