Page 1 of 1

[PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 13 Jan 2010 23:19
by ivan386
It need to continue share info of torrent after restart of programm.
recode3.zip
(25.59 KiB) Downloaded 96 times

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 02 Feb 2010 11:02
by ivan386

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 02 Feb 2010 18:47
by raspopov
Хм... а зачем вообще эта проверка при записи? и почему ты используешь ручную проверку, а не честный разбор при помощи BENode?

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 02 Feb 2010 20:27
by ivan386
Там может оказаться недогруженный info.

CheckInfoData устанавливает начало (m_nInfoStart) и размер (m_nInfoSize) info в буфере. Логичнее конечно их записывать и читать (в дальнейшем так и будет). Но на данный момент m_nInfoSize устанавливается до полной загрузки (у себя я уже это изменил и значение m_nInfoSize устанавливается только в CheckInfoData). Если CheckInfoData не пройдет при записи соответственно при загрузке потеряем m_nInfoStart и m_nInfoSize а в m_pSource будет "мусор".

Зачем использовать кучу объектов там где может справится одна функция.

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 03 Feb 2010 04:52
by raspopov
Это называется повторное использование кода! Пожалуйста, сделай код универсальным, нового кода должно быть как можно меньше.

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 17 Feb 2010 21:43
by ivan386
Переписал. Много чего изменено. Надеюсь не напортачил.

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 11 Mar 2010 13:38
by ivan386
Отредактированно.

Re: [PATCH] Save/Load m_pSource in BTinfo.cpp

PostPosted: 11 Mar 2010 13:59
by cyko_01
If I am not mistaken, development discussion and code submission are english language forums ;)