テーマインスタンスでIntegratedのカラムちゃんと使いたいから修正できないかやってみよう
テーマインスタンスでIntegratedのカラムちゃんと使いたいから修正できないかやってみよう
NowPlayingを連続でクリックしたらipcRenderer.sendが何度も呼ばれる上にipcRenderer.onceも何度も呼ばれて多重投稿になるって感じだからメモリリークを防ぐためにもsendを繰り返さないようにしてonceも1度だけ呼んで2回同じことを繰り返すっていう可能性を潰してやる
@Cutls @toneji 二重取得、二重投稿を確実に防げるようになるはずの修正をしてみました。確認していただければと思います
https://github.com/cutls/TheDesk/pull/38
画像のuploadをuploadnow変数で無理矢理制御してるの、リスナーの数で制御するように変更してみたのであとでpr送ります(家出てるので作業出来ない)
@Cutls あぁgithub:のプレフィックスがやはりだめでしたか(最初いやな予感してたけど意外とほかの環境で動いてるから問題ないのかと思いました…)
もしかしてsourceビルドでprepareコマンドコケたらoptionalDependencies関係なく失敗する…?
このアカウントは、notestockで公開設定になっていません。
TheDesk、listenerCountとsucceeded変数使って楽曲情報の重複取得防げるようにした
こんな感じで
```
var ipc = require("electron").ipcRenderer;
if (ipc.listenerCount('itunes-np') > 0) {
return false;
}
ipc.send('itunes', "");
ipc.once('itunes-np', function (event, arg) {});
```
listenerCountってメソッドあるからそれが>0ならそもそもイベント発火もリスナー起動もしなくていいのでは…?ってドキュメント見ながら思ってるので後で試してみるか
画像を複数添付してしまう問題、onceを宣言?する前にremoveAllListener実行してやれば良さそう?
onceで画像の重複するの、まぁ仕方がないところはあるんだけど、removeAllListenerにすれば変わるかしら?
https://qiita.com/dojyorin/items/a6b5ae7676e3cda2b86f#イベントリスナ乱立対策
TheDeckのNowPlaying(iTunes)でのアートワーク重複、以前から内部では起きてて、画像っていうどんどん追加されるもので露呈したバグですね…