Replace strtok with thread-safe strtok_r in MPS parser

[rgsl888prabhu]

Summary

Replace strtok() with strtok_r() in parse_string() to make MPS file parsing thread-safe. strtok uses internal static state, so concurrent parsing from multiple threads corrupts tokenization.

Testing

No new tests added. Existing MPS parser tests cover this code path.

Documentation

No documentation changes needed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 45e1fe4d-77bc-47d9-97df-f3b0dc31e682

📥 Commits

Reviewing files that changed from the base of the PR and between 378add0 and e1b919b.

📒 Files selected for processing (1)

• cpp/libmps_parser/src/mps_parser.cpp

---

📝 Walkthrough

Walkthrough

Replaced the non-reentrant strtok function with the reentrant strtok_r function in the parse_string method of the MPS parser. A saveptr context pointer is initialized to track tokenization state across newline-delimited line iterations.

Changes

Cohort / File(s)	Summary
Strtok Reentrancy Fix cpp/libmps_parser/src/mps_parser.cpp	Replaced strtok with strtok_r to improve thread safety. Added saveptr context pointer for managing tokenization state in the parse_string method.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: replacing strtok with thread-safe strtok_r in the MPS parser, which aligns with the primary objective of the pull request.
Description check	✅ Passed	The description is directly related to the changeset, explaining the rationale for replacing strtok with strtok_r and addressing thread-safety concerns in MPS file parsing.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[rgsl888prabhu]

/merge