Skip to content

fixed thread hanging issue#406

Merged
mr-c merged 2 commits into
masterfrom
fix/infinite_threadloop
May 6, 2014
Merged

fixed thread hanging issue#406
mr-c merged 2 commits into
masterfrom
fix/infinite_threadloop

Conversation

@ctb

@ctb ctb commented Apr 23, 2014

Copy link
Copy Markdown
Member

Corrected a regression where ReadParser was being created once for each thread, causing a hang (see #384).

@ctb ctb mentioned this pull request Apr 23, 2014
@ctb

ctb commented Apr 23, 2014

Copy link
Copy Markdown
Member Author
  • Is it mergable
  • Did it pass the tests?
  • N/A If it introduces new functionality in scripts/ is it tested?
    Check for code coverage.
  • Is it well formatted? Look at pep8/pylint, cppcheck, and
    make doc output. Use autopep8 and astyle -A10 if needed.
  • Is it documented in the Changelog?

@ctb

ctb commented Apr 24, 2014

Copy link
Copy Markdown
Member Author

jenkins, please test this.

@mr-c mr-c added this to the v1.0.1 milestone Apr 25, 2014
@mr-c mr-c mentioned this pull request Apr 25, 2014
mr-c added a commit that referenced this pull request May 6, 2014
@mr-c mr-c merged commit c661547 into master May 6, 2014
@mr-c mr-c deleted the fix/infinite_threadloop branch May 6, 2014 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants