Skip to content

2026PKUCourseHW5: Replace fixed-size char array with std::string#7138

Open
GenZeal-Lin wants to merge 3 commits intodeepmodeling:developfrom
GenZeal-Lin:hw5-case8-string
Open

2026PKUCourseHW5: Replace fixed-size char array with std::string#7138
GenZeal-Lin wants to merge 3 commits intodeepmodeling:developfrom
GenZeal-Lin:hw5-case8-string

Conversation

@GenZeal-Lin
Copy link

Unit Tests and/or Case Tests for my changes

  • No new unit test was added.
  • The modified code path was manually reviewed.
  • The change replaces a fixed-size character buffer with std::string to avoid potential buffer overflow risk.

What's changed?

  • Replace a fixed-size char array used for string input with std::string.
  • Keep the original logic unchanged while making the string handling safer and clearer.

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