Ticket #538 (closed todo: done)

Opened 2 years ago

Last modified 2 years ago

review and merge thread-integration

Reported by: ichthyo Owned by: ichthyo
Priority: normal Milestone: 0integration
Component: lumiera Keywords: integration
Cc: Blocked By:
Blocking:

Description

The new additions and changes to the thread handling, together with the C++ integration should be reviewed and merged to master rather quickly. Additional benefit would be that we'll get an uniform NoBug version level on all branches again.

I know that work on the threadpool is ongoing; question is if we can determine a point for the merge, where the existing functionality is reasonably stable for master.

Change History

comment:1 Changed 2 years ago by ichthyo

  • Owner set to ichthyo
  • Status changed from new to accepted

mostly done now. I've one open topic (isValid()) on the threadwrapper. After that, I'll merge the current state from Plouj (which also includes recent changes by Cehteh) and land this to master.

comment:2 Changed 2 years ago by ichthyo

  • Keywords meeting removed
  • Status changed from accepted to closed
  • Resolution set to done

Rounded up the thread wrapper and clarified the issue regarding isValid(). Merge to master went well.

There remains an (unrelated) issue with a test regexp failing, which I'll investigate separately

Note: See TracTickets for help on using tickets.