fix: windows build

[namecallfilter]

Changed timespan to timestamp

Summary by CodeRabbit

• Chores

  • Windows platform dependencies updated to the latest available patch version to ensure optimal system compatibility and ongoing stability improvements.

• Bug Fixes

  • Frame capture timing refinements: The frame capture engine's timing calculations have been improved to provide more accurate timestamp processing, resulting in better synchronization and overall timing accuracy for frame capture operations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This pull request updates the Windows dependency version pin in Cargo.toml and modifies frame timestamp calculation in the Windows frame capture engine to use timestamp() instead of timespan() when computing elapsed time.

Changes

Cohort / File(s)	Summary
Dependency Version Update Cargo.toml	Updated windows crate version from "0.58" to "0.58.0" (explicit patch version pinning)
Frame Capture Timing Logic src/capturer/engine/win/mod.rs	Changed frame elapsed time calculation from frame.timespan().Duration to frame.timestamp().Duration in on_frame_arrived handler

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• src/capturer/engine/win/mod.rs: Verify that switching from timespan() to timestamp() correctly computes elapsed time and doesn't introduce timing regressions. Understand the semantic difference between the two methods in the context of frame processing.
• Cargo.toml: Confirm explicit patch version pinning to "0.58.0" is intentional and compatible with existing feature usage.

Poem

🐰 A timestamp hops more truly than before,
From timespan's leap to precision's door,
Windows 0.58.0 pins the way,
Capture frames with perfect timing today! ⏱️✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check	❓ Inconclusive	The title 'fix: windows build' is vague and does not clearly describe the specific changes made. The PR contains two distinct changes: a dependency version update and a method name change from timespan() to timestamp().	Provide a more specific title that describes the primary change, such as 'fix: use timestamp() instead of timespan() for frame timing' or include the specific fix being addressed.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}