Dan,

>>  I made my comments on github.
> 
> Thanks Dan. Let me rework the patch and get back to you. I think I will:
> * Put the startElement callback back where it belongs.
> * Add code to bail out after the first pass if there is an error.
> * Add a handler for the "warning" callback which uses 
> mlt_log_warning().

Have a look at this one:
https://github.com/pez4brian/mlt/commit/fb55f5e0af62c4d9a713eea34587bd91e04601c6

It handles errors, warnings and fatals. It also skips the second pass if the 
first pass fails.

I can provide a patch to convert the entire module to use mlt_log() later.

~BM


------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel

Reply via email to