Add friends table and implement initial tox data save/load#7
Add friends table and implement initial tox data save/load#7Gravechapa wants to merge 1 commit intogravechapa/message_dbfrom
Conversation
| qDebug() << "Request received"; | ||
| tox_friend_add_norequest(tox_c, public_key, NULL); | ||
| uint32_t friend_id = tox_friend_add_norequest(tox_c, public_key, NULL); | ||
| if (friend_id == UINT32_MAX) |
There was a problem hiding this comment.
std::numeric_limits<uint32_t>::max()
| { | ||
| savedata_file.write(reinterpret_cast<char*>(savedata.data()), savedata.size()); | ||
| } | ||
| tox_kill(_tox); |
There was a problem hiding this comment.
Безопасно так делать, если мы создали объект, но не успели аутентифицироваться?
There was a problem hiding this comment.
Нет, но мы так не делаем. Там вообще нужна хорошая обработка ошибок. Пока просто чтобы работало сделал.
There was a problem hiding this comment.
Решается банальным if-ом разве нет?
There was a problem hiding this comment.
Там надо переделывать систему, чтобы в любом случае данные не повредились. Плюс бэкапы и более частое сохранение.
There was a problem hiding this comment.
Я пока занимаюсь функционалом. Обработка ошибок сейчас не самое важное. Там еще 20 раз код поменяется…
There was a problem hiding this comment.
Оставь комментарий с TODO И заведи ишью
There was a problem hiding this comment.
TODO - плохая вещь. В мастер заливать буду, заведу.
| Tox_Options *options = tox_options_new(NULL); | ||
| if (!options) | ||
| { | ||
| throw std::runtime_error("Can't initialize tox options"); |
There was a problem hiding this comment.
Наверняка в токсе есть способ получить ошибку, выводи её вместе с сообщением
| ToxModel::~ToxModel() | ||
| { | ||
| _finalize = true; | ||
| _finalize.store(true); |
There was a problem hiding this comment.
Ее нет. Я просто поменял на всякий случай. У меня проблемы с атомиком были, но это на сколько я понимаю касается только чтения.
There was a problem hiding this comment.
Оставит комментарий по поводу чего?
There was a problem hiding this comment.
Могу вернуть если тебя это очень напрягает.
2 participants
No description provided.