[FFmpeg-cvslog] cmdutils: allow specifying the file for -report

[Michael Niedermayer]

On Thu, Nov 01, 2012 at 10:28:07AM +0100, Nicolas George wrote:
> Le primidi 11 brumaire, an CCXXI, Michael Niedermayer a écrit :
> > ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Thu Nov  1 02:35:22 2012 +0100| [841bf0ef240f0d862b1f07076e079829456fb117] | committer: Michael Niedermayer
> > 
> > cmdutils: allow specifying the file for -report
> 
> I am sorry, but that does not work. The -report option is parsed early, just
> like loglevel, so that it takes effect before anything else had time to
> print something (including the banner). This option is parsed normally, so
> when it is parsed, the report file is already open (leak, easily fixed) and
> has already useful things in it (start time, exact command line, banner with
> versions).
> 
> It could be fixed by adding the necessary code to parse -report_file early
> too, but that is starting to become a little bit insane, and I was still
> trying to come up with a better solution.

better suggestion than this commit, posted
please review

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20121101/c7157733/attachment.asc>