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

R4SAS error: cannot bind non-const lvalue reference of type 'i2p::http::HTTPServer::DaemonStop&' {aka 'std::function<void()>&'} to an rvalue of type 'i2p::http::HTTPServer::DaemonStop' {aka 'std::function<void()>'}

R4SAS 426 | d.httpServer->SetDaemonStop (std::bind (&Daemon_Singleton::stop, this));

R4SAS | ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

orignal да нет я примено сказал как

orignal SetDaemonStop (DaemonStop& f); или const поставь или & убери

R4SAS ужасно выглядит

orignal ну тогда сделай код и напиши где надо сделать комментарий что надо сделать то то и то то

orignal я сделаю

R4SAS виндовый тоже переделаешь?

R4SAS а то сейчас таймеры для линя и винды каждый свой, не в синглтоне

orignal не знаю

orignal ну ты там по аналогии сделаешь как я для лиункса

R4SAS так а как я должен из DaemonWin32 обращаться с синглтону, если инстанс i2p::util::DaemonWin32::Instance объявляется в Daemon.h?

orignal ну и что?

orignal ну так объяви его там со словом extern делов то

R4SAS эм, а как?

R4SAS вот такое используется в Daemon.h:

R4SAS #define Daemon i2p::util::DaemonWin32::Instance() class DaemonWin32 : public Daemon_Singleton

R4SAS ***

orignal счас

R4SAS #define Daemon i2p::util::DaemonWin32::Instance()

R4SAS class DaemonWin32 : public Daemon_Singleton

R4SAS я понимаю если тип указывать

orignal а все понял ты о чем

orignal какой то мудак использует Daemon как сингелтон

orignal а не как класс

orignal это уебище надо бы переделать

orignal к твоей задаче это не имеет отношения

R4SAS к текущей - нет

R4SAS но если делать отвязку от инклуда Daemon.h то я сразу в стенку упираюсь

orignal &Daemon_Singleton::stop короче надо

orignal где ты делаешь отязку?

orignal в win32?

R4SAS да

orignal покажи строку где ты его вызываешь

orignal что то сомневаюсь чтобы я такое написал ))

R4SAS github.com/PurpleI2P/i2pd/blob/openssl/Win32/DaemonWin32.cpp#L12

R4SAS github.com/PurpleI2P/i2pd/commit/0b210ee99d9784cfd33d9db9a6a46c235b8f6559

R4SAS но тогда это все было в одной папке и работало

R4SAS если переделывать, то капитально

R4SAS как обращаться колбэками я понял

R4SAS и из DaemonWin32 вызывать Win32App будет не такой проблемой

orignal вот и начни с этого

orignal через коллбэк

R4SAS блин

R4SAS у меня команда остановки вызывается из класса HTTPConnection

R4SAS а устанавливаю функцию в HTTPServer

orignal а у него ссылки на сервер нету?

orignal да нету

R4SAS ага, нет

orignal вот в этом и беда сингелтонов

orignal значит надо добавить

orignal в момент его конутркутор передавать ссылку на сервер

R4SAS или через конструктор передавать указатель на команду?

R4SAS на функцию*

R4SAS хотя не безопасно

orignal нет именно на север ссылкой

orignal не надо держать указатеои на фнукции в коннкешинах

R4SAS auto conn = std::make_shared<HTTPConnection> (m_Hostname, newSocket);

R4SAS и как самого себя туда передать?

R4SAS &this ?

orignal *this

orignal посмотри как сделаено в стримах ))

orignal только имей ввиду ссылку можно только иниицализировать другой а не копировтаь

R4SAS HTTPConnection (std::string serverhost, std::shared_ptr<boost::asio::ip::tcp::socket> socket, std::shared_ptr<const i2p::http::HTTPServer> server);

orignal & забыл

orignal так нельзя

orignal ты не знаешь какая природа server

orignal HTTPServer& server

orignal лучше так

orignal техничеки это тот же 8 байтный адрес

R4SAS т.е. не shared_ptr?

orignal нет

orignal потому что ты не знаешь что такое server

orignal может это просто локальная переменная

R4SAS теперь еще придется перемещением кода заниматься?

R4SAS говорит что знать не знает про HTTPServer

R4SAS оно объявлено ниже

orignal не надо ничего перемещать

orignal перед ним напиши class HTTPServer;

orignal это же в куче мест есть

R4SAS ок

R4SAS HTTPConnection (std::string serverhost, std::shared_ptr<boost::asio::ip::tcp::socket> socket, HTTPServer& server);

R4SAS const HTTPServer& m_Server;

R4SAS auto conn = std::make_shared<HTTPConnection> (m_Hostname, newSocket, *this);

R4SAS HTTPConnection::HTTPConnection (std::string hostname, std::shared_ptr<boost::asio::ip::tcp::socket> socket, HTTPServer& server): m_Socket (socket), m_BufferLen (0), expected_host(hostname), m_Server (server)

orignal нет не const

orignal ты еще переименную объявить должен

orignal m_Server

orignal ну так нормально вроде

R4SAS HTTPServer& m_Server; - так это оно и есть объявление, не?

orignal да

orignal просто ты ему ссылкой с const инициализовать не сможешь

R4SAS понял

R4SAS а мне m_DaemonStop чтоль в паблик надо?

R4SAS error: 'i2p::http::HTTPServer::DaemonStop i2p::http::HTTPServer::m_DaemonStop' is private within this context

orignal естественно

R4SAS или делать функцию опять же, которая будет возвращать этот m_DaemonStop?

R4SAS ужос

orignal это где опять такое вызывается напрмямую?

orignal конечно функцию

orignal нельзя обрашаться к полям напрямую

R4SAS orignal: а как делается с всякими интами и булями

R4SAS указателями

orignal приведи пример

R4SAS например есть в DaemonWin32 bool isGraceful

orignal это тоже неправильно

R4SAS к которому надо получить доступ

R4SAS из Win32App

orignal написать методы Get и Set

R4SAS я знаю

orignal точно также через коллбэк

R4SAS из DaemonWin32

R4SAS я буду вызывать колбек, но что туда передавать?

R4SAS std::bind?

R4SAS внутри Win32App.h я так понимаю надо как и в тот раз писать подобное:

R4SAS void SetIsGraceful (const bool& b) { m_isGraceful = b; };

R4SAS bool m_isGraceful;

R4SAS а внутри DaemonWin32 как вызываю?

R4SAS i2p::win32::SetIsGraceful (&isGraceful) ?

orignal const bool& b

orignal это еще что за дичь?))

R4SAS а вот такая дичь

orignal не надо там по ссылке

R4SAS просто указателем?

orignal просто bool b

R4SAS так а оно разве не поставит единоразово?

orignal а вызов m_SetIsGraceful (true) например

orignal я тебе просто говорю как надо

R4SAS смотри в чем дело

orignal по ссылке только сложные типа передаются

R4SAS почему надо хранить в демоне - потому что этот буль надо получать и в морде, и в трее в меню

orignal логично

orignal короче такое во многих местах сделано

orignal вместо std::bind можно лямбду

R4SAS i2p::win32::SetIsGraceful (std::bind (isGraceful, this)); ?

orignal нет

orignal ты можешь привязку только методов делать

R4SAS напиши уже просто как должно быть)))

orignal i2p::win32::SetIsGraceful (std::bind (&Win32Daemon::SetIsGraceful, this, std::placeholders::_1))

orignal как то так

orignal посмотри I2PTunnels.cpp

R4SAS это сработает с чистым bool?

orignal тип не иммет значения

R4SAS стоп, это передача из демона в app, или установка указателя на значие в демоне?

R4SAS мне второе надо

orignal это установка коллбэка

R4SAS при запросе значения bool isGraceful что мне отдаст?

R4SAS из Win32App

R4SAS хотя я понял к чему ты ведешь

R4SAS добавление в демоне переменной внетренней, хранящей значение, set и get для него

orignal да

orignal переменная геттер и сеттер

R4SAS и прокидывание функции get в win32фзз

orignal правильно

R4SAS извращенсво

orignal и если надо Set

R4SAS надо будет

orignal нет

orignal потому что win32 у тебя ничего не знает о демоне

orignal а знает только что он как тот суслик

orignal его не видит а он есть

R4SAS daemon:

R4SAS private bool m_isGraceful;

R4SAS bool GetIsGraceful() { return m_isGraceful; }

R4SAS void SetIsGraceful(bool state) { m_isGraceful = state; };

R4SAS DaemonWin32:

orignal Get надо const

R4SAS i2p::win32::SetIsGraceful (std::bind (&Win32Daemon::SetIsGraceful, this, std::placeholders::_1));

R4SAS i2p::win32::GetIsGraceful (std::bind (&Win32Daemon::GetIsGraceful, this));

R4SAS ок

orignal да примерно так

R4SAS далее в Win32App

R4SAS typedef std::function<void (bool)> DaemonSetIsGraceful;

R4SAS DaemonSetIsGraceful m_setIsGraceful;

orignal правильно

R4SAS а сам SetIsGraceful как описывать?

R4SAS и кого в дальнейшем вызывать то?

R4SAS не m_setIsGraceful же

orignal m_setIsGraceful надо вызывать

R4SAS void SetIsGraceful (const DaemonSetIsGraceful& f) { m_setIsGraceful = f; }; -- если брать то что до этого делал

R4SAS т.е. далее я в коде вызываю m_setIsGraceful(true) ?

orignal все правильно

orignal да

orignal только с проверкой на nullptr

orignal что установлен

R4SAS а он может быть не установлен?

orignal так если нкто не вызвал SetIsGraceful то чему он будет равен?

R4SAS я сомневаюсь что такое произойдет в процессе работы

R4SAS оно же в демоне при старте устанавливается

orignal ты не имеешь право полагаться на это

orignal мало ли кто и как в дальнейшем будет твою либу использовать

R4SAS и что делать если оно не установлено?

R4SAS ладно если для set это еще приемлемо

R4SAS if (m_setIsGraceful) m_setIsGraceful(true);

orignal да

orignal а для get надо m_getIsGracefuk ? m_getIsGraceful : false;

R4SAS if(m_getIsGraceful)

R4SAS {

R4SAS if(m_getIsGraceful())

R4SAS InsertMenu (hPopup, -1, MF_BYPOSITION | MF_STRING | MF_GRAYED, ID_ACCEPT_TRANSIT, "Accept &transit");

R4SAS } else

R4SAS InsertMenu (hPopup, -1, MF_BYPOSITION | MF_STRING, ID_ACCEPT_TRANSIT, "Accept &transit");

R4SAS так чтоли?

R4SAS логика что если грейсфул, то добавляется еще MF_GRAYED

R4SAS как пример

orignal да так

R4SAS github.com/PurpleI2P/i2pd/blob/openssl/Win32/DaemonWin32.cpp#L47-L52

R4SAS что не так с этим классом?

R4SAS как я могу в него передать?

R4SAS функции уже описал

orignal а что там не так?

R4SAS если брать service-> то орет так: base operand of '->' has non-pointer type 'I2PService'

orignal так надо .

orignal I2PService service((PSTR)SERVICE_NAME);

R4SAS если . то начинает так орать: Win32/Win32Service.h:32:30: error: 'function' in namespace 'std' does not name a template type 32 | typedef std::function<bool ()> DaemonStart;

orignal видишь же его объявление

R4SAS вижу

R4SAS только почему когда я правильно использую он начинает орать что нет std

orignal я не знаю что ты далаешь

R4SAS typedef std::function<bool ()> DaemonStart; void SetDaemonStart (const DaemonStart& f) { m_daemonStart = f; };

R4SAS все как всегда

orignal и как вызываешь?

R4SAS service.SetDaemonStart (std::bind (&DaemonWin32::start, this));

R4SAS m_daemonStart ? m_daemonStart() : SetServiceStatus(SERVICE_STOPPED, "Unable to create callback");

orignal а у I2PService она есть?

R4SAS в нем и делаю

orignal возможно надо где то написать #include <functional>

orignal в Win32Service.g

R4SAS ага, так прокатило

R4SAS осталось только одно

R4SAS это тупость компилятора

R4SAS multiple definition of

R4SAS он почему то считает что я 2 раза дефайню SetIsGraceful, GetIsGraceful

R4SAS типа из Win32App и DaemonWin32

R4SAS оба инклудят Win32App.h

R4SAS и если верно помню, то фиксится это inline

orignal ты точно внутри класса их объявляешь?

R4SAS и если верно помню, то фиксится это inline

R4SAS ой

R4SAS Win32App не содержит классов

R4SAS даже не спрашивай почему

orignal тогда так нельзя

orignal надо писать или inline

orignal или внутри C++

R4SAS сделал inline

orignal будет работать

orignal а вот с переменнными так не прокатит

orignal надо C++17

R4SAS залил эту кучу в свою ветку

R4SAS далее надо пофиксить для лини обращение из морды....

R4SAS пока что просто затычка там

orignal дык