Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

update man page text about delay #971

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

update man page text about delay #971

wants to merge 1 commit into from

Conversation

davemq
Copy link

@davemq davemq commented Dec 17, 2019

It has bothered me for quite some time that running htop -d 590 does not delay 59 seconds but something much less. I finally looked at the code today and see that the delay is clamped to `1 <= delay <= 100', i.e. a minimum of 0.1 seconds and maximum of 10.0 seconds. It's not clear to me in looking at the code history if there's some design reason for this, so I propose this change to the manual page. I'd be happy to change the code to remove this restriction and revert this proposed change to the manual page, but I think that requires a bit more discussion.

@davemq
Copy link
Author

davemq commented Dec 19, 2019

Looking a bit further into this, the delay value eventually gets used in a call to halfdelay() in ncurses. The halfdelay(3) man page says

The halfdelay routine is used for half-delay mode, which is similar to cbreak mode in that characters typed by the user are immediately available to the program. However, after blocking for tenths tenths of seconds, ERR is returned if nothing has been typed. The value of tenths must be a number between 1 and 255. Use nocbreak to leave half-delay mode.

@natoscott
Copy link
Collaborator

Merged here: htop-dev/htop@f4b0067

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants