search

Xiaofei-it / Hermes

A smart, novel and easy-to-use framework for Android Inter-Process Communication (IPC). （简单易用的安卓进程间通信IPC框架）
Other
1.28k stars 193 forks source link

Concurrency Issue #12

Closed wfmarques closed 7 years ago

wfmarques commented 7 years ago

Guys i found a concurrency problem with the property mMethod in the class Sender -> Reply send(Method method, Object[] parameters) , calling 2 methods in exactly same time cause the mMethod switch the Method name so i'm calling method1 with 2 parameters and Hermes complain about calling method2 than expect Zero parameters, i did a fix and is working until now.

Xiaofei-it commented 7 years ago

@wfmarques it is interesting. Does the code you added fix the bug? If so, I will merge the pull request and make some more modifications.

wfmarques commented 7 years ago

Yeah fixed. I'm using this version and os working until now.

Em 5 de mai de 2017 00:31, "Xiaofei" notifications@github.com escreveu:

@wfmarques https://github.com/wfmarques it is interesting. Does the code you added fix the bug? If so, I will merge the pull request and make some more modifications.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Xiaofei-it/Hermes/pull/12#issuecomment-299362224, or mute the thread https://github.com/notifications/unsubscribe-auth/AA-oNKBLa0e1JLfq0nSGVuSnRpHes79eks5r2pgAgaJpZM4NDA0a .

Xiaofei-it commented 7 years ago

@wfmarques OK. I will check and merge it soon.

Thank you for this pull request.