bonsai/harakit
5
4
Fork 3
Code Issues 42 Pull Requests 5 Projects 1 Releases 34 Activity

eep(1) #103

Open
trinity wants to merge 16 commits from sleep into main
pull from: sleep
merge into: bonsai:main
Conversation 3 Commits 16 Files Changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b7fe7294c6 - Show all commits

2
src/eep.rs
View File

@ -36,7 +36,7 @@ fn main() -> ExitCode {
if argv.len() == 2 {
if let Ok(s) = argv[1].parse::<u64>() {
emma marked this conversation as resolved
emma commented 2024-07-15 14:13:03 -06:00
Review
Copy Link

This could theoretically be a u128 to allow larger numbers.

This could theoretically be a `u128` to allow larger numbers.
trinity commented 2024-07-15 14:58:09 -06:00
Review
Copy Link

Perhaps that can be a feature for coma(1).

Perhaps that can be a feature for coma(1).
trinity commented 2024-07-15 15:47:13 -06:00
Review
Copy Link

Duration::from_secs needs a u64 which was the basis of that decision.

[Duration::from_secs](https://doc.rust-lang.org/std/time/struct.Duration.html#method.from_secs) needs a u64 which was the basis of that decision.
sleep(Duration::from_secs(s));
sleep(Duration::from_secs(s)); /* from_secs needs u64 */
ExitCode::SUCCESS
} else {
usage(&argv[0])