Page 1 of 1
[BugFix] Check status display

Posted:
31 Jan 2015 12:11
by ivan386
Re: [BugFix] Check status display

Posted:
01 Feb 2015 16:34
by raspopov
Не понял смысла, поясни.
(new BYTE [ 0 ] вобще-то допустимая инструкция, если что)
Re: [BugFix] Check status display

Posted:
01 Feb 2015 17:38
by ivan386
Re: [BugFix] Check status display

Posted:
01 Feb 2015 18:40
by raspopov
Это понятно, а зачем DownloadWithTorrent.cpp-то менять?
Re: [BugFix] Check status display

Posted:
01 Feb 2015 20:08
by ivan386
Так не будет иллюзии что есть хешсет торрента. Я то все думал откуда там мусор. Если нулевой хешсет задан спциально то оставь только DownloadWithTiger.cpp. Но мне кажется нулевой хешсет будет за зря память фрагментиовать и сбивать столку.
Re: [BugFix] Check status display

Posted:
02 Feb 2015 04:46
by raspopov
Если это править, то логично и в других местах тоже, и с TigerTree, и с ED2K хэшсетами тоже самое же...
Re: [BugFix] Check status display

Posted:
02 Feb 2015 05:45
by ivan386
У торрента выделение нулевого хешсета при btih в магните гарантированно происходит. У остальных насколько я понимаю только если нам подсунут нулевой хешсет а до этого сохраняется нулевой указатель.
В общем надо все места.
if ( m_p[Torrent|Tiger|Hashset]Block )
заменить на
if ( m_p[Torrent|Tiger|Hashset]Block && m_n[Torrent|Tiger|Hashset]Block )
либо макрос написать
Попробую сделать полностью.
Re: [BugFix] Check status display

Posted:
02 Feb 2015 07:54
by ivan386
Re: [BugFix] Check status display

Posted:
09 Feb 2015 17:43
by raspopov
I hope that some time I will find a time for Shareaza to analyze this issue completely but now I committed a fix inspired by your suggestions:
r9532.
ivan386, проверь, пожалуйста, что я всё правильно сделал.