[PATCH] Save/Load m_pSource in BTinfo.cpp

After you have edited the source code, post your patch here.
Forum rules
Home | Wiki | Rules

[PATCH] Save/Load m_pSource in BTinfo.cpp

Postby ivan386 » 13 Jan 2010 23:19

It need to continue share info of torrent after restart of programm.
recode3.zip
(25.59 KiB) Downloaded 75 times
Last edited by ivan386 on 11 Mar 2010 13:35, edited 3 times in total.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

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

Postby ivan386 » 02 Feb 2010 11:02

ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

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

Postby raspopov » 02 Feb 2010 18:47

Хм... а зачем вообще эта проверка при записи? и почему ты используешь ручную проверку, а не честный разбор при помощи BENode?
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30

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

Postby ivan386 » 02 Feb 2010 20:27

Там может оказаться недогруженный info.

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

Зачем использовать кучу объектов там где может справится одна функция.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

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

Postby raspopov » 03 Feb 2010 04:52

Это называется повторное использование кода! Пожалуйста, сделай код универсальным, нового кода должно быть как можно меньше.
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30

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

Postby ivan386 » 17 Feb 2010 21:43

Переписал. Много чего изменено. Надеюсь не напортачил.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

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

Postby ivan386 » 11 Mar 2010 13:38

Отредактированно.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

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

Postby cyko_01 » 11 Mar 2010 13:59

If I am not mistaken, development discussion and code submission are english language forums ;)
User avatar
cyko_01
 
Posts: 938
Joined: 13 Jun 2009 15:51


Return to Code Submission

Who is online

Users browsing this forum: No registered users and 1 guest