On Wed, Sep 26, 2012 at 07:08:05PM +0200, Luca Barbato wrote: > On 09/26/2012 06:52 PM, Diego Biurrun wrote: > > .. more boredom on the train .. > > > > After this patchset all commented-out debug output is either removed > > or transformed to av_dlog. My FATE instance will then make sure it > > keeps compiling and working. > > > > The rule of thumb I used when removing debug output was to delete > > instances that only print the function name or something like a > > single int value. IOW stuff that barely takes more time to implement > > afresh than to uncomment. > > > > Most of the more complex instances I converted to av_dlog. Cases that > > failed to compile and were not completely trivial to fix or printed > > struct members that were removed many years ago, I often also removed > > the debug output. > > if fate debug works after them ok.
You mean you're OK with the whole patchset? And yes, it passes FATE with -DDEBUG enabled... Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
