Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

vegasettings: replace C-style strings and string operations with std::string and boost::format #996

Open
stephengtuggy opened this issue Jan 16, 2025 · 3 comments · Fixed by #999
Open

vegasettings: replace C-style strings and string operations with std::string and boost::format #996

stephengtuggy opened this issue Jan 16, 2025 · 3 comments · Fixed by #999
Assignees
@stephengtuggy
Labels
enhancement
Milestone
0.10.x

Comments

@stephengtuggy
Copy link
Contributor 

          If you're going to touch code like this, please convert it std::string. There's no reason to keep using C in C++ projects.

Originally posted by @royfalk in #974 (comment)
@royfalk
Copy link
Contributor

royfalk commented Jan 16, 2025

I did a quick search of printf and got 905 results in 105 files. So that's going to take a bit of work...

@stephengtuggy
Copy link
Contributor Author

@royfalk Yep. That it will.

@stephengtuggy stephengtuggy self-assigned this Jan 17, 2025
@stephengtuggy stephengtuggy added this to the 0.10.x milestone Jan 17, 2025
@stephengtuggy stephengtuggy mentioned this issue Jan 18, 2025
Merged
1 task
@stephengtuggy
Copy link
Contributor Author

Not quite finished.

@stephengtuggy stephengtuggy reopened this Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stephengtuggy stephengtuggy

Labels
enhancement
Projects
None yet
Milestone
0.10.x
2 participants
@royfalk @stephengtuggy