[Bug 731853] New tool : Add a CodecAnalyzer

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=731853
  GStreamer | gst-devtools | unspecified

--- Comment #18 from sreerenj <bsreerenj at gmail.com> 2014-07-07 20:00:02 UTC ---
(In reply to comment #17)
> Not a showstopper for submission, but it would be nice to have some top-level
> comment block in main classes to describe how it works, e.g. gst-analyzersink
> could have one.

Thanks for the review. I will add those. 

> The gst-pipeline is producing intermediate files at
> ~/tmp/codecanalyzer/{hex,xml}/
> the code is from analyzer_create_dirs(). This should use g_get_user_cache_dir()
> if the data is reused or g_get_tmp_dir().

I will add this too.

> 
> On the other hand I am not super fond of the filesystem interaction. What about
> using nested GstStructures for the data and emitting them as GstBusMessages?

Do you mean to keep pipeline alive all the time during the analysis?
Right now the pipeline is active only during parsing time. After that it is all
about xml data manipulation and there is no gstreamer interaction.

Anyway Edward had also some ideas about emitting parser info as GstMessages.
But that we can discuss later since it needs more changes in videoparser
plugins too. what do you think?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.