#43965 closed defect (fixed)
tclsh8.5 takes 100% CPU drawing fetch progress bar
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | base | Version: | 2.3.99 |
Keywords: | Cc: | harciga | |
Port: |
Description
While fetching a large distfile, tclsh8.5 takes 100% CPU, presumably to draw the new progress bar.
Change History (6)
comment:1 Changed 10 years ago by neverpanic (Clemens Lang)
comment:2 follow-up: 4 Changed 10 years ago by harciga
I've experienced two times now —since macports upgrade— issues with the progress bar bugging out with it filling over the entire screen while flashing progress status text at different offsets, leaving me after I ctrl-brk with an inverted terminal text.
I have no idea how to report this, perhaps I can log all terminal output for when it happens again.
comment:4 Changed 10 years ago by neverpanic (Clemens Lang)
Replying to starkhalo@…:
I've experienced two times now —since macports upgrade— issues with the progress bar bugging out with it filling over the entire screen while flashing progress status text at different offsets, leaving me after I ctrl-brk with an inverted terminal text.
That's a different issue, please open a separate ticket about this. I don't think there's anything we can do about that as long as we don't have signal handling (which is what I'm currently working on on trunk), though. You'll probably also get weird results when resizing the terminal or in very small terminals. We need to handle SIGWINCH to fix this.
comment:5 Changed 10 years ago by neverpanic (Clemens Lang)
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:6 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Thanks. I forgot to let you know that yes, I did try 2.3.0, and did not see the same problem there.
I noticed the same thing, but I'm not entirely convinced it's the progress bar. Do you see the same thing in 2.3.0?
This could just as well be caused by the new code using curl_multi_perform to be able to quickly handle Ctrl+C.