Page 1 of 1

[BugFix] Check status display

PostPosted: 31 Jan 2015 12:11
by ivan386

Re: [BugFix] Check status display

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

Re: [BugFix] Check status display

PostPosted: 01 Feb 2015 17:38
by ivan386

Re: [BugFix] Check status display

PostPosted: 01 Feb 2015 18:40
by raspopov
Это понятно, а зачем DownloadWithTorrent.cpp-то менять?

Re: [BugFix] Check status display

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

Re: [BugFix] Check status display

PostPosted: 02 Feb 2015 04:46
by raspopov
Если это править, то логично и в других местах тоже, и с TigerTree, и с ED2K хэшсетами тоже самое же...

Re: [BugFix] Check status display

PostPosted: 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

PostPosted: 02 Feb 2015 07:54
by ivan386

Re: [BugFix] Check status display

PostPosted: 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, проверь, пожалуйста, что я всё правильно сделал.