[FFmpeg-devel] [PATCH 2/3] ffprobe: replace fmt callback with str callback.

[Alexander Strasser]

Hi

Clément Bœsch wrote:
> On Tue, Sep 06, 2011 at 03:31:36PM +0200, Stefano Sabatini wrote:
[...]
> > Also I'm afraid that all this malloc/free may slow down the overall
> > performance.
> > 
> 
> This is a concern; though, I'm not sure adding a fixed buffer length
> limitation is appropriate for the ffprobe tool. While I think there is a
> real problem with tools like ffmpeg, I'm not sure it really is an issue
> here. Do you have something better to propose for variable-length
> printing?

  I think it would be instructive to make some measurements of the
old version of ffprobe vs the new version. I believe there are many
ways to change it, but blindly doing changes is IMHO not desirable
before we know for sure that there is a problem.

[...]

  Alexander