Skip to content

Conversation

@teeworlds-mirror
Copy link
Collaborator

ViMaSter and others added 30 commits December 3, 2022 17:48
…_space_in_systemc

Remove trailing space in system.c teeworlds#3220
…unused

Mark unused NETMSG_SNAPSMALL for 0.8 removal teeworlds#3210
When a new player joins it is being sent all the client info of the
currently connected tees. This is marked as non silent which is
misleading because the client does not actually print these as join
messages. The reason why the client does not print it is because it does
not have its m_LocalClientID set yet.

Setting it explicitly to silent makes it more obvious that this is not
printing a chat message when reading the server code.

This change will not change anything that can be witnessed by end users.

This also ensures that when the client implementation changes it does
not start printing unexpected "joined the game" messages in chat.
https://github.com/teeworlds/teeworlds/blob/a1911c8f7d8458fb4076ef8e7651e8ef5e91ab3e/datasrc/network.py#L253

NetEvent("Damage:Common", [ # Unused yet
	NetIntRange("m_ClientID", 0, 'MAX_CLIENTS-1'),
	NetIntAny("m_Angle"),
	NetIntRange("m_HealthAmount", 0, 9),
	NetIntRange("m_ArmorAmount", 0, 9),
	NetBool("m_Self"),
]),

Snap items with a m_ClientID out of range will be dropped during snap item validation.
…filenames

Port str_comp_filenames from ddnet to fix buffer overflow teeworlds#3219
Fixed cursor offset in the editor and removed brush display when picking tiles teeworlds#3222
Add CJsonParser, revise io_read_all/io_read_all_str teeworlds#3141
Polyfill distutils.copy_tree because it was removed teeworlds#3229
It's overflowing easily, especially considering colorcodes.
fixes

	$ python ./scripts/cmd5.py
	/Users/chillerdragon/Desktop/git/teeworlds/./scripts/cmd5.py:11: SyntaxWarning: invalid escape sequence '\/'
	  d = re.sub("\/\*.*?\*/".encode(), "".encode(), d) # remove /* */ comments
	#define GAME_NETVERSION_HASH "e9800998ecf8427e"
…llerDragon_pr_cmd5_fix_esc_seq_warning

Fix cmd5.py SyntaxWarning: invalid escape sequence teeworlds#3225
tempral
tempral previously approved these changes Apr 9, 2024
@ChillerDragon ChillerDragon dismissed tempral’s stale review August 20, 2024 06:31

The merge-base changed after approval.

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.