Skip to content

ProgressListener: clean #100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

cchampet
Copy link
Member

  • Add IProgress base class.
  • Rename ProgressListener to ConsoleProgress.
  • Add NoDisplayProgress.
  • Split into one class per file: IProgress, ConsoleProgress, NoDisplayProgress.

Clement Champetier added 6 commits September 23, 2014 18:25
* Base class of all possible Progress.
* ProgressListener inherits this base class.
* Add documentation.
* Swig: add feature "director" to the base class.
the class implements the IProgress, to display a progress bar in
console.
* Manage cases when we need an IProgress but don't care of a progress
bar.
* Use it in avMeta app: we want the file infos, but don't care of the
progress bar.
* Create ProgressListener.cpp.
* Add include to iostream in Transcoder.
* Contains all files related to the progress.
* Also contains corresponding swig interface file.
* 3 classes / files: IProgress, ConsoleProgress, NoDisplayProgress.
* Add a cpp file just for ConsoleProgress (the implementation is more
'complicated' than the others).
@cchampet cchampet force-pushed the dev_Progress_fix_java_binding branch from a41e28f to 6f603c1 Compare September 24, 2014 08:55
MarcAntoine-Arnaud added a commit that referenced this pull request Sep 24, 2014
@MarcAntoine-Arnaud MarcAntoine-Arnaud merged commit ae27b3c into avTranscoder:master Sep 24, 2014
@cchampet cchampet deleted the dev_Progress_fix_java_binding branch October 20, 2014 07:33
cchampet pushed a commit to cchampet/avTranscoder that referenced this pull request Feb 27, 2015
* Functions are encapsulate in Logger class.
* Fix avTranscoder#100
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants