Ticket #2192 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Parrot_cx_scheduler_sleep is wrong in the no threads case

Reported by: NotFound Owned by: NotFound
Priority: normal Milestone:
Component: core Version: 3.7.0
Severity: low Keywords:
Cc: Language:
Patch status: Platform:

Description

Parrot_cx_scheduler_sleep has a conditional compilation branch depending on PARROT_HAS_THREADS. In the non thread case it uses Parrot_sleep and Parrot_usleep, but the call to Parrot_usleep is wrong: (UINTVAL) time*1000000 should be (UINTVAL) (time*1000000).

This makes most Timer tests fail, and thus are skipped when no threads, but they should work unless te platform does not provide sleep and usleep (de we support any platform like that?).

I've created the branch NotFound/fix_sleep to test the fix.

Change History

Changed 3 years ago by NotFound

  • status changed from new to closed
  • resolution set to fixed

The branch has been merged into master together with whiteknight/kill_threads. Problem solved, closing.

Note: See TracTickets for help on using tickets.