Skip to content

chore: VLCKit更新 - #85

Merged
ci7lus merged 1 commit into
mainfrom
fix/paused-seek-vlckit
Aug 11, 2026
Merged

chore: VLCKit更新#85
ci7lus merged 1 commit into
mainfrom
fix/paused-seek-vlckit

Conversation

@ci7lus

@ci7lus ci7lus commented Aug 11, 2026

Copy link
Copy Markdown
Owner

一時停止中にシークすると挙動がおかしくなる件の修正

Fixes #82

neneka/vlckit#68

Summary by Sourcery

一時停止中のシーク動作を改善するために、VLCKit のリビジョンを更新し、プレーヤー設定を調整しました。

Enhancements:

  • シーク中の再生を安定させるため、VLC プレーヤー初期化時にプリフェッチバッファサイズのオプションを追加しました。

Build:

  • VLCKit セットアップスクリプトを最新のリリース済みリビジョンを使用するように更新しました。
Original summary in English

Summary by Sourcery

Update VLCKit revision and adjust player configuration to improve seek behavior while paused.

Enhancements:

  • Add a prefetch buffer size option to the VLC player initialization to stabilize playback during seeking.

Build:

  • Bump VLCKit setup script to use the latest released revision.

一時停止中にシークすると挙動がおかしくなる件の修正
@sourcery-ai

sourcery-ai Bot commented Aug 11, 2026

Copy link
Copy Markdown
レビュアー向けガイド(小さな PR では折りたたみ)

レビュアー向けガイド

VLCKit の依存関係のリビジョンを更新し、VLC プレーヤーにプリフェッチバッファサイズを調整するオプションを追加します。これは、一時停止中のシーク時に発生する異常な挙動を修正することを目的としています。

ファイルレベルの変更

変更内容 詳細 ファイル
VLCKit の依存関係を新しいリリース済みリビジョンに更新。
  • セットアップスクリプトで使用している VLCKit リリースリビジョン識別子を変更。
  • URL、出力先パス、およびフレームワークの出力先ロジックは変更しない。
setup.sh
プリフェッチバッファサイズを制御する VLC メディアプレーヤーオプションを追加。
  • デフォルトの VLC オプション配列を拡張し、プリフェッチバッファサイズのパラメータを追加。
  • 既存のスナップショットおよび字幕関連のオプションはそのまま残す。
kiririn/Features/Player/PlayerState.swift

紐づく Issue に対する評価

Issue 目的 対応状況 説明
#82 一時停止中にシークすると誤った再生状態になるバグを修正した新しいリビジョンに VLCKit を更新する。
#82 一時停止中のシークに関する VLCKit の更新後の挙動が正しく動作するよう、必要に応じてプレーヤーの設定/オプションを調整する(例:バッファリング/プリフェッチ設定)。

関連している可能性のある Issue

  • 動画一時停止中にシークすると再生状態がおかしくなる #82: この PR は、Issue で説明されている、一時停止中のシーク時に発生する VLCKit の再生バグを修正するために、VLCKit のリビジョンとプレーヤーオプションを更新しています。
  • #unknown: この PR は、Issue で「シークが変」として報告されている異常なシーク挙動を修正するために VLCKit を更新しています。

Tips とコマンド

Sourcery との対話

  • 新しいレビューをトリガーする: プルリクエストに @sourcery-ai review とコメントします。
  • ディスカッションを続ける: Sourcery のレビューコメントに直接返信します。
  • レビューコメントから GitHub Issue を作成する: レビューコメントに返信して、そこから Issue を作成するよう Sourcery に依頼します。レビューコメントに @sourcery-ai issue と返信しても Issue を作成できます。
  • プルリクエストのタイトルを生成する: プルリクエストのタイトル内のどこかに @sourcery-ai と書くと、いつでもタイトルを生成できます。プルリクエストに @sourcery-ai title とコメントして、タイトルをいつでも(再)生成することもできます。
  • プルリクエストのサマリーを生成する: プルリクエスト本文の任意の場所に @sourcery-ai summary と書くと、その場所に PR サマリーをいつでも生成できます。プルリクエストに @sourcery-ai summary とコメントして、サマリーをいつでも(再)生成することもできます。
  • レビュアー向けガイドを生成する: プルリクエストに @sourcery-ai guide とコメントして、レビュアー向けガイドをいつでも(再)生成します。
  • すべての Sourcery コメントを解決済みにする: プルリクエストに @sourcery-ai resolve とコメントして、すべての Sourcery コメントを解決済みにします。すでにすべてのコメントに対応済みで、それ以上表示されたくない場合に便利です。
  • すべての Sourcery レビューを破棄する: プルリクエストに @sourcery-ai dismiss とコメントして、既存の Sourcery レビューをすべて破棄します。新しいレビューでやり直したい場合に特に便利です。新しいレビューを開始するには @sourcery-ai review とコメントするのをお忘れなく!

体験のカスタマイズ

ダッシュボード にアクセスして次のことができます:

  • Sourcery が生成するプルリクエストサマリー、レビュアー向けガイドなどのレビュー機能を有効/無効にする。
  • レビュー言語を変更する。
  • カスタムレビュー指示を追加・削除・編集する。
  • その他のレビュー設定を調整する。

サポートを受ける

Original review guide in English
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Updates the VLCKit dependency revision and adds a VLC player option to adjust the prefetch buffer size, intended to fix abnormal seeking behavior while paused.

File-Level Changes

Change Details Files
Update VLCKit dependency to a newer released revision.
  • Change the VLCKit release revision identifier used by the setup script.
  • Keep URL, destination paths, and framework destination logic unchanged.
setup.sh
Add VLC media player option to control prefetch buffer size.
  • Extend the default VLC options array with a prefetch buffer size parameter.
  • Leave existing snapshot and caption-related options intact.
kiririn/Features/Player/PlayerState.swift

Assessment against linked issues

Issue Objective Addressed Explanation
#82 Update VLCKit to a newer revision that fixes the bug where seeking while paused causes incorrect playback state.
#82 Adjust player configuration/options as needed so that the updated VLCKit behavior for seeking while paused works correctly (e.g., buffering/prefetch settings).

Possibly linked issues


Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こんにちは。変更内容を確認しましたが、とても良い出来です!


Sourcery はオープンソース向けには無料です — レビューを気に入っていただけたら、ぜひ共有をご検討ください ✨
もっと役に立てるようにしたいので、各コメントに対して 👍 または 👎 をクリックしてください。そのフィードバックをもとに、今後のレビューを改善していきます。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@ci7lus
ci7lus merged commit 0a16461 into main Aug 11, 2026
8 checks passed
@ci7lus
ci7lus deleted the fix/paused-seek-vlckit branch August 11, 2026 14:11
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

Successfully merging this pull request may close these issues.

動画一時停止中にシークすると再生状態がおかしくなる

1 participant