DDraceNetwork - developer

DDraceNetwork

Development / developer

Development discussion. Logged to https://ddnet.org/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 2024-01-14 00:00:00Z and 2024-01-15 00:00:00Z

What’s cowrie but no I did not use any tools or libs. It’s pure self written typescript

Replying to @Cipy29 ChillerDragon did you use cowrie for the discord shell bot?

02:08

@TsFreddie: good morning c:

chillerdragon

@TsFreddie: good morning c:

why

Aren’t u in utc+8 ?

$Image attachment$

Sorry if I woke you or something Fred ._.

02:31

Just wanted to know if we are timezone buddies now

owo

03:07

did you go to australia or something

Indonesia

for?

So wats ur timezone

+8

Secret

03:08

So we timezone buddies uwu

HUH

I will ping you every morning!

are you like just there now?

Yea

~~did germany kick you out~~

Yes I was too edgy

kek

03:10

imma take a nap actually

03:10

woke up at 3am, it wasn't great

Bruv

03:11

I was still awake at that time

03:11

Jet lag moment I guess

HammyShock

chillerdragon

Yes I was too edgy

Agreed, we germans abandoned irc enjoyers (except scrumplex because He cool)

Don’t you have to go to work or something?

me?

03:11

no

Wowo

why would i go to work on sunday

Oh

03:12

I totally knew it was Sunday

HUH

But then you are in china

03:13

Do you even do weekends?

Catxplosion

we do

woah lazy

HammyShock

Ewan

he is using those things they are just a weakness of his

yeh, i facing my weaknesses like batman

true

it's 7:30 in the morning here

9 pm

left or right

04:17

Image attachment

bro, reading and writing to a file is so faking fast, i never tougth i could read, delete, and write again to a file every single tick

04:25

without a single mistake

i wonder how fast it's gonna be if i use hard drive instead of ssd

<ChillerDragon> ye writin smol files is fast af

04:50

<ChillerDragon> faster than priting to the terminal iirc

04:50

<ChillerDragon> i think i boosted fps in a terminal based game once by instead of printing writing to a file and then using cat to print the entire file :D

1

04:51

<ChillerDragon> buy chillerdragons bash game engine today

04:52

<ChillerDragon> https://zillyhuhn.com/cs/.1705207908.png

Embedded image

04:52

<ChillerDragon> this looks wrong. Jopsti should one highlite him self?

04:52

<ChillerDragon> i wonder if i could have replaced "him" with something genderless in that sentence

-21 out tonight

04:53

feelsbadman

<ChillerDragon> i envy u

04:54

<ChillerDragon> cold is pog

04:54

<ChillerDragon> for me it doesnt get much colder than 28 durin da nite i think

04:55

<ChillerDragon> @Ewan fahrenheit?

c

<ChillerDragon> o.O

i got yelled at for using freedom units last time

<ChillerDragon> xd

04:55

<ChillerDragon> freedom units

04:55

<ChillerDragon> watfak

murica

<ChillerDragon> i think i never got to xp -21

04:56

<ChillerDragon> sounds fun

04:56

<ChillerDragon> i didnt know its that cold in murica

04:56

<ChillerDragon> time to move

04:57

<ChillerDragon> ewan u eng pro

04:57

<ChillerDragon> send woke generless sentence

04:57

<ChillerDragon> > Jopsi should one highlite him self?

04:58

<ChillerDragon> how can i replace "him" with something genderless like "one" @Ewan ?

04:58

<ChillerDragon> or how to make the sentence cool in general

04:59

<ChillerDragon> apparently "oneself" is a thing xd

what context

04:59

yea

04:59

themselves

<ChillerDragon> the context i used it in

05:00

<ChillerDragon> oke thank

should one highlight themselves

<ChillerDragon> i put tripple A english on my CV now

lol

[ddnet-maps:master] 1 new commit

f17401f A Lonely Travel, M Purple Panic, M Depressed II, M The Shire, M Halloween Night, M Harvest, M Kobra 2, M Kobra, M Multeasymap, M Sunny Side Up - ddnet-maps

<ChillerDragon> Okay thats kinda cool you can close issues in other repos with commit messages

06:15

<ChillerDragon> https://zillyhuhn.com/cs/.1705212891.png

Embedded image

06:15

<ChillerDragon> can codeberg do that too? -.-

ws-client

<ChillerDragon> i didnt know its that cold in murica

it's huge country

06:47

it just depends on where you are

ah man

06:51

i run a server on linux, people can register and make an account, the physical file of that account is made but it can not be readed from the server once the player tries to "/login username password"

06:52

and it's working on windows with the same code

06:52

i run the server as root, but still having the same problem

06:53

and once i restarting the server, you can /login with previously register accounts (edited)

06:54

i gave the DDnet-Server chmod +rwx but still the same

06:55

also i can give you my last commit link if you want

never mind, the code is having problem, in windows too

07:06

the problem was stupid as, not closing the file: io_close() troll

troll

Classic

ws-client

<ChillerDragon> this looks wrong. Jopsti should one highlite him self?

No

ye dats wat i thought ima open an issue and will probably not fix before end of 2028

<ChillerDragon> did you know the word procrastination includes pro?

with improved knowledge from work im retaking edlang

i cant tell if ddnet singapore servers lagg or my wifi idk how to test

ChillerDragon

@Meeu when meet?

when

08:50

what's your ping to Singapore?

37

08:51

it feels smoother than at home haha

08:51

but i have lag spikes that make it super faily to play

XD good

idk if its my connection or the server

CHN8 has good ping too from here

i have 78 in chn8

oh I have 60ish

jumps to 90 ye its rly bad xd

probably your wifi then

meh

08:54

sucks to be me

08:55

shiet mobile data is even way worse

hi meeu

09:47

[ddnet-web:pr-18.0] 1 new commit

3d7bf19 Version 18.0 - def-

how is singapore fellas

[ddnet-web:pr-18.0] 1 new commit

a264d42 Version 18.0 - def-

hi Ewan o/

10:00

the singapore server is empty most of the time XD

damn

i saw chiller but he was afk earlier

ah

10:00

i wonder why he is there

10:02

how are u

sad cuz bad ping to USA :c

10:02

it's not so bad to USA2 ig, 200 ping

ah messed up

[ddnet/ddnet] Pull request opened: #7801 Restore ability to move points of all selected quads

I think this broke some time ago, currently we can no longer move the points of all selected quads when moving any points, it only applies to the selected quad. I don't think this was intended, which is why I made this PR to restore that functionality. It also works with quad points alignment/snapping that I made some time ago as well. Before: https://github.com/ddnet/ddnet/assets/13364635/652957ae-b848-4e88-8313-2033b08c51f4 After: https://github.com/ddnet/ddnet/assets/1336463...

What’s your in game name?

Replying to @Meeu i saw chiller but he was afk earlier

10:39

Best ping ddnet server

Replying to @Ewan i wonder why he is there

chillerdragon

What’s your in game name?

Meeu

chillerdragon

Best ping ddnet server

Lol

[ddnet/ddnet] Pull request closed: #7801 Restore ability to move/edit points of all selected quads

11:36

github-merge-queue[bot]

[ddnet:master] 2 new commits

a3115d7 Restore ability to move points of all selected quads - archimede67 6ad92ed Merge pull request #7801 from archimede67/editor-fix-quads-edit - def-

I seriously do not understand MSVC

11:49

Accessing an unscoped enumeration that's a member of a class through a fully qualified name is legal since C++11

11:51

It should even be part of the outside scope, so should let me refer to it using just the class too...

oh it's not the scoping that's the issue, it's that stray constant...

Damn

Let's just drop support for MSVC, no self respecting developer uses it anyway

12:10

Ok, I can't do this without a windows machine, can one of you find out where this constant is coming from?

Ach

https://godbolt.org/z/K1nbbno9K this is what's happening, so you are looking for a #define ERROR that only happens on windows

Compiler Explorer - C++ (x64 msvc v19.37)

class CHttp { public: enum EState { UNINIT, RUNNING, ERROR }; }; #define RUNNING 10 int a = CHttp::RUNNING; int b = CHttp::EState::RUNNING;

what’s the constant

Or maybe I should just rename my error state to something like FAILED

12:12

It might be easier

Ewan

what’s the constant

I have no idea where it's coming from https://github.com/ddnet/ddnet/pull/7683 but somehow after all the includes in http.cpp there is ERROR #defined

Use curl-multi. Supersedes #5842 by Learath2 · Pull Request #7683 ·...

This rebases and partially rewrites #5842. Tested very roughly I still am not the biggest fan but this is the best I can do without going far too deep into a jobs rewrite: This iteration uses the ...

12:13

Or maybe I can just do #ifdef ERROR #undef ERROR after the include

[ddnet/ddnet] Issue opened: #7802 [Assertion Error] Demo recorder alreay recording

Learath2

Let's just drop support for MSVC, no self respecting developer uses it anyway

before that let's drop support for macos xd

this

1

No macos > msvc

12:22

At least it’s unix

12:23

xcode is the poggest of them all

[ddnet/ddnet-discordbot] Pull request closed: #72 New Trial Tester role, remove testing lead role

14:09

[ddnet-discordbot:master] 2 new commits

c8ed6e1 New Trial Tester role, remove testing lead role - murpii 13fe202 Merge pull request #72 from murpii/pr_mt_role_changes - Learath2

@heinrich5991 what's the main reason of using warn instead of using Result<_, _>?

@gerdoe warn is for inconsistencies / oddities which should not cause the operation to fail, but can be logged

yeaah but warn goes trhu aall the codebase of libtw2

14:14

looks monstrous just for logging

Patiga

@gerdoe warn is for inconsistencies / oddities which should not cause the operation to fail, but can be logged

btw, inconsistencies and oddities are the ones that should not occur?

yes

14:19

for example, in gamenet, snap object decoding warns whenever there is excess data

14:20

excess data is not an huge issue, since we expect backwards compatibility by future extensions

14:20

but this does indicate that something should be updated

14:20

or fixed elsewhere

but what about using result + error handling?

14:21

i just don't get the point

because then the entire operation would unnecessarily fail

no why

14:22

ah

14:22

okey i get it xd

nice :)

14:23

its kinda weird to work with at first

14:23

and its an additional parameter everywhere, which can be a bit annoying at times ^^

yeah like every function signature every function call has this warn field

14:24

also callbacks

yea

14:24

its kinda like logging, but with a very different interface

c-way in rust

hm, logging is also quite common in rust

have never seen such approach tho

14:27

having log! is cleaner imho

14:28

repo is quite old maybe that's the reason

gerdoe

but what about using result + error handling?

do the "it must exist" extremist path every unwrap is a potential logic bug (edited)

no unwraps only match 8)

every error u must handle will introduce more potential to do it wrong

Learath2

https://godbolt.org/z/K1nbbno9K this is what's happening, so you are looking for a #define ERROR that only happens on windows

the code also fails to compile with clang and gcc

15:18

the macro is replaced in both case

Chairn

the code also fails to compile with clang and gcc

Yes, I hadn’t noticed the define was the issue

15:19

The define only happens in windows, thus I thought it was a msvc issue

who the hell defines an ERROR macro ?

15:19

maybe you can try the MSVC equivalent -E flag to see where it comes from

Learath2

I have no idea where it's coming from https://github.com/ddnet/ddnet/pull/7683 but somehow after all the includes in http.cpp there is ERROR #defined

29.86 KB

15:44

so yeah it's curl.h that includes windows.h

15:45

you don't wanna know how many hours of my life were wasted debugging some cursed windows-only issue that turned out to just be due to windows.h defining some weird macro

Jupstar ✪

every error u must handle will introduce more potential to do it wrong

so better to panic you mean lol

[ddnet/ddnet] Pull request opened: #7803 Fix multiple editor undo/redo issues

Fixed crash when runing tool 'Remove unused envelopes' and trying to undo the addition of an envelope (fixes #7738). Fixed issues where undo/redo would not work properly when changing a layer's order or group property. Added missing action for changing the position of a sound source. I don't know if there are any more problems, so feel free to open an issue when you encounter a crash/unexpected behavior while using the feature.

Checklist

[x] Tested the change ingame
[ ] Pro...

gerdoe

so better to panic you mean lol

hard to say, but sometimes i like it more than creating long unhandled err chains, or handle errors everywhere

15:53

in code where you dont expect errors, your error handling will probably suck

15:53

xd

tell me a feature u would like in edlang

Ryozuki

tell me a feature u would like in edlang

hiarc

terminating pc

i need a nice logo

avx2 support

gerdoe

avx2 support

u can do that on any systems lang

15:56

arch intrinsics

Jupstar ✪

hiarc

does this feature fit on a language feature?

15:56

not a library i mean

Ryozuki

does this feature fit on a language feature?

imo yes, i abuse it across half of my code base already

i need to find some new papers on borrow checkers

it's basically the alternative to borrow checking

15:57

and u need RAII

hmm

bcs i am also huge RAII abuser

how would u call edlang's cargo?

walkstop

xd

car go walk stop xd

15:58

epic

lmao

15:59

ed is like a ancient editor right

Ryozuki

ed is like a ancient editor right

really?

gats

Yea edge case error paths are often bugged them selfs. Because they were never tested.

Replying to @Jupstar ✪ every error u must handle will introduce more potential to do it wrong

[ddnet/ddnet] Pull request closed: #7799 Update translations for upcoming DDNet 18.0

17:39

github-merge-queue[bot]

[ddnet:master] 3 new commits

a9b0787 Update translations for upcoming DDNet 18.0 - def- 835c759 Update German translations - def- 11c1957 Merge pull request #7799 from def-/pr-lang-18.0 - Robyt3

[ddnet/ddnet] Pull request opened: #7804 Update Swedish translations for 18.0

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 (especially base/) or added coverage to integration test
[ ] 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-ad...

is this two conditions are equal ?

if(pPlayer->GetCharacter() == 0)

if(pPlayer->GetCharacter())

if my cpp knowledge doesn't fail me they are the opposite ^^

justatest

1

i think they are the same if you use it in an if,

18:51

it, in an if

18:52

bro

if(pPlayer->GetCharacter() == 1) should be the same as if(pPlayer->GetCharacter()). Can't you just test it anyways?

i testing the people here lol

lol

0 gets evaluated to false while any other int is true

is there a function to get the ClientID by the players name ?

რილია

is there a function to get the ClientID by the players name ?

There isn't a function for it, you will have to loop over all players and check if the name matches. But I guess it would be pretty easy to extract something like this to a function.

char aPlayerName[MAX_NAME_LENGTH] = "nameless tee";
int ClientID = -1;
for(int i = 0; i < MAX_CLIENTS; i++)
{
    if(str_comp(aPlayerName, Server()->ClientName(i)) == 0)
    {
        ClientID = i;
        break;
    }
}

ClientID will be -1 if a player named nameless tee wasn't found.

1

Devinci

if(pPlayer->GetCharacter() == 1) should be the same as if(pPlayer->GetCharacter()). Can't you just test it anyways?

Lol, that's hella wrong. if (pPlayer->GetCharacter() != 0) is the same as if (pPlayer->GetCharacter() but definitely not the opposite using 1 is true xD

yeah... that's what I said?

Ok, I won't argue with you. You're a troll

Are you sure you are not hating on me

21:17

no but fr

21:17

that's exactly what I said

Ok

21:18

You sure you aren't victimizing yourself?

Can someone not biased tell me if I'm actually dumbing hard or just fokko trolling me?

Like last time

21:18

Maybe just learn to read

21:18

And not victimizing yourself

21:18

that's very toxic

21:19

I told you last time even directly that I was not hating. But you're dumbing hard again

21:19

And u victimized urself last time too by not accepting my statement :)

Devinci

if(pPlayer->GetCharacter() == 1) should be the same as if(pPlayer->GetCharacter()). Can't you just test it anyways?

that's right though

thanks

fokkonaut being fokkonaut

@Nouaa no wonder you want to steal code. Too dumb to code on your own

21:22

I'll explain.

21:23

GetCharacter() does not return 1 if it exists. It's a pointer, and it points to a memory location. So when you take it as an integer, it'll not be 1. That would imply all characters point to the same memory

21:23

that's why you check != 0

atleast it should return a truthy value if it does exist

aka != 0

21:24

And not == 1

21:24

As both of you seemed to agree on

21:25

Oh boi...

I assumed it wasn't necessarily a pointer, but yeah u're right

Devinci

Can someone not biased tell me if I'm actually dumbing hard or just fokko trolling me?

@Devinci So I think you're the one that is biased. Work on yourself

21:26

DDNet hiring anyone these days

Devinci

if(pPlayer->GetCharacter() == 1) should be the same as if(pPlayer->GetCharacter()). Can't you just test it anyways?

it wont work because the pointer is probs implicitly casted to a int, and a pointer is like a numeric address

21:26

and a lower address like 0x1 is probs not what u will get

21:27

if(pPlayer->GetCharacter())

21:27

this works because

21:27

the address is not 0

21:27

so its a truthy value

all values > 0 are truthy

1

21:27

if(pPlayer->GetCharacter() > 0)

Ryozuki

all values > 0 are truthy

And < 0, but that doesn't matter in this context

fokkonaut

And < 0, but that doesn't matter in this context

true i forgot about negatives xd

xd

anyway its wrong to compare a pointer to a number

21:28

for lot of complex reasons

21:28

do if(pPlayer->GetCharacter()) or if(pPlayer->GetCharacter() != nullptr)

That aswell

21:28

Does nullptr actually work when the DDNet code sets it to 0 or NULL?

yeah

Nice

btw

21:29

0 is the nullptr address on the default address space

hello world poggers2

poggers2

Image attachment

there can be address spaces where a adress at 0 is valid

Ryozuki

0 is the nullptr address on the default address space

Ahhh

but thats another talk

I see

21:29

ty

An address space is a fundamental part of the type of a pointer value and the type of operations that manipulate memory. LLVM affords a default address space (numbered zero) and places a number of assumptions on pointer values within that address space: The pointer must have a fixed integral value.

21:30

https://llvm-swift.github.io/LLVMSwift/Structs/AddressSpace.html

21:30

A pointer is an address that provides a translation within some virtual address space to a piece of physical memory.

Ryozuki

all values > 0 are truthy

that's the reason I proposed == 1. I could have also said == 123123. Doesn't 1 get evaluated to true aswell?

Devinci

that's the reason I proposed == 1. I could have also said == 123123. Doesn't 1 get evaluated to true aswell?

ur missing the point

21:32

if u compare a pointer type to a int

21:32

it wont be casted to a bool

Oh man.

there will never be a true or false

21:32

it will be

21:32

does pointer has address 0x1 through this cast to int

oh really? That's what I was missing then. thanks for your answer ryo

@Devinci #staytoxic

21:34

That from a moderator

@Devinci

21:34

g++ /tmp/G2nc4Yh94s.cpp
/tmp/G2nc4Yh94s.cpp: In function 'int main()':
/tmp/G2nc4Yh94s.cpp:14:10: error: ISO C++ forbids comparison between pointer and integer [-fpermissive]
   14 |     if(a == 1) {
      |        ~~^~~~

21:34

the compiler will also tell u ur doing bad stuff

yeah I didn't know that method? was returning a pointer.

21:34

I just assumed, mb there

xxxxD

21:35

0 self awareness

fokkonsomething

21:36

?

Yeah, I'm on my daily dose of talking to idiots

ah that makes sense

u didnt talk to me yet

fokkonaut

Yeah, I'm on my daily dose of talking to idiots

shit, I'm included

self awareness: 1

21:37

> devinci (edited)

21:38

good night :)

gn :)

heartw

1

fokkonaut

Yeah, I'm on my daily dose of talking to idiots

friendly fire will not be tolerated

another self-made teedrama™?

@ChilleDragon where can i check your votes.cfg for zilly-insta?

Teero

i think ill make a server that exploits the tune zone parameters and lets people play in slowmo.

is it still on?

github-merge-queue[bot]

[ddnet:master] 2 new commits

d2ea9cb Fix multiple editor undo/redo issues - archimede67 7e73d3e Merge pull request #7803 from archimede67/fix-editor-undo-redo - def-

22:58

[ddnet/ddnet] Pull request closed: #7803 Fix multiple editor undo/redo issues

22:58

[ddnet/ddnet] Issue closed: #7738 Segfault with undo/redo when using the "Remove unused envelopes" tool

Super3559

is it still on?

nop

Teero

nop

rip

22:58

got any way for me to doo it myself

didnt finish it

since ive been trying to get it to work

22:59

but i have no idea what to change

github-merge-queue[bot]

[ddnet:master] 2 new commits

c7fb6dd Update Swedish translations for 18.0 - furo321 225b729 Merge pull request #7804 from furo321/swedish-18.0 - def-

23:00

[ddnet/ddnet] Pull request closed: #7804 Update Swedish translations for 18.0

Teero

didnt finish it

can you tell me how you did it?

GitHub

Click to see attachment 🖼️

build no 2800 btw!!

poggers2

1

i need help with cmake to build the source code

what exactly

[ddnet/ddnet] Pull request opened: #7805 Update russian.txt

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 (especially base/) or added coverage to integration test
[ ] 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-ad...

apparently it din't recognize visual studio code

23:34

even if i had it installed

23:34

and oppened in another tab

23:35

i looked it up and it said

what did not recognize vscode?

23:37

if you mean cmake. vscode does not have an integrated compiler. if youre on windows then use visual studio community

Teero

if you mean cmake. vscode does not have an integrated compiler. if youre on windows then use visual studio community

don't!

23:39

install compiler and cmake

23:39

then you will be able to work with vscode

ik msvc is bad. but its easier xdd

Exported 370 message(s)