On Wed, Jan 25, 2012 at 8:21 PM, Brian Matherly <pez4br...@yahoo.com> wrote: > 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
That looks great. I merged it. -- +-DRD-+ ------------------------------------------------------------------------------ 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
