#dev (ilita) | Обсуждение разработки I2Pd | i2pd development discussion | http://i2pd.website/

Vort "<onon> Вроде не так уж и медленно" - для одного туннеля - да. но когда этот канал делится между сотнями и тысячами туннелей - то уже получается чёрти-что

Vort а если ещё учесть фоновый шум сети, то выходит что даже без пропускания полезного трафика эти узлы перегружены

sha512sum libi2pd/HTTP.cpp:86:4: warning: moving a temporary object prevents copy elision [-Wpessimizing-move] 86 | std::move (std::string (line.substr(pos + len))));

sha512sum libi2pd/HTTP.cpp:85:25: note: remove std::move call here 85 | return std::make_pair(std::move (std::string (line.substr(0, pos))),

sha512sum libi2pd/HTTP.cpp:67:24: note: remove std::move call here 67 | std::stringstream ss(std::move(std::string(line)));

sha512sum Какого фига там мув вообще?

sha512sum ~orignal

orignal это с 20 или с 17?

orignal правильно там мув

sha512sum 17, там гарантированное RVO будет

orignal где именно у тебя не собирается.

orignal поправлю

sha512sum У меня собирается, но мув там не нужен и уменьшает перф

sha512sum Варнинг выдаёт только

orignal нужен

sha512sum Там выполняется Copy Elision, std::string там была prvalue, а после std::move становится xvalue, из-за будет вызван доп перемещающий конструктор

orignal там где в stringstream там move конструктор есть

sha512sum Там только что созданная std::string мувается

sha512sum Это только вредит. Любой только что созданный объект с конструктором, будет считаться prvalue и при принятии такого объекта по значению, мув или копи конструктора не будет

sha512sum А с std::move будет гарантированно мув конструктор

sha512sum Лучше без конструкторов лишних и деструкторов, чем с ними

orignal мне надо вызвать именно мув конструтор там

sha512sum А зачем?

orignal чтобы избежать лишнего копирования

sha512sum Так копирования и не будет

sha512sum И мува тоже не будет

sha512sum Объект сразу создастся внутри параметров конструктора, без копирований или перемещений

orignal договорились

orignal выпилю

sha512sum godbolt.org/z/KTq475cbK У меня даже вот ссылка есть, где я такое активно использую.

sha512sum Да и зачем вообще std::make_pair использовать, есть же CTAD 17 и можно писать просто std::pair{value1, value2}

orignal потому что он там давно был

orignal счас переделаю

orignal так stingtream передавать через move или нет?

orignal если мне надо запхнуть в него string_view

sha512sum Ну так если надо запихнуть std::string_view, можно же создать из неё std::string. std::string будет prvalue в таком случае. А любое prvalue дополнительно мувать не надо

orignal ты это проверяла?

sha512sum Так в стандарте написано, можешь ту ссылку проверить, если бы оно так не работало, тот код бы просто не компилировался

sha512sum privatebin.i2p/?8e2b46ffd38eb508#9xbbpWVnzkA666hma3Juj7PUxS6aMJkeLrxc4juXnecL Или вот попроще пример

orignal счас поправлю

orignal а давай ты остальной код на 17 переделеаешь

sha512sum А давай ты поможешь xmpp клиент писать

orignal мне некогда

sha512sum )))

orignal закоммтил

orignal счас нормально?

sha512sum Ну компилятор не жалуется

sha512sum Значит должно быть нормально

orignal меня волнует чтобы оверхеда не было

sha512sum Ну теперь лучше да

shushkebab Hi

shushkebab how od i learn anything about libi2pd?

orignal you can't. it's secrect knowledge

shushkebab not ready yet?

shushkebab how would you store address book and everything.

orignal ready and works

orignal sorry I don't have time to explain all of this

shushkebab start a new one each time in memory or still have it in user directory

orignal for addredssbook you need libi2pd_client

shushkebab libi2pd too powerful in wrong hands?

orignal no, it's just you who is not ready to use it

shushkebab gotta figure it out on my own to be?

orignal yes please