style: fix clippy::redundant_closure_for_method_calls warnings

[Brooooooklyn]

Replace redundant closures with direct method references:

• with_argv.rs: Replace |p| p.is_null() with <*const i8>::is_null
• which.rs: Replace |s| s.to_owned() with BStr::to_owned

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

[Copilot]

Pull Request Overview

This PR addresses Clippy warnings by replacing redundant closures with direct method references, improving code conciseness without changing functionality.

• Replaced closure |p| p.is_null() with direct method reference <*const i8>::is_null
• Replaced closure |s| s.to_owned() with direct method reference BStr::to_owned

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
crates/fspy_shared_unix/src/exec/which.rs	Simplified closure to method reference for BStr::to_owned in test
crates/fspy_preload_unix/src/interceptions/spawn/exec/with_argv.rs	Simplified closure to method reference for pointer null check

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Brooooooklyn]

• style: fix clippy::redundant_clone warning #233
• style: fix clippy::uninlined_format_args warnings #232
• style: fix clippy::items_after_statements warnings #231
• style: fix clippy::redundant_closure_for_method_calls warnings #230 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Brooooooklyn]

Merge activity

• Oct 17, 6:08 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Oct 17, 6:08 AM UTC: @Brooooooklyn merged this pull request with Graphite.