Skip to content

unlock event queue when handling event.#67

Open
xpol wants to merge 1 commit into
NetEase:masterfrom
xpol:unlock-event-queue-when-handling-event
Open

unlock event queue when handling event.#67
xpol wants to merge 1 commit into
NetEase:masterfrom
xpol:unlock-event-queue-when-handling-event

Conversation

@xpol

@xpol xpol commented Apr 29, 2016

Copy link
Copy Markdown
Contributor
  1. unlock event queue when handling event.
  2. and make lock scope smaller.

@xpol xpol force-pushed the unlock-event-queue-when-handling-event branch from f7347f0 to 3ce13be Compare April 29, 2016 04:44
@cynron

cynron commented Apr 29, 2016

Copy link
Copy Markdown
Member

@xpol

I'm afraid PC_PRE_ALLOC_SET_IDLE inside pc__handle_event should be protected by event_mutex as we release the mutex before entering pc__handle_event

Thanks!

@xpol

xpol commented Apr 29, 2016

Copy link
Copy Markdown
Contributor Author

@cynron
Sorry.
This is last day that I work on lua-pomelo (going for another job), and I can get it working.
I'm considering shutdown lua-pomelo.

Thanks for you great help and good luck to you!

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