Total Commander Forum Index Total Commander
Форум поддержки пользователей Total Commander
Сайты: Все о Total Commander | Totalcmd.net | Ghisler.com | RU.TCKB
 
 RulesRules   SearchSearch   FAQFAQ   MemberlistMemberlist   UsergroupsUsergroups   RegisterRegister 
 ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

Single Post  Topic: [wlx/wdx/wcx] TCTorrent 
Author Message
Loopback



PostPosted: Fri Apr 06, 2012 22:16    Post subject: Reply with quote

Фикс - 2 строчки кода. Стоило из-за этого спорить?

Что касается стандартов. Спецификация торрентов допускает любые нестандартные сегменты. При этом парсер, разумеется, не должен падать, если встретит такой сегмент. Обрабатывать его или нет - это дело разработчика парсера, но если не обрабатывать - значит нужно корректно пропускать.

TCTorrent - плагин для получения информации о торрент-файле, в этом случае информация лишней не бывает. Хуже от того, что пользователь будет знать о наличии этих ссылок в файле точно не будет. Так или иначе, было принято решение показывать содержимое этого сегмента. То, что он нестандартный, я вообще узнал только сейчас.

Итак, фича уже добавлена, но неправильно, в результате плагин падает. После обнаружения ошибки есть два пути: добавить 2 строки кода, исправляющие проблему, или убрать поддержку "url-list" совсем, причем для этого надо убрать несколько кусков кода в разных местах: код парсера, список для URL, вывод поля, локализацию поля. По-моему, выбор пути очевиден.
View user's profile Send private message


Powered by phpBB © 2001, 2005 phpBB Group