Skip to content

improvements #3

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

improvements #3

wants to merge 30 commits into from

Conversation

karasikov
Copy link

@karasikov karasikov commented Mar 17, 2020 •
edited
Loading

Features added:

  • thread safety
  • adjust the bar size automatically when the terminal is resized
  • special logging format when redirected to file
  • use operator++ to increment the progress
  • other cleanups and minor fixes

karasikov and others added 20 commits March 29, 2019 01:48
karasikov and others added 9 commits October 29, 2020 17:45
@karasikov
flush new line right after the total process is achieved
f3b1b02
@karasikov
fix progress increments
1598014
@badshah400
Cast (const char *) to (char *) for snprintf arg.
c19fc52 
snprintf() accepts (char *) rather than (const char *) for its first
argument. This fixes an `-fpermissive` warning/error when compiling
progress_bar.o.
@badshah400
Fix atomic initialisation.
2abb8c2
@badshah400
main: Replace Progressed() by increment operators.
677bf88 
ProgressBar::Progressed() has been replaced by increment operators.
Adapt examples in main code accordingly.
@badshah400
Pass char instead of (char *) to SetStyle().
f8dfa61 
This fixes compiling main.o without `-fpermissive`.
@karasikov
Update main code to current API and some -fpermissive fixes
007b6b2
@karasikov
cleanup, fixes of examples in main.cpp
a4f217f
@karasikov
minor changes
7208674
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@karasikov @hmusta @badshah400