Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Рефакторинг] Изменить алгоритм проброса feat параметра в UserConnectionListener::Supports #1149

Open
pavel-pimenov opened this issue Aug 24, 2015 · 5 comments

Comments

@pavel-pimenov
Copy link
Owner

From Pavel.Pimenov@gmail.com on July 26, 2013 16:08:59

  1. Сначала полученная строка через токенайзер превращается в StringList& feat
    а потом в методе
    void ConnectionManager::on(UserConnectionListener::Supports, UserConnection* conn, const StringList& feat) noexcept

снова склеивается в строку.
Не эффективно.

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=1112

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on July 28, 2013 09:32:21

Уже поправил, скоро залью :)

Status: Started
Owner: a.rain...@gmail.com

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on July 29, 2013 09:40:47

This issue was closed by revision r14788 .

Status: Fixed

@pavel-pimenov
Copy link
Owner Author

From Pavel.Pimenov@gmail.com on July 29, 2013 10:22:00

не эффективно. я вообще думаю тут разбивку по токенам делать не нужно
лишние релокации на vector-е

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on July 29, 2013 10:40:15

Что бы лишних выделений памяти не было, надо строчки на свои заменить, в которых вместо копирования куска памяти происходит инициализация указателей начала и конца этого куска. У меня кстати такие строчки есть уже, и есть планы прикрутить их к Флаю.

@pavel-pimenov
Copy link
Owner Author

From Pavel.Pimenov@gmail.com on November 12, 2013 07:33:08

Status: Accepted
Owner: Pavel.Pimenov@gmail.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant