Skip to content

Add C++20 support for Client#3636

Closed
G-Moris wants to merge 91 commits into
multitheftauto:masterfrom
G-Moris:client_c20
Closed

Add C++20 support for Client#3636
G-Moris wants to merge 91 commits into
multitheftauto:masterfrom
G-Moris:client_c20

Conversation

@G-Moris

@G-Moris G-Moris commented Aug 9, 2024

Copy link
Copy Markdown

С++20 for Client, my test version

@TracerDS TracerDS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You want to introduce C++20 for Client but you change server code?
You also didnt tackle any issues regarding game_sa and multiplayer_sa

Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/CCore.cpp Outdated
Comment thread Client/core/CCore.cpp Outdated
Comment thread Client/core/CEntryHistory.h
Comment thread Server/launcher/premake5.lua Outdated
Comment thread Server/mods/deathmatch/premake5.lua Outdated
Comment thread Shared/sdk/SharedUtil.Logging.hpp Outdated
Comment thread Shared/sdk/SString.h
Comment thread vendor/sparsehash/src/sparsehash/internal/hashtable-common.h
Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/CCore.cpp Outdated
Comment thread Client/core/CVersionUpdater.Util.hpp Outdated
Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/CCommands.cpp Outdated
Comment thread Client/core/ServerBrowser/CServerBrowser.cpp Outdated
Comment thread Client/core/ServerBrowser/CServerBrowser.cpp Outdated
Comment thread Client/game_sa/CWeaponSA.cpp Outdated
Comment thread Shared/sdk/SString.h
Comment thread Shared/sdk/SharedUtil.Win32Utf8FileHooks.hpp Outdated
@G-Moris

G-Moris commented Aug 25, 2024

Copy link
Copy Markdown
Author

@botder @tederis Do my corrections fit your requirements?

@G-Moris

G-Moris commented Aug 31, 2024

Copy link
Copy Markdown
Author

@Dutchman101 Hello! Sorry for the trouble and for the fact that this is not on the topic of PR, can you please unblock me in the Multi Theft Development group? I'm not going to bother anyone or prescribe something bad. It's just that in some developments I want to work together with those who want it.

@G-Moris

G-Moris commented Aug 30, 2025

Copy link
Copy Markdown
Author

I will soon create a new PR to switch game_sa and multiplayer_sa to C++23

@botder

botder commented Aug 30, 2025

Copy link
Copy Markdown
Member

There is no need for that. I am working on the code for that, including a tool for hook checking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants