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.

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

webi2c.jsのメモリリーク #24

Open
tadfmac opened this issue Dec 22, 2016 · 2 comments
Open

webi2c.jsのメモリリーク #24

tadfmac opened this issue Dec 22, 2016 · 2 comments

Comments

@tadfmac
Copy link
Contributor

tadfmac commented Dec 22, 2016

Expected Behavior

I2CSlaveDevice配下のメソッド内部で呼び出されるwindow.WorkerOvserve.observe()に対応するunobserve()が実装されていないため、特にwrite8()write16()read8()read16()などを同一デバイスに対して繰り返し呼び出すことで、これらメソッド内部のwindow.WorkerOvserve.notify()の実行時間がどんどん長くなる現象が発生しています。

Actual Behavior

I2CSlaveDevicewrite8()write16()read8()read16()等を繰り返し呼び出してもメモリリークが発生せず、呼び出し回数に応じて実行時間が長くなるような現象は発生しない。

Steps to Reproduce the Issue

Specifications

(The version of the project, browser etc.)

@MSakamaki
Copy link
Contributor

発見ありがとうございます。
#22 マージ後に対応します!

@tadfmac
Copy link
Contributor Author

tadfmac commented Feb 7, 2017

不具合報告だけで修正案提示できておらず申し訳ありません。
念のため、#25 と併せて修正案を作成中です。

@tadfmac tadfmac mentioned this issue Feb 10, 2017
3 tasks
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

No branches or pull requests

2 participants