time.sleep(-1.0) causes panic #5895

Activity

1. youknowone

   added

   C-bugSomething isn't workingSomething isn't working

   on Jul 4, 2025
2. youknowone

   added the

   Bug

   issue type on Jul 4, 2025
3. youknowone

   added

   good first issueGood for newcomersGood for newcomers

   on Jul 4, 2025
4. DominicPrince2003 commented on Jul 5, 2025

   

   DominicPrince2003

   on Jul 5, 2025

   Last edited by DominicPrince2003

   More actions

   Hi, can I take up this issue?

   I'm planning to modify the time.sleep() implementation to safely handle:

   • Negative values

   Proposed solution:

   • Modify the dur type to f64 insted of Duration
   • Validate the f64 input before conversion
   • Raise a ValueError if the input is invalid
   • For valid values, use Duration::from_secs_f64()

   Let me know if this approach is okay.
5. youknowone commented on Jul 6, 2025

   

   youknowone

   on Jul 6, 2025

   Last edited by youknowone

   MemberAuthor

   More actions

   Yes, that's one of the possible solutions. Thanks!
6. youknowone

   closed this as completedin #5906on Jul 7, 2025