SHS/pip
4
3
Fork 0
Code Issues 16 Pull Requests 2 Actions Releases Wiki Activity

some fixes #198

Merged
peri4 merged 1 commits from some_fixes into master 2026-03-18 11:08:49 +03:00
Conversation 3 Commits 1 Files Changed 5
+14 -25
andrey commented 2026-03-17 20:14:54 +03:00
Owner
Copy Link
Copy Source
  • PIEthernet::listen
  • PIClientServer::Server::~Server
  • PISystemTime const sleep
  • ClientServer tests fast and stable
- PIEthernet::listen - PIClientServer::Server::~Server - PISystemTime const sleep - ClientServer tests fast and stable
andrey added 1 commit 2026-03-17 20:14:54 +03:00
some fixes c02b627d47 
PIEthernet::listen
PIClientServer::Server::~Server
PISystemTime const sleep
ClientServer tests fast and stable
andrey reviewed 2026-03-18 08:58:54 +03:00
libs/client_server/piclientserver_server.cpp
@@ -77,3 +75,1 @@
c->destroy();
delete c;
}
closeAll();
andrey commented 2026-03-18 08:58:54 +03:00
Author
Owner
Copy Link
Copy Source

closeAll выполняет тоже самое но под мьютексом

closeAll выполняет тоже самое но под мьютексом
peri4 commented 2026-03-18 10:03:28 +03:00
Owner
Copy Link
Copy Source

Экономия в 3 строки, но лишние мьютексы и нотифаер, там вроде какие-то особенности (особенно на Винде) при статическом деструкторе, поэтому тут минимум лишнего

Экономия в 3 строки, но лишние мьютексы и нотифаер, там вроде какие-то особенности (особенно на Винде) при статическом деструкторе, поэтому тут минимум лишнего
peri4 commented 2026-03-18 10:04:07 +03:00
Owner
Copy Link
Copy Source

+ я не люблю менять что-то если нет прям нужды, работает хорошо, проверено

\+ я не люблю менять что-то если нет прям нужды, работает хорошо, проверено
andrey commented 2026-03-18 10:18:15 +03:00
Author
Owner
Copy Link
Copy Source

тут не про экономию, а про то что мьютекса не было. если бы работало хорошо я туда не лез бы. проблема в том что иногда тест падает в дестректоре с сегфолтом и сообщением corrupted size ... я не уверен что проблема тут, но после этих правок больше не падало.
на винде да - надо проверить - погонять тесты

тут не про экономию, а про то что мьютекса не было. если бы работало хорошо я туда не лез бы. проблема в том что иногда тест падает в дестректоре с сегфолтом и сообщением corrupted size ... я не уверен что проблема тут, но после этих правок больше не падало. на винде да - надо проверить - погонять тесты
peri4 commented 2026-03-18 10:55:32 +03:00
Owner
Copy Link
Copy Source

ок

ок
andrey reviewed 2026-03-18 09:01:52 +03:00
libs/main/io_devices/piethernet.cpp
@@ -527,6 +527,7 @@ bool PIEthernet::listen(bool threaded) {
listen_threaded = true;
server_bounded = false;
server_thread_.start(server_func);
server_thread_.waitForStart();
andrey commented 2026-03-18 09:01:52 +03:00
Author
Owner
Copy Link
Copy Source

после вызова PIEthernet::listen(threaded=true) сервер ещё не стартанул и при попытке коннекта получаю ошибку. А с этим вызовом ошибка сильно реже, т.к. хотя бы поток сервера уже стартанул. По хорошему тут бы дождаться что сокет забиндили уже, тогда ошибок бы совсем не было. Пока поставил костыли в виде piMinSleep после PIEthernet::listen

после вызова PIEthernet::listen(threaded=true) сервер ещё не стартанул и при попытке коннекта получаю ошибку. А с этим вызовом ошибка сильно реже, т.к. хотя бы поток сервера уже стартанул. По хорошему тут бы дождаться что сокет забиндили уже, тогда ошибок бы совсем не было. Пока поставил костыли в виде piMinSleep после PIEthernet::listen
peri4 commented 2026-03-18 10:55:17 +03:00
Owner
Copy Link
Copy Source

зачем серверу вообще чего-то ждать? для тестов?? я как обычно

зачем серверу вообще чего-то ждать? для тестов?? я как обычно
andrey commented 2026-03-18 12:02:08 +03:00
Author
Owner
Copy Link
Copy Source

ага, это в тестах

ага, это в тестах
andrey reviewed 2026-03-18 09:05:21 +03:00
tests/client_server/client_server_test.cpp
@@ -148,3 +150,3 @@
TEST(ClientServer, DynamicClients) {
auto const loop_timeout = 100_ms;
auto const loop_timeout = 10_ms;
andrey commented 2026-03-18 09:05:21 +03:00
Author
Owner
Copy Link
Copy Source

теперь эти тесты выполняются 600мс вместо 2600мс раньше, и всегда стабильны (раньше ~10% падало), проверял стабильность скриптом
for i in {1..100}; do echo -n "."; ctest --test-dir build/tests -R ClientServer.DynamicClients --output-on-failure 2>&1 | grep -q "Failed" && echo " FAILED $i" && echo "FAIL!"; done; echo " All OK!"

теперь эти тесты выполняются 600мс вместо 2600мс раньше, и всегда стабильны (раньше ~10% падало), проверял стабильность скриптом `for i in {1..100}; do echo -n "."; ctest --test-dir build/tests -R ClientServer.DynamicClients --output-on-failure 2>&1 | grep -q "Failed" && echo " FAILED $i" && echo "FAIL!"; done; echo " All OK!"`
peri4 merged commit 35140ee002 into master 2026-03-18 11:08:49 +03:00
andrey deleted branch some_fixes 2026-03-18 12:03:03 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
andrey
Labels
Clear labels
Баг
Косяк
 В работе
Ставим как приступили к выполнению
 Назначена
Ставим когда назначен исполнитель
 Отложено
Выполнение задачи отложено или ожидает других
 Сделано
Ставим когда выполнили задачу, и ожидаем одобрения
 Тестируется
Ставим когда отправили задачу на тестирование
 Требует обсуждения
Задача вызывает трудности или вопросы
 Фича
Новый функционал
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
andrey (Бычков Андрей) drone Gama (Дима Федоренко) jenkins (jenkins) maakshishov (Максим Шишов) peri4 (Пелипенко Иван) tambaru
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SHS/pip#198