メインループの異常時間判定見直し

• Reporter: tsugehara
• Owner: tsugehara
• Type: Feature Requests
• Status: Closed
• Component: (None)
• MileStone: 2.2 (deleted)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: None
• File: None

Details Reply

if (this.tick > (t+10000) || (this.tick+10000) < t) {

この「1回の更新に10秒かかっているようならタイマーのバグと判定」という処理だが、ちょっと悠長すぎる。

1秒くらいで十分。

Ticket History (3/5 Histories)

2013-07-24 09:44 Updated by: tsugehara

• New Ticket "メインループの異常時間判定見直し" created

2013-07-24 09:45 Updated by: tsugehara

Comment Reply

本当は1秒でも長過ぎるくらいだが、500msecくらいは普通に起こりえる。

500msec以上更新に時間がかかるようなら、時間ベースの更新を諦めフレームベースにする、とかの解決策でも可。

2013-07-28 10:43 Updated by: tsugehara

Comment Reply

ただスタンバイからの復帰などを想定してrefreshをしているので、フレームベースへの切り替えと強制refreshは分けて考えるべき。

2段階がいい？

500msecで時間単に諦め、5秒で強制refresh。

2013-09-01 12:40 Updated by: tsugehara

• Milestone Update from (None) to 2.2 (deleted)

2013-09-10 18:16 Updated by: tsugehara

• Status Update from Open to Closed
• Ticket Close date is changed to 2013-09-10 18:16

Comment Reply

r161 にて対応。予定通り2段階判定にした。

500msecで時間単位諦めてフレームベース系更新に。10秒で強制リフレッシュ。

また、全体的にループのif文をすっきりさせたつもり。