Guild icon
DDraceNetwork
Development / developer
Development discussion. Logged to https://ddnet.tw/irclogs/ Connected with DDNet's IRC channel, Matrix room and GitHub repositories — IRC: #ddnet on Quakenet | Matrix: #ddnet-developer:matrix.org GitHub: https://github.com/ddnet
Between 2022-06-03 00:00:00Z and 2022-06-04 00:00:00Z
Avatar
GitHub BOT 2022-06-03 00:04:16Z
Author icon
def-
[ddnet/ddnet] Issue reopened: #4424 can't deinitialize, connections remaining
00:04
Author icon
HiRavie
[ddnet/ddnet] Pull request opened: #5323 Fix deleting envelopes not updating sounds
Deleting envelopes can lead to sounds having an invalid envelope index.

Checklist

  • [x] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [x] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+UBSan or val...
Avatar
Avatar
Chairn
i don't know any player using dyncam
heinrich5991 2022-06-03 00:35:00Z
I use dyncam
Avatar
cauldron 2022-06-03 00:50:21Z
it is doomed on fng, i don't get why, you have more view range? so what
Avatar
Voxel 2022-06-03 01:11:51Z
it's considered to be cheating against professional players that don't use it
Avatar
Dragonwood 2022-06-03 04:04:28Z
[suggestion] add a max ping filter for the server browser + hide servers with unresolved ping (CHN etc.). Reason: Ordering after ping and then players brings alot of useless/unplayable servers to the top of the list
Avatar
Voxel 2022-06-03 04:20:56Z
there used to be a ping filter, but i don't know what happened with it
Avatar
chillerdragon BOT 2022-06-03 06:49:08Z
Why does it say „ChillerDra…“ in the hud now and not „ChillerDragon“ -.-
Avatar
nouaa 2022-06-03 06:56:24Z
Your name is blacklisted
Avatar
chillerdragon BOT 2022-06-03 07:02:56Z
._.
Avatar
GitHub BOT 2022-06-03 07:49:51Z
Author icon
ChillerDragon
[ddnet/ddnet] Issue closed: #5315 Integration test script should output asan issues on failure
Avatar
GitHub BOT 2022-06-03 08:01:10Z
Author icon
ChillerDragon
[ddnet/ddnet] Issue opened: #5324 Cleanup terminal colors on quit
Not too dramatic but would be nice to restore the colors in the terminal when client or server quit.
Avatar
[quakenet] ChillerDragon BOT 2022-06-03 08:34:25Z
its probably some unicode chopping off whatevery but i really feel like information gets lost
08:35
Embedded image
08:35
ChillerDra!
Avatar
[quakenet] MrGh0s7 BOT 2022-06-03 08:41:34Z
yeah I agree. Clipping off names isn't/wasn't a good idea
Avatar
deen 2022-06-03 08:42:05Z
Oh well, we could revert it (again)
Avatar
[quakenet] MrGh0s7 BOT 2022-06-03 08:43:00Z
I'm curious though it should have occured to you what's the other side of the coin?
Avatar
GitHub BOT 2022-06-03 08:55:43Z
Author icon
def-
[ddnet/ddnet] Pull request opened: #5325 Reset colors at end of line (fixes #5324)
@ChillerDragon Can you test this?

Checklist

  • [ ] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet...
Avatar
Avatar
Dragonwood
[suggestion] add a max ping filter for the server browser + hide servers with unresolved ping (CHN etc.). Reason: Ordering after ping and then players brings alot of useless/unplayable servers to the top of the list
heinrich5991 2022-06-03 08:57:23Z
the max ping filter was removed because it caused servers to disappear entirely if you had a bad ping on them once
08:57
because the ping is only updated when you join the servers
Avatar
GitHub BOT 2022-06-03 09:26:56Z
Author icon
heinrich5991
[ddnet/ddnet] Pull request opened: #5326 Explicitly delete game server from list on shutdown
Also delete it explicitly when sv_register is set to register via less protocols. On shutdown, wait at most 1 second for the masterserver to respond, otherwise allow up to 15 seconds. Fixes https://github.com/ddnet/ddnet/issues/5157.

Checklist

  • [x] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ]...
Avatar
[quakenet] ChillerDragon BOT 2022-06-03 09:50:09Z
heinrich when triangulate users ping to the tw server from the master server? By using maffs and ping from differnt masters to user and then from masters to the destinatin server :D
Avatar
GitHub BOT 2022-06-03 10:02:04Z
Author icon
bors[bot]
[ddnet:master] 3 new commits
ce08cc0 mastersrv: Add Action: delete to /ddnet/15/register - heinrich5991 7fccfaf Explicitly delete game server from list on shutdown - heinrich5991 2d53aea Merge #5326 - bors[bot]
10:02
Author icon
bors[bot]
[ddnet/ddnet] Issue closed: #5157 HTTP master: Server should send delete message when shutting down gracefully
10:02
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5326 Explicitly delete game server from list on shutdown
Avatar
GitHub BOT 2022-06-03 10:11:59Z
Author icon
def-
[ddnet/ddnet] Pull request opened: #5327 Add CLockScope
Remove unused lock and scope in threading.h Not sure if this is preferred We could also try switching to std::mutex

Checklist

  • [x] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect existing map...
Avatar
Avatar
[quakenet] ChillerDragon
its probably some unicode chopping off whatevery but i really feel like information gets lost
c0d3d3v 2022-06-03 10:35:43Z
have a time and your name is fully displayed
10:36
we could not clip names in mini score hud, if that would be prefered (edited)
Avatar
heinrich5991 2022-06-03 10:36:23Z
they were probably displayed completely there before?
Avatar
c0d3d3v 2022-06-03 10:36:40Z
before the name overflowed to the left (edited)
10:37
now we let the name begin to the left of the score box
10:37
and clip it on the right side
Avatar
Avatar
c0d3d3v
before the name overflowed to the left (edited)
heinrich5991 2022-06-03 10:38:09Z
ah, I think that overflow was intentional
10:38
there wasn't a visible box that it overflowed, it just naturally extended to the left 🙂
Avatar
c0d3d3v 2022-06-03 10:40:45Z
It is still kind of strange that it overflows... does not fit to the score boxes... but yes we could revert the clipping for the mini score hud.
Avatar
heinrich5991 2022-06-03 10:45:02Z
you mean the box in the code? visually, there isn't really a box
10:45
visually it looks like it's supposed to extend to the left, for me
Avatar
GitHub BOT 2022-06-03 10:47:04Z
Author icon
bors[bot]
[ddnet/ddnet] Issue closed: #5324 Cleanup terminal colors on quit
10:47
Author icon
bors[bot]
[ddnet:master] 2 new commits
129205d Reset colors at end of line (fixes #5324) - def- 6b7554f Merge #5325 - bors[bot]
10:47
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5325 Reset colors at end of line (fixes #5324)
Avatar
GitHub BOT 2022-06-03 10:55:56Z
Author icon
def-
[ddnet/ddnet] Pull request opened: #5328 [WIP] Attempt to use std::mutex

Checklist

  • [ ] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--u...
Avatar
GitHub BOT 2022-06-03 11:17:57Z
Author icon
bors[bot]
[ddnet:master] 2 new commits
51909c1 Add CLockScope - def- ce79557 Merge #5327 - bors[bot]
11:17
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5327 Add CLockScope
Avatar
Avatar
heinrich5991
visually it looks like it's supposed to extend to the left, for me
c0d3d3v 2022-06-03 11:47:13Z
I mean this box, since they are part of the same context... Even better would be to make the rounded rectangle around the name too.
Image attachment
11:47
yes it is only a context box that is not drawn... but it is the same context
Avatar
heinrich5991 2022-06-03 11:49:20Z
yes, I realize it's in the code
11:49
but the visuals look like it's intentional that there's no red box like you drew
11:49
to me, at least
11:49
that it's intended to overflow to the left
Avatar
GitHub BOT 2022-06-03 11:58:37Z
Author icon
C0D3D3V
[ddnet/ddnet] Pull request opened: #5329 Let the names in the mini score hud overflow to the left

Checklist

  • [ ] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [x] Considered possible null pointers and out of bounds array indexing
  • [x] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--und...
Avatar
Avatar
[quakenet] ChillerDragon
Click to see attachment 🖼️
c0d3d3v 2022-06-03 12:21:43Z
@ChillerDragon is this a team server with point counter?
Avatar
GitHub BOT 2022-06-03 13:07:26Z
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5329 Let the names in the mini score hud overflow to the left
13:08
Author icon
def-
[ddnet/ddnet] Pull request opened: #5330 Actually fix capital /WHISPER
As reported by kiwq

Checklist

  • [x] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#us...
Avatar
[quakenet] ChillerDragon BOT 2022-06-03 13:32:41Z
there is vanilla score not ddrace but there are no teams as u can see by green instead of red dragon :)
Avatar
GitHub BOT 2022-06-03 13:58:14Z
Author icon
Robyt3
[ddnet/ddnet] Pull request opened: #5331 Pass OVERLAPPED parameter to WriteFile according to specification
Maybe closes #5322. I can't reproduce the crash. Can you try if this fixes it, @Chairn?

Checklist

  • [ ] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect existing maps
  • [ ] Tested the change with [ASan+...
13:58
Author icon
Robyt3
[ddnet/ddnet] Pull request opened: #5332 Pass output parameter to WriteFile that is required on Windows 7
Maybe closes #5322. I can't reproduce the crash. Can you try if this fixes it, @Chairn? Looks like Windows 7 needs this parameter.

Checklist

  • [ ] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [ ] Considered possible null pointers and out of bounds array indexing
  • [ ] Changed no physics that affect exis...
Avatar
GitHub BOT 2022-06-03 15:14:39Z
Author icon
bors[bot]
[ddnet:master] 2 new commits
9807625 Actually fix capital /WHISPER - def- 4ead727 Merge #5330 - bors[bot]
15:14
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5330 Actually fix capital /WHISPER
15:16
Author icon
def-
[ddnet-libs:pr-drmingw-win7-mingw] 1 new commit
ecf45cb Build our own drmingw with mingw - def-
15:16
Author icon
def-
[ddnet/ddnet-libs] New branch created: pr-drmingw-win7-mingw
15:16
Author icon
def-
[ddnet/ddnet-libs] Pull request opened: #28 Build our own drmingw with mingw
Get WinSDKDebuggingTools compatible with Windows 7: https://www.microsoft.com/en-us/download/details.aspx?id=8279&ranMID=46131&ranEAID=a1LgFw09t88&ranSiteID=a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw&epi=a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw&irgwc=1&OCID=AID2200057_aff_7806_1243925&tduid=%28ir__qxvdeeyzhskf6zzy0axab6w3ye2xvkxn3xgwkihq00%29%287806%29%281243925%29%28a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw%29%28%29&irclickid=_qxvdeeyzhskf6zzy0axab6w3ye2xvkxn3xgwkihq00
Avatar
deen 2022-06-03 15:18:24Z
@Jupstar ✪ Do I still need your patch? https://github.com/Jupeyy/drmingw/commit/08ab91c4897c04b5919d14fc2d7c21
Print entry · Jupeyy/drmingw@08ab91c
Thumbnail
Avatar
Jupstar ✪ 2022-06-03 15:19:46Z
yes, tho currently we build without PIE
Avatar
deen 2022-06-03 15:19:55Z
ok
Avatar
Jupstar ✪ 2022-06-03 15:20:01Z
but the parser already relies on it xd
15:20
could also extend it to print the client/server version 😄
Avatar
deen 2022-06-03 15:20:35Z
/home/deen/isos/ddnet/win64/drmingw/tests/test_exchndl.h: In function ‘int main(int, char**)’: /home/deen/isos/ddnet/win64/drmingw/tests/test_exchndl.h:90:16: error: too few arguments to function ‘void ExcHndlInit(void*)’ 90 | ExcHndlInit(); | ~~~~~~~~~~~^~ Doesn't build anymore
Avatar
Jupstar ✪ 2022-06-03 15:20:55Z
there are 2 patches actually
15:21
https://github.com/Jupeyy/drmingw/commit/c04387280fa3c33e70cc083ff664edf07192278f
ddnet-offset · Jupeyy/drmingw@c043872
Thumbnail
15:21
https://github.com/Jupeyy/drmingw/commit/e141208d915c3c107d037b06f28d307937121a49
Print entry · Jupeyy/drmingw@e141208
Thumbnail
15:21
mh ok
15:21
first is just disabling stuff
Avatar
deen 2022-06-03 15:22:00Z
ok
Avatar
GitHub BOT 2022-06-03 15:23:54Z
Author icon
def-
[ddnet-libs:pr-drmingw-win7-mingw] 1 new commit
9138d1e Build our own drmingw with mingw - def-
Avatar
Jupstar ✪ 2022-06-03 15:24:30Z
now that i think about it @deen
Avatar
GitHub BOT 2022-06-03 15:24:31Z
Author icon
def-
[ddnet/ddnet] Pull request opened: #5333 Build our own drmingw with mingw
Get WinSDKDebuggingTools compatible with Windows 7: https://www.microsoft.com/en-us/download/details.aspx?id=8279&ranMID=46131&ranEAID=a1LgFw09t88&ranSiteID=a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw&epi=a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw&irgwc=1&OCID=AID2200057_aff_7806_1243925&tduid=%28ir__qxvdeeyzhskf6zzy0axab6w3ye2xvkxn3xgwkihq00%29%287806%29%281243925%29%28a1LgFw09t88-hqwNB0ABR6xeVqDZwkGZAw%29%28%29&irclickid=_qxvdeeyzhskf6zzy0axab6w3ye2xvkxn3xgwkihq00

Checklist

  • [ ] Tested the ch...
Avatar
Jupstar ✪ 2022-06-03 15:24:39Z
i think LTO might have optimized the code lately
15:24
remember the issue with the "bad" function calls
15:25
i assume LTO optmizes it too much
Avatar
deen 2022-06-03 15:25:43Z
yes, sounds plausible
15:25
Like I said, everything breaks 😄
Avatar
Jupstar ✪ 2022-06-03 15:26:45Z
drmingw wasnt that helpful yet 😄 some kind of ubsan would probs be nicer, but dunno if the gcc ubsan trap has "big" runtime overhead
Avatar
deen 2022-06-03 15:26:49Z
would be cool to test a crash with and without LTO, maybe we should disable it
15:27
You are not supposed to ship/productively use asan/ubsan instrumented code
Avatar
Jupstar ✪ 2022-06-03 15:27:55Z
well the gcc flag is added to not ship ubsan as a library
15:28
https://gcc.gnu.org/onlinedocs/gcc/Instrumentation-Options.html#:~:text=The%20%2Dfsanitize%2Dundefined%2Dtrap,usable%20even%20in%20freestanding%20environments
Instrumentation Options (Using the GNU Compiler Collection (GCC))
Instrumentation Options (Using the GNU Compiler Collection (GCC))
15:29
if it doesnt have huge runtime overhead its just like some kind of assert i guess
15:29
it generates an invalid instruction
Avatar
GitHub BOT 2022-06-03 15:37:14Z
Author icon
def-
[ddnet-scripts:master] 1 new commit
e40623b Add drmingw instructions - def-
Avatar
GitHub BOT 2022-06-03 16:28:53Z
Author icon
C0D3D3V
[ddnet/ddnet] Pull request opened: #5334 fix pixel porridge in freeze and ninja bar

Checklist

  • [x] Tested the change ingame
  • [ ] Provided screenshots if it is a visual change
  • [ ] Tested in combination with possibly related configuration options
  • [ ] Written a unit test if it works standalone, system.c especially
  • [x] Considered possible null pointers and out of bounds array indexing
  • [x] Changed no physics that affect existing maps
  • [x] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--u...
Avatar
Chairn 2022-06-03 16:44:17Z
when you finish coding for ddnet https://www.reddit.com/r/ProgrammerHumor/comments/v3xdyp/to_refresh_my_mind_body_and_soul/
r/ProgrammerHumor - To refresh my mind, body and soul
9,301 votes and 100 comments so far on Reddit
Thumbnail
Avatar
cauldron 2022-06-03 16:48:41Z
git for map editor when
Avatar
GitHub BOT 2022-06-03 16:53:16Z
Author icon
bors[bot]
[ddnet:master] 2 new commits
c33e0c1 fix pixel porridge in freeze and ninja bar - C0D3D3V 36d0c0d Merge #5334 - bors[bot]
16:53
Author icon
bors[bot]
[ddnet/ddnet] Issue closed: #5319 still fragments in freezebar
16:53
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5334 fix pixel porridge in freeze and ninja bar
Avatar
Avatar
cauldron
git for map editor when
c0d3d3v 2022-06-03 17:29:21Z
vanilla has already a editor history 😄 you could try to merge it with our editor
Avatar
cauldron 2022-06-03 17:50:19Z
i thought of it, because i read something here about colaborative map editing.
Avatar
Avatar
c0d3d3v
vanilla has already a editor history 😄 you could try to merge it with our editor
cauldron 2022-06-03 17:51:11Z
thats nice are the code bases far apart?
Avatar
c0d3d3v 2022-06-03 17:51:44Z
they said they made it new from scratch so probably
17:52
https://github.com/teeworlds/teeworlds/tree/editor
GitHub - teeworlds/teeworlds at editor
A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.
Thumbnail
Avatar
Avatar
c0d3d3v
https://github.com/teeworlds/teeworlds/tree/editor
cauldron 2022-06-03 17:52:52Z
the history feature or remade the whole editor?
17:53
from scratch
Avatar
c0d3d3v 2022-06-03 17:53:27Z
see https://github.com/teeworlds/teeworlds/issues/2486
Editor v2: Project and Contributing · Issue #2486 · teeworlds/teewo...
I've been working on remaking the editor for quite some time now and we recently decided to make a public branch to make it easier to follow and contribute to its development. The Project A...
Thumbnail
Avatar
cauldron 2022-06-03 17:57:22Z
i thought before ddnet is kind of synced or based or whatever on newest teeworlds vanilla + ddnet code 😅 im funny
Avatar
c0d3d3v 2022-06-03 17:58:07Z
https://www.youtube.com/watch?v=sCs3a3Uz8cw
LordSk
Teeworlds: brush automap
Avatar
Avatar
cauldron
i thought before ddnet is kind of synced or based or whatever on newest teeworlds vanilla + ddnet code 😅 im funny
c0d3d3v 2022-06-03 17:58:32Z
On our facebook page... I would say our relationship is "complicated"
Avatar
cauldron 2022-06-03 18:00:14Z
i guess it's more apart than i thought. like the 0.7 what i dont knoebabout
18:01
anyway ill try out the vanilla editor.
18:02
and ckeck out facebook
Avatar
c0d3d3v 2022-06-03 18:03:07Z
🧐 (edited)
18:03
I meant that with facebook as a joke (I have not used facebook since ages) (edited)
Avatar
cauldron 2022-06-03 18:06:06Z
for ddnet: ddnet insta when xd
18:07
😅
Avatar
n000b 2022-06-03 19:52:57Z
Yeah it sad the dev time spend is not on the same client. It now tw vs ddnet and everyone does his own things.
19:53
Was always wondering about clients based on ddnet with extra features. Why don't u just contribute to the original client
Avatar
cauldron 2022-06-03 19:58:51Z
ill keep my mouth shut about politics
20:04
i mean i am a dev, but i am not contributing. 1 i am very unproductive, the only coding i did recently was projecteuler.net math problems xd 2 i did this easy team colors feature, but wont just create a pr. but im just not getting into the other stuff. im not c++ savvy so refactoring or enhancing / expanding features is not an option.
20:05
so ill just spam this channel 😅
Avatar
saminho 2022-06-03 20:27:21Z
we should be able to add time checkpoints by ourselves to optimise our speedruns ^^
Avatar
GitHub BOT 2022-06-03 20:30:20Z
Author icon
hus3h
[ddnet/ddnet] Pull request opened: #5335 Unfreeze character after teleporting in practice
From my experience and seeing others play, this would be more convenient. Currently teleporting (while being frozen) to retry a part that doesn't have much space or a platform to stand on is a bit hard, often requiring the player to teleport quickly to a position in air multiple times till the character is unfrozen or teleporting to a different part till the character is ready to be put in the original part again. This can also save some time when practicing in general, and will make /tp feel...
Avatar
GitHub BOT 2022-06-03 20:38:03Z
Author icon
MEmileTW
[ddnet/ddnet] Issue opened: #5336 Client side TimeCP entities
It would be nice if we had the option of adding timeCP's on any map you wanted even if it was client-side. I asked players who speedrun maps regularly and they all said they would like being able to add them on the maps they're trying to speedrun.
Avatar
GitHub BOT 2022-06-03 22:02:29Z
Author icon
bors[bot]
[ddnet:master] 2 new commits
db52931 Unfreeze character after teleporting in practice - hus3h 926d158 Merge #5335 - bors[bot]
22:02
Author icon
bors[bot]
[ddnet/ddnet] Pull request closed: #5335 Unfreeze character after teleporting in practice
Exported 124 message(s)