Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix event loop #42

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

Fix event loop #42

wants to merge 5 commits into from

Commits on Oct 19, 2013

  1. Fix locker status for QWaitCondition. 

    Should lock mutex first before wait() and wakeAll().
Also use QMutexLocker to prevent posible deadlock.
    @legnaleurc
    legnaleurc committed Oct 19, 2013
    Configuration menu
    Copy the full SHA
    7043736 View commit details
    Browse the repository at this point in the history

  2. Don't queue AsyncWork again in AsyncAfter. 

    This causes many problems.
    @legnaleurc
    legnaleurc committed Oct 19, 2013
    Configuration menu
    Copy the full SHA
    b43fd46 View commit details
    Browse the repository at this point in the history

  3. Use reference counter to stop event loop. 

    This may prevent endless processEvent() at exit.
    @legnaleurc
    legnaleurc committed Oct 19, 2013
    Configuration menu
    Copy the full SHA
    de00932 View commit details
    Browse the repository at this point in the history

  4. Get result only if succeed.

    @legnaleurc
    legnaleurc committed Oct 19, 2013
    Configuration menu
    Copy the full SHA
    53f5616 View commit details
    Browse the repository at this point in the history

  5. Use smart pointer to prevent possible leak.

    @legnaleurc
    legnaleurc committed Oct 19, 2013
    Configuration menu
    Copy the full SHA
    0ff96f5 View commit details
    Browse the repository at this point in the history