Replace string concatenation with std::format()

[mikomikotaishi]

Repos I am interested in contributing to

hyprland, hyprctl, hyprpm

What I want to contribute right now

I want to move all string concatenations (except string joining through +=) to just using std::format(). For the time being, I plan to avoid changing those in the test files, as I think the gain from modifying the test files is not as profound as it is from modifying the actual source code where the benefits of code clarity is more immediate and apparent.

Why the above bug / feature specifically?

There seems to be an inconsistent usage across the codebase between using string concatenation through chaining + operators and std::format(). Using std::format() would be much cleaner than breaking up things like path or name constructions, or throwing in std::to_string() calls, and would be better to follow as a rule of thumb moving forward seeing as parts of the codebase already have it.

I acknowledge that:

• I have read the AI Usage Policy and agree to follow it.

• I have read the Code of Conduct and agree to follow it.

• I wrote this request with my own hands, my own keyboard, and in my own words.

[vaxerski]

true. Some parts still use vsnprintf. Yuck!

Good way to get yourself accustomed with the codebase(s). Welcome aboard!

[mikomikotaishi]

Thanks!