Teeworlds - bridge

[quakenet] <rand> how to trigger the crash ?

[quakenet] <TeeSlayer> Catch16 (now 64) is back! Search for L#07 :)

teeslayer update ur race servers + restart them, very unstable

10:43

check it out yourself if u play on it

[quakenet] <Dune> would be interesting to know what happens

10:51

[quakenet] <Dune> rand: join different large maps, in my experience. not easy to reproduce, I got Fudgy's settings and that helped

10:57

[quakenet] <Dune> oh, didn't know @Zatline did catch16 maps :)

[quakenet] <rand> grmbl, I tried connected to a bunch of server with unofficial big map, nothing happened, not fun :|

12:51

[quakenet] <TeeSlayer> FluffiexD: what exactly is unstable?

12:51

[quakenet] <Dune> yeah takes a bunch of tries

12:52

[quakenet] <TeeSlayer> FluffiexD: I can't reproduce it either

12:52

[quakenet] <TeeSlayer> everything seems fine

12:55

[quakenet] <rand> each time I run tw, my laptop goes wild to +66°C, bunch of tries will participate to global warming

12:55

[quakenet] <TeeSlayer> I just asked a bunch of regulars and nobody is having any problems

12:58

[quakenet] <TeeSlayer> therefore I'm not restarting or updating (there are no updates available anyway)

[quakenet] <Dune> oh ouch

13:50

[quakenet] <Dune> been looking at the code wondering where it went wrong

13:50

[quakenet] <Oy> :P

13:51

[quakenet] <Dune> why did you remove the check in the for loop btw?

13:51

[quakenet] <Oy> it's on top now

13:51

[quakenet] <Dune> ah right

13:51

[quakenet] <Oy> would be duplicate

13:51

[quakenet] <Oy> it was missing on the bottom, where it changed the tiles

13:52

[quakenet] <Oy> but don't think that we need to update the release

13:52

[quakenet] <Oy> easter is over tomorrow :D

13:52

[quakenet] <Dune> yeah that's what I was thinking

13:53

[quakenet] <Oy> well, we could silently sneak it in :D

13:53

[quakenet] <Dune> what is https://github.com/teeworlds/teeworlds/issues/1390 btw?

Add flag for ai-controlled players · Issue #1390 · teeworlds/tee...

Add a flag for ai-controlled players (client, server) so you can filter them within the server browser (similar to spectators). Updatet: visual indication missing (in the server browser and or ingame)

13:53

[quakenet] <Dune> countryflags?

13:55

[quakenet] <Oy> yeah, just the countryflag left

13:55

[quakenet] <Dune> okay

13:55

[quakenet] <Oy> so u see the bots right away in the server browser

13:55

[quakenet] <Dune> is the info there in the server browser? I forgot

13:55

[quakenet] <Dune> it's not just the bot count?

13:56

[quakenet] <Oy> every play in the server info should have a type field (player, spec, bot) and then client calculates the numbers

13:56

[quakenet] <Dune> I see, cool

13:58

[quakenet] <Oy> https://github.com/teeworlds/teeworlds/blob/master/src/engine/server/server.cpp#L1154

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

13:59

[quakenet] <Oy> updated first post in https://github.com/teeworlds/teeworlds/issues/1390

Add flag for ai-controlled players · Issue #1390 · teeworlds/tee...

13:59

[quakenet] <Dune> ack

14:01

[quakenet] <Oy> hm, when i disable compatible version in the server browser i get 30 more servers

14:02

[quakenet] <Oy> oh, seems i'm banned. lol

14:03

[quakenet] <Oy> thought there was some bug with the server infos

14:05

[quakenet] <Dune> huh

14:05

[quakenet] <Dune> got one ip?

14:07

[quakenet] <Oy> it hosts lots of servers, sec

14:07

[quakenet] <Oy> 51.15.34.109

14:10

[quakenet] <Dune> huh okay, not sure why he'd ban you lol

14:10

[quakenet] <Dune> don't think they're doing anything against the rules

14:12

[quakenet] <Oy> yeah, no idea, only play rarely anyway, no problem :)

14:12

[quakenet] <Dune> maybe you were too good and got flagged as a bot :P

14:12

[quakenet] <Oy> maybe someone bad had that ip in the past

14:12

[quakenet] <Oy> :P

14:13

[quakenet] <Dune> without the loli servers you must be seeing like half the players that are on though

14:14

[quakenet] <Dune> 60/83 players are on those currently

14:14

[quakenet] <Oy> 99 servers with 25 players

14:14

[quakenet] <Dune> heh, there are 83 players right now

14:14

[quakenet] <Oy> :D

14:15

[quakenet] <Dune> you must have been underestimating the playerbase :D

14:15

[quakenet] <Oy> yeah :D

14:15

[quakenet] <Oy> usually play on that ctf5 server and that one's still there :)

14:17

[quakenet] <Dune> :)

14:17

[quakenet] <Dune> catch16 looks nice with the 0.7.3 stuff, gj TeeSlayer

14:17

[quakenet] <Dune> (and oy for making it possible I guess :P)

14:18

[quakenet] <Dune> I only miss the sound of killing people

[quakenet] <Oy> :D

14:38

[quakenet] <rand> (client side on death ?)

14:39

[quakenet] <Dune> hm?

14:40

[quakenet] <rand> the death sound

14:41

[quakenet] <TeeSlayer> Oy: can you pm me your IP address? you're caught in a range ban

14:42

[quakenet] <rand> blame your people !

14:43

[quakenet] <TeeSlayer> e ~/config/bans.cfg

14:43

[quakenet] <TeeSlayer> eh

14:43

[quakenet] <TeeSlayer> wrong window :D

14:43

[quakenet] <Dune> at least that wasn't a sudo command

14:44

[quakenet] <TeeSlayer> vim^^

14:45

[quakenet] <Dune> "[5cbdd33a][demo_recorder]: Unable to open mapfile 'Grasscatch'" not sure why I'm getting this

14:51

[quakenet] <Oy> odd

14:52

[quakenet] <Oy> when do u get that message?

14:52

[quakenet] <Dune> every start of a game

14:52

[quakenet] <Dune> it's the first time I join a server with that map

14:53

[quakenet] <Dune> okay, it does that even on second try

14:53

[quakenet] <Dune> oooh I think it's because it has the new map format hash

14:54

[quakenet] <Dune> Grasscatch_996c94da872064682c17e3889a1448c03846ba8c234e8c914da8a75fec234743.map

14:56

[quakenet] <TeeSlayer> >catch16 looks nice with the 0.7.3 stuff, gj TeeSlayer

14:56

[quakenet] <TeeSlayer> Dune: it's not my mod

14:57

[quakenet] <TeeSlayer> AssassinTee wrote it

14:57

[quakenet] <Oy> demo recorder doesn't work

14:57

[quakenet] <Dune> oh, so that's who this person was

14:57

[quakenet] <TeeSlayer> the git repo is in the motd of the server

14:57

[quakenet] <TeeSlayer> I used the map_update script to convert the map

14:58

[quakenet] <Dune> the map is fine

14:58

[quakenet] <Dune> it's something with 0.7.3 servers, unrelated the the mod

14:58

[quakenet] <TeeSlayer> okay

[quakenet] <Oy> dunno if we should update releases now :)

15:10

[quakenet] <Dune> what for, eggs?

15:11

[quakenet] <Oy> no the demo recorder

15:11

[quakenet] <Dune> ah didn't see the commit

15:11

[quakenet] <Dune> irc bot still waiting for his secret :p

15:11

[quakenet] <Dune> well it's a tough one, but probably not

15:11

[quakenet] <Oy> :D

15:11

[quakenet] <Dune> it affects all OS

15:11

[quakenet] <Dune> 0.7.3 is already in some distributions

15:11

[quakenet] <Oy> yeah

15:11

[quakenet] <Oy> ok

15:11

[quakenet] <Oy> then we should do it with the next release

15:12

[quakenet] <Dune> yeah

15:12

[quakenet] <Dune> or start using the fourth digit that is available in version.h

15:12

[quakenet] <Oy> could do a smaller one sooner maybe

15:12

[quakenet] <Oy> :P

15:13

[quakenet] <Dune> I mean, we always end up having a bug correction one day after

15:13

[quakenet] <Dune> maybe it would be nice to number those fixes without having to do a big announcement

15:14

[quakenet] <Dune> steam and distribs would handle that well

15:15

[quakenet] <Oy> yeah

[quakenet] <Oy> so do a 0.7.3.1? :)

16:05

[quakenet] <Dune> what do you think, heinrich5991 ?

16:06

[quakenet] <Dune> servers will show up as 0.7.3.1 instead of 0.7.3 too right

16:12

[quakenet] <Oy> no, would just change https://github.com/teeworlds/teeworlds/blob/master/src/game/version.h#L9 and keep the game_version as 0.7.3

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:13

[quakenet] <Dune> ah, that's smart

16:13

[quakenet] <Dune> but would that show up in the client menus creen?

16:13

[quakenet] <Oy> no

16:13

[quakenet] <Dune> maybe modify https://github.com/teeworlds/teeworlds/blob/404430b56cb3bf7d78b5e98a9c855c502094cd5a/src/game/client/components/menus_start.cpp#L91-L103 then

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:18

[quakenet] <Oy> that check seems to work, just displaying the version needs to change https://github.com/teeworlds/teeworlds/blob/404430b56cb3bf7d78b5e98a9c855c502094cd5a/src/game/client/components/menus_start.cpp#L103

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:19

[quakenet] <Dune> right

16:20

[quakenet] <Dune> sounds good then

https://github.com/teeworlds/teeworlds/issues/2098 Does negative ClientID is acceptable ?

Creating a server with a lot of dummies crashes it · Issue #2098 ...

./teeworlds_srv dbg_dummies 30 crashes. Will have a look later and fill in details

[quakenet] <Oy> Savander huh?

cause, DBG_DUMMIES send negative clientid

16:25

https://github.com/teeworlds/teeworlds/blob/master/src/game/server/gamecontext.cpp#L1521

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:25

to here

16:25

https://github.com/teeworlds/teeworlds/blob/master/src/game/server/gamecontext.cpp#L673

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:25

i used vs debugger to check it ouit

[quakenet] <Dune> oh, it does?

yea

16:25

if you set up dbg_dummies highter than maxclients

16:25

then it starts from the hightest possible maxclients

16:25

and decrease to the point, where it's negative and more

16:26

also, what's weird. If you set up dbg_dummies to 8 (with max_clients 8)

16:27

and you join to server, it crashes

[quakenet] <Dune> makes sense because there is no room for dummies

can't do nothing more, not C specialist

Hope i helped

16:27

somehow

[quakenet] <Oy> yeah, should fix that someday :D

Idk, maybe something like that will help: if (ClientID >= g_Config.m_SvMaxClients && ClientID < 0) return;

16:30

in OnClientConnected() method

16:31

it should be "||"

16:31

xD nope, still crashes

[quakenet] <Oy> or clamp number of dummies to max clients

16:32

[quakenet] <Dune> - online clients

16:35

[quakenet] <Oy> Savander: https://github.com/teeworlds/teeworlds/blob/master/src/game/server/gamecontext.cpp#L1520 -> for(int i = 0; i < g_Config.m_DbgDummies && i < Server()->MaxClients(); i++)

teeworlds/teeworlds

A retro multiplayer shooter. Contribute to teeworlds/teeworlds development by creating an account on GitHub.

16:40

[quakenet] <Oy> anything crucial for 0.7.3.1?

16:40

[quakenet] <Dune> not that I can think of

16:40

[quakenet] <Dune> https://github.com/teeworlds/teeworlds/pull/2107

Fix platform-specific client libraries for Linux by AMDmi3 · Pull...

Do not hardcode library names, detect and depend on them properly instead.

16:41

[quakenet] <Dune> best to avoid features probably :p

16:42

[quakenet] <Oy> yeah

That doesn't fixed the crash problem :PP

16:43

and i can't trace it back, it loops in "dbg_break" method.

[quakenet] <Dune> loops?

16:44

[quakenet] <Oy> hm, you shouldn't be able to connect when the server is full with dummies

yea, like. If i press forword, it keep back to that function, it doesn't even go out of it

[quakenet] <Dune> well it writes to memory address zero

16:44

[quakenet] <Dune> i guess the debugger refuses to proceed

yeah, but I think, I should have the possibility to go back.

16:45

after all, server says: game\server\player.cpp(11): already used

[quakenet] <Dune> I don't know how you print the backtrace in this debugger, but you should be able to (?)

16:46

[quakenet] <Oy> ah right

Visual Studio Debbuger

[quakenet] <Oy> engine doesn't know about the dummies

have options to go back, forward into etc

[quakenet] <Oy> other issue then

At least, we found another one

16:46

it's good, no? :DD

[quakenet] <Oy> well, yeah :D

In weird way...

[quakenet] <Dune> alright

16:56

[quakenet] <Oy> Dune: you can do ur linux magic :D

16:56

[quakenet] <Dune> gotta do steam and the forum short news

16:57

[quakenet] <Dune> ah, you didn't draft, straight up publish :D

16:57

[quakenet] <Oy> yeah :D

[quakenet] <Oy> grr

[quakenet] <rand> dummies are made to crash

[quakenet] <Dune> :D

[quakenet] <rand> https://github.com/teeworlds/teeworlds/commit/f469fb41da51b1351abca785f0635340bdc8b8b4

added a 5 second cooldown for responding to a banned user that wan...

…to connect (econ, server). otherwise silently ignore the attempt

18:21

[quakenet] <rand> is it mandatory to drop the "const" for IsBanned ?

18:22

[quakenet] <rand> (looks like it builds with "const" at least)

18:25

[quakenet] <Dune> Oy, cmake uses a hack to check for versions

18:25

[quakenet] <Dune> :(

18:25

[quakenet] <Oy> yeah :(

18:26

[quakenet] <Dune> i was uploading the linux tarballs

18:26

[quakenet] <Dune> i wonder what git log | grep fix.* cmake would give :P

18:26

[quakenet] <Oy> rand: didn't build for me

18:26

[quakenet] <Oy> :D

18:29

[quakenet] <rand> in fact, I only made the method const, not the first paramater of MakeBanInfo

18:29

[quakenet] <rand> CNetBan::MakeBanInfo<CNetRange>(CBan<CNetRange> *pBan...) const

18:33

[quakenet] <rand> it make sense not to use const for the first arg since it can be modified

18:40

[quakenet] <Dune> i updated the release (but the cmakefiles are not fixed)

[quakenet] <Oy> seems to work

18:50

[quakenet] <Oy> update the release?

18:51

[quakenet] <Oy> updating the src files might suffice

[quakenet] <Dune> aight

19:09

[quakenet] <Dune> uploading

[quakenet] <Dune> shouldn't you move the 0.7.3.1 tag, oy?

19:29

[quakenet] <Dune> > 1 commit to master since this release

19:29

[quakenet] <minus> 0.7.3.1.1 when

19:32

[quakenet] <Dune> not supported by the protocol :D

19:33

[quakenet] <Dune> I'll remove the 0.7.3 downloads entry when this is done

19:37

[quakenet] <Dune> just need steam now

[quakenet] <Dune> my laptop keeps going to sleep before the upload completes lol

[quakenet] <Oy> yeah, will move the release and update the source files

19:58

[quakenet] <Dune> okay

[quakenet] <Oy> done

20:08

[quakenet] <Dune> completing steam upload

20:12

[quakenet] <Dune> all set (need minus magic too)

20:12

[quakenet] <minus> abra cadabra

[quakenet] <Oy> good :)

[quakenet] <minus> versionsrv doesn't actually support patch-level changes

20:37

[quakenet] <minus> or does it

20:38

[quakenet] <Oy> should

20:38

[quakenet] <minus> aye, the array is 8 bytes big but only used 5 before

20:38

[quakenet] <Dune> better not update it anyway?

20:39

[quakenet] <minus> too late

20:40

[quakenet] <Oy> hm

20:41

[quakenet] <Oy> minus: did u update the 0.7 version server?

20:41

[quakenet] <minus> do we have one?

20:41

[quakenet] <Oy> yeah

20:41

[quakenet] <minus> i have no idea how this works

20:41

[quakenet] <Oy> the one with port 8285

20:42

[quakenet] <minus> no such thing is running

20:43

[quakenet] <Oy> oh :)

20:49

[quakenet] <Dune> so move the news?

20:50

[quakenet] <minus> i guess? the new version is alreadyon the front page

20:50

[quakenet] <minus> very coordinated release

20:52

[quakenet] <Dune> done, steam is done too

20:52

[quakenet] <minus> yo where's my soldat for linux release