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 2019-09-02 00:00:00Z and 2019-09-03 00:00:00Z
Avatar
@Learath2 Could the locale also ne the reason file score did not work everytime for me?
11:36
Because it doesnt save a rank everytime (dont know if its fixed now)
11:37
In DDNetPP it works, and between 9.0.2 and now is not much changed
11:37
in that file
Avatar
[quakenet] Learath2 BOT 2019-09-02 11:37:22Z
hmm, i'm not completely sure if locale would affect a file we opened
Avatar
Maybe there is a problem with teams.cpp? Because from there we call savescore
Avatar
[quakenet] Learath2 BOT 2019-09-02 11:37:55Z
I'd take a look at the standard on how ostream behaves and whether it involves any locale-defined behaviour
11:38
Well, I dont even know where the problem is, because I get the "x finished in y" message everytime
11:38
so its most likely in file_score.cpp
11:39
But as I said, in 9.0.2 it works obviously (ChillerDragons mod has no problems)
Avatar
I dont have permission for http://phpwomen.org/ bcs i am a men. This is sexist
troll 1
Avatar
omg
Avatar
Why do Freeze Inputs not work reliably with vanilla client?
Avatar
[quakenet] deen BOT 2019-09-02 13:02:23Z
because the client has to send the direct input more often to trigger it reliably
13:02
can be simulated by altering the direct input very quickly with some tools, which pros did, which is why DDNet client implemented it a bit more reliably, even though still hacky
Avatar
Can we make this serverside?
Avatar
it used to be
15:41
then i switched it to clientside since we didn't control most servers unfortunately
Avatar
But in my case (0.7) it would be good to have it serverside. Can you provide a fix from the history somewhere?
Avatar
That old fix checking for ninja should probably be replaced with the m_Freeze flag from the extended object
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 16:42:57Z
Learath2: "int m_Type = {};"?
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 16:50:21Z
what does that do?
16:50
default-initialize the variable? (i.e. do nothing for int)
Avatar
[quakenet] Learath2 BOT 2019-09-02 17:02:03Z
heinrich5991: that zero-initializes the variable
17:02
(as defined by the c++ standard that is)
17:02
in the case of int it'd set it 0
Avatar
why not = 0? Looks simpler.
18:27
int m_Type{0}; should be used in class definition if you want it initialized the same way in every constructor
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 18:38:28Z
why not int m_Type = 0;?
18:38
which c++ standard btw?
18:41
where can I read about that?
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 19:33:26Z
deen: is there a reason why we ship a bundled libcurl on linux? it seems that it is installed everywhere
Avatar
[quakenet] Learath2 BOT 2019-09-02 19:48:01Z
I think I read it in C++14, but I don't think the types of initialization changed between standards
19:49
Section 8.5
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 19:57:09Z
wait
19:57
is int() == 0?
19:57
or int a();
19:58
nah, the latter is a function declaration
20:03
8.5.4, paragraph 3:
20:03
List-initialization of an object or reference of type T is defined as follows:
20:03
— If T is an aggregate, aggregate initialization is performed (8.5.1).
20:03
20:03
— Otherwise, if the initializer list has no elements, the object is value-initialized.
20:04
example: int** pp {}; // initialized to null pointer
Avatar
[quakenet] Learath2 BOT 2019-09-02 20:17:32Z
8.5p8 tells you why those end up getting zero initialized
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 20:18:10Z
yep
Avatar
so that ddnet keeps the same dependencies as vanilla
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 21:03:50Z
sounds like we could drop our custom lib if it's basically installed everywhere
Avatar
[quakenet] heinrich5991 BOT 2019-09-02 21:25:08Z
debian oldoldstable provides libcurl.so.4, which is the so version we link to: https://packages.debian.org/jessie/amd64/libcurl3/filelist
Exported 52 message(s)