Size: a a a

Конкурсы Telegram

2020 March 10

MR

Markus Ra in Конкурсы Telegram
Max Fomichev
Основная проблема в TgVoipEndpoint
Пожалуйста, посмотрите обновленные исходники здесь:
https://github.com/telegramdesktop/tgvoip-test-suite/commit/f979dc3dcb1d4442e88707ae4551b025a957de6c

Запускаемый tgvoipcall в репозитории был правильным.
источник

MF

Max Fomichev in Конкурсы Telegram
Markus Ra
Пожалуйста, посмотрите обновленные исходники здесь:
https://github.com/telegramdesktop/tgvoip-test-suite/commit/f979dc3dcb1d4442e88707ae4551b025a957de6c

Запускаемый tgvoipcall в репозитории был правильным.
Спасибо. Ну, и еще хочу напомнить, что std::chrono появился в одном стандарте с auto. Если вы используете второе, то зачем городить "uint64_t get_microseconds()"?
источник

MF

Max Fomichev in Конкурсы Telegram
Markus Ra
Пожалуйста, посмотрите обновленные исходники здесь:
https://github.com/telegramdesktop/tgvoip-test-suite/commit/f979dc3dcb1d4442e88707ae4551b025a957de6c

Запускаемый tgvoipcall в репозитории был правильным.
Ничего, что коммит был сделан 14 минут назад?
источник

MR

Markus Ra in Конкурсы Telegram
Max Fomichev
Ничего, что коммит был сделан 14 минут назад?
Вот и пишу: посмотрите обновлённый исходник; а запускаемый уже был ОК.
источник

MF

Max Fomichev in Конкурсы Telegram
Markus Ra
Вот и пишу: посмотрите обновлённый исходник; а запускаемый уже был ОК.
Про "запускаемый" - пропустил. ОК, спасибо за ответ и исправления.
источник

MF

Max Fomichev in Конкурсы Telegram
И опять С99 :) Уж простите за занудство, но так не пишут на С++:
TgVoipEdpointHost host = {
           .ipv4 = std::string(buf),
           .ipv6 = std::string(),
};
источник

IG

Ilya Gusev in Конкурсы Telegram
Max Fomichev
И опять С99 :) Уж простите за занудство, но так не пишут на С++:
TgVoipEdpointHost host = {
           .ipv4 = std::string(buf),
           .ipv6 = std::string(),
};
Только будут же писать, это часть c++20
источник

IG

Ilya Gusev in Конкурсы Telegram
Да и почему бы не использовать удобную фичу?
источник

MF

Max Fomichev in Конкурсы Telegram
Ilya Gusev
Да и почему бы не использовать удобную фичу?
Потому, что -Werror не даст скомпилировать этот код.
источник

MF

Max Fomichev in Конкурсы Telegram
Ilya Gusev
Только будут же писать, это часть c++20
Предлагаете С++20 на Debian 10.1 использовать?
источник

IG

Ilya Gusev in Конкурсы Telegram
Max Fomichev
Предлагаете С++20 на Debian 10.1 использовать?
Компиляторы, которые его поддерживают - да (только их ещё нет)
источник

IG

Ilya Gusev in Конкурсы Telegram
Max Fomichev
Потому, что -Werror не даст скомпилировать этот код.
А конкретно это можно обойти, отрубив ворнинг
источник

MF

Max Fomichev in Конкурсы Telegram
Ilya Gusev
А конкретно это можно обойти, отрубив ворнинг
Спасибо, буду знать! :)
источник

GK

Gregory Klyushnikov in Конкурсы Telegram
блин, в новом коммите изменения просто замечательные — в почти каждом файле какие-нибудь отступы потрогали, поэтому изменилось вообще всё и у меня конфликты =/
источник

GK

Gregory Klyushnikov in Конкурсы Telegram
вот такие изменения 👍
источник

GK

Gregory Klyushnikov in Конкурсы Telegram
а, я понял — это моё же смержили, но с кривыми отступами 🙈
источник

MF

Max Fomichev in Конкурсы Telegram
Gregory Klyushnikov
а, я понял — это моё же смержили, но с кривыми отступами 🙈
Ща Илья объяснит, почему такие отступы делать надо :)
источник

GK

Gregory Klyushnikov in Конкурсы Telegram
они там небось ещё и пробелами, а не табами))
источник

AE

Andrey Egorov in Конкурсы Telegram
Правильнее было бы, конечно, сначала отрефакторить, заменить табы пробелами отдельным комитетом, а затем уже мерджить исправления.
источник

MF

Max Fomichev in Конкурсы Telegram
Gregory Klyushnikov
они там небось ещё и пробелами, а не табами))
ide все табы на пробелы меняют уже лет 15 как...
источник