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

VSync無効時にフレームレートを制限する機能を追加 #1205

Draft
wants to merge 2 commits into
base: v6_develop
Choose a base branch
from

Conversation

m4saka
Copy link
Contributor

@m4saka m4saka commented Feb 16, 2024

下記Issueについて実装しました。

#1180

実装にあたって気になっている点は下記2点です。

  1. 目標フレームレート(fps引数)にゼロや負の値を指定した時に例外を投げているのは問題ないか?
  2. 目標フレームレート(fps引数)にNaNや+inf、-infを指定した場合も例外を投げるべきか?
    補足事項として、Qiita記事での実装ではMaxDrift(=10ミリ秒)という定数値を導入していましたが、now < sleepUntilの場合のみsleepを実行することで不要になったので、今回の実装には入っていません。

手元では現状Windows版のみで動作確認しているので、macOS版、Linux版、Web版でも正常動作するかどうかは今後確認予定です。
(Xcodeプロジェクトへのソースファイル追加についても現状は未対応です)

下記の残作業が完了したらDraft PR→PRに変更予定です。

  • Xcodeプロジェクトのソースファイル一覧更新
  • コメントアウトで残っている旧SetTargetFrameRateHz用のコードの削除

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants