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

Refactored from Thread to Task and some Task patterns to Parallel #33

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

semihokur
Copy link

I am a PhD student in the CS department at the University of Illinois. I'm currently doing research on refactoring tools for parallel programming. I developed a tool that migrates Thread constructs to Task and convert some Task patterns to higher-level Parallel class.

The tool found some opportunities in your application and performed some changes. The program behavior does not change at all. If you are not interested in the Pull Request, please let me know why.

Thanks!

@semihokur
Copy link
Author

Ok, I'm done.

@danielmarbach
Copy link
Member

Hy

Thanks for the pull. We need to review and see if changing to task in the places you highlighted is worth taking the risk of breaking existing behavior. But I assume your tool got that all covered ;)?

Am 08.12.2013 um 23:17 schrieb semihokur notifications@github.com:

Ok, I'm done.


Reply to this email directly or view it on GitHub.

@semihokur
Copy link
Author

Yes, the behaviour needs to be the same

On Mon, Dec 9, 2013 at 12:20 AM, danielmarbach notifications@github.com
wrote:

Hy
Thanks for the pull. We need to review and see if changing to task in the places you highlighted is worth taking the risk of breaking existing behavior. But I assume your tool got that all covered ;)?

Am 08.12.2013 um 23:17 schrieb semihokur notifications@github.com:

Ok, I'm done.

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub:
#33 (comment)

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.

None yet

2 participants