Anonymous
кстати, посмотрел бы репозитории и у вас
Have you seen my shawermabuilder? Oh my goodness it was awesome
Denis
На чо мьютекс то? На шеред птр чтоль?
сомневаюсь что ты до конца понимаешь зачем он там
Denis
тебе в кайф к словам придираться?
Ioann_
Мы пришли к выводу, что нет
Ioann_
но вот в чате в этом, чуваки пришли к выводу, что да
Ioann_
Кто прав, где чья правда ?
Denis
действительно, зачем? лучше не думая кидать необоснованную критику
Ioann_
Надо стандарт читать
8.18 [expr.ass] «In all cases, the assignment is sequenced after the value computation of the right and left operands, and before the value computation of the assignment expression»
Ioann_
assignment-operator: one of = *= /= %= += -= »= «= &= ^= |=
Aidar
Вангую ошибку в локах
Ioann_
Так что это не UB
Denis
Ну так жду разъяснений
были проблемы когда перехватывал функцию, вызываемую в разных потоках. мьютекс исправил это
брат
А чем он отличается от этого?
Artemiy
Что такое UB?
Первый запрос в гугле же выдает ответ...
брат
прочел
Aidar
Тут скорее надо лочить конкретный участок памяти в хуке
брат
у вас ub это undefined behaviour?
Aidar
Как ты это сделаешь сам придумывай
🐙 Antony
x86 win32 и linux. работает без проблем, используется в плагинах для самп-сервера
Инфраструктурно: Побольше бы тестов... При том с функциями принимающими 10-15 аргументов по ссылке, указателю и проч. Немного причесать, написать доку - и будет пальчики оближешь. И очень хочется дружелюбный интерфейс для новичков вида make_hook(function, Id). Такая функция сама сможет определять сигнатуру функции, thiscall или по умолчанию ставить stdcall
брат
Не очень люблю сокращения, если да
брат
Не сразу понял
Aidar
Сейчас если я сделаю два хука на одно и то же кажется все вывалится
🐙 Antony
x86 win32 и linux. работает без проблем, используется в плагинах для самп-сервера
Есть еще у меня подозрение, что va_args не поддерживаются.
Ioann_
А чо за хуки то ? Dll inj ?
Aidar
нет
Дык при вызове
Aidar
Многопоточно
Aidar
Мьютекс у тебя закреплён за объектом
Aidar
А не за куском памяти
Denis
там разве не cdecl?
Denis
соглашение о вызове
Denis
если так, то должно поддерживать
Denis
в чем проблема?
Denis
что им мешает?
🐙 Antony
если так, то должно поддерживать
стоит перепроверить, я помню что в std::function есть место Args...... думаю это сделано не с проста
Denis
Дык при вызове
Denis
Многопоточно
Denis
Мьютекс у тебя закреплён за объектом
Denis
А не за куском памяти
Denis
это не создаёт проблем
Denis
не посыпется ничего
Denis
там мьютекс стоит и он как раз захватывает управление до манипуляций с байтами
Denis
обрати внимание на hook::raii
Denis
в call_cb
Denis
помогло, потому что предотвратило data race в манипуляции с байтами, когда идёт их временная перезапись
Denis
возможно правильнее было поставить мьютекс как раз только в участок с этими манипуляциями
Denis
тогда сама функция не блокировалась бы полностью
Denis
271
Denis
поправил
Denis
на строчке 271 запись в память происходит, что непонятно-то?
Denis
установка патча
Denis
у тебя есть публичные репозитории, кстати? мне так никто и не показал своих работ
Deleted Account
у тебя есть публичные репозитории, кстати? мне так никто и не показал своих работ
у меня есть крутая либа тг бот апи но тебе не понравится
Denis
любому, кто участвовал в критике
🐙 Antony
Да пожалуйста :) https://github.com/apolukhin
Deleted Account
посмотрим
https://github.com/zaizupro/TelegramBotApi
Denis
https://github.com/zaizupro/TelegramBotApi
конструктор и деструктор можно default пометить и все. без этих пустых определений
Denis
NULL привет из си
Denis
query builder был бы лучше, чем ручная склейка параметров запроса
Denis
ну и всё, вроде
брат
мимо
Deleted Account
ну и всё, вроде
круто, спс
Denis
antony контрибьютор boost?
Крылатый
Да пожалуйста :) https://github.com/apolukhin
Хотел нажать Follow, но уже.
Constantine
ну почему, почему все авторы библиотек для reactive programming такие извращенцы?
🐙 Antony
antony контрибьютор boost?
Ну да. Вот тут даже можно посмотреть, за что отвечаю https://github.com/boostorg/boost/blob/master/libs/maintainers.txt
Denis
круто, надо будет сурсы почитать
Constantine
@antoshkka вы знакомы с концепцией reactive programming?