chore: remove unused jan_utils::string::parse_c_string helper

[dev-miro26]

Describe Your Changes

• parse_c_string in src-tauri/utils/src/string.rs was pub but had no callers anywhere in the repo — a grep across src-tauri, web-app, and core only matched its own #[cfg(test)] module. It existed solely as tests-for-tests.
• Remove the function and its four test_parse_c_string* cases.
• Verified: cargo check -p jan-utils passes; cargo test (run from src-tauri/utils/) passes 32/32 with 0 failures.

Fixes Issues

• Closes chore: remove unused jan_utils::string::parse_c_string helper #8072

Self Checklist

• Added relevant comments, esp in complex areas (n/a — pure deletion)
• Updated docs (for bug fixes / features) (n/a — internal helper, no docs referenced it)
• Created issues for follow-up changes or refactoring needed

[tokamak-pm]

Review: remove unused jan_utils::string::parse_c_string helper

Summary: Removes the parse_c_string function and its four associated tests from src-tauri/utils/src/string.rs. Total: -41 lines, 0 additions.

Findings:

• Correctness: Verified -- parse_c_string had no callers anywhere in the repo outside its own test module. It was effectively tests-for-tests dead code.
• Risks: None. Pure dead-code removal.
• Style: Clean deletion with no leftover references.
• Tests: The four tests covering the removed function are deleted. Remaining tests in the module are unaffected.

Recommendation: can merge

[tokamak-pm]

Code Review

Summary: This PR removes the unused parse_c_string helper function and its associated test cases from src-tauri/utils/src/string.rs. The function had no callers outside its own test module.

Findings:

• The removal is clean and correct -- only the function definition and its four test cases are deleted.
• No other code in the repo references parse_c_string, confirming it is truly dead code.
• The remaining code and tests in string.rs are unaffected.
• No functional or behavioral changes introduced.

Recommendation: Can merge. This is a straightforward dead-code removal with no risk.

[tokamak-pm]

Review: chore: remove unused jan_utils::string::parse_c_string helper

Summary: Removes the parse_c_string function and its four test cases from src-tauri/utils/src/string.rs. The function was pub but had zero callers anywhere in the codebase outside its own test module.

Positive observations

• Pure dead-code removal: -41 lines, 0 additions.
• Verified with cargo check and cargo test (32/32 pass).
• The PR description correctly notes the function existed solely as "tests-for-tests."

Issues and observations

• Merge conflict potential. This PR, chore: remove unused find_memory_pattern and is_memory_pattern from jan_utils #8075, and chore: remove unused jan_utils::string::parse_uuid #8077 all modify the same file (src-tauri/utils/src/string.rs). Whichever merges first will cause conflicts for the others. Coordinate merge order or rebase after the first lands. Not a blocker for approval, just a heads-up.

Recommendation: can merge