mc-dev
mc-dev@conference.jabber.ru
[03:46:30] monkey зашёл в конференцию
[04:11:53] smind зашёл в конференцию
[04:42:13] andrew_b зашёл в конференцию
[04:43:15] monkey вышел из конференции
[05:45:37] theMIROn вышел из конференции: Этого участника выгнали из комнаты за то, что он послал присутствие с ошибкой: remote-server-not-found
[06:22:49] ASM вышел из конференции
[06:22:51] ASM зашёл в конференцию
[06:30:31] slavazanko/h зашёл в конференцию
[06:56:54] <andrew_b> Всем привет.
[06:57:43] andrew_b вышел из конференции
[06:57:45] andrew_b зашёл в конференцию
[06:58:23] <andrew_b> После #3026 появились другие проблемы с поиском.
[07:01:48] <andrew_b> Дело не в самом движке, а в его использовании везде, кроме поиска по файлам.
[07:01:57] <andrew_b> Например.
[07:05:39] <andrew_b> Локаль KOI8-R.
Файл в UTF-8 с таким содержимым:
йцукен
Йцукен
Если искать "йц" (вне зависимости от того, регистрозависимо или нет), то только "йц" и находится. "Йц" не неходится совсем.
Если включить "Все кодировки", то не находится вообще ничего.
[07:07:28] <andrew_b> Проблема из-за того, что что в редакторе/вьювере/диффвьювере слишком много перекодировок текста.
В поиске по файлам перекодировок вообще нет, и всё ищется прекрасно.
[07:07:42] <andrew_b> Вариантов решения два.
[07:09:21] <andrew_b> Первый. В mc_search_new () строка должна быть в экранной кодировке. Всегда и независимо ни от чего.
[07:10:34] <andrew_b> Второй. Нужно добавить в mc_search_new() ещё один параметр -- кодировку строки и её использовать в движке поиска вместо cp_display.
[07:12:10] <andrew_b> По факту, у нас в большинстве случаем данные для поиска берутся с экрана (кроме поиска в mc.ext и, может быть, где-то ещё), поэтому первый вариант вполне себе.
[07:12:24] <andrew_b> Тем более там только латиница.
[07:12:41] <andrew_b> Патч на этот случай у меня есть.
[07:12:51] <andrew_b> Второй вариан более универсален.
[07:13:50] <slavazanko/h> > Нужно добавить в mc_search_new() ещё один параметр
это, пожалуй, лучше всего
[07:14:52] <andrew_b> Да, тем более что можно следать
if (charset == NULL)
charset = cp_display;
[07:21:04] slavazanko/h вышел из конференции
[07:21:22] <andrew_b> Пока патч такой:
http://www.pastebin.ca/2439773
[07:24:53] slavazanko/h зашёл в конференцию
[07:29:40] slavazanko/h вышел из конференции
[07:34:14] slavazanko/h зашёл в конференцию
[07:41:16] <slavazanko> is_all_charsets = mcdiffview_search_options.all_codepages
|| cp_source != cp_display
то есть, поиск по всем кодировкам будет врубаться автоматом, если кодировка источника не совпадает с кодировкой отображения?
[07:41:36] <andrew_b> Да.
[07:41:48] <andrew_b> Лучшего варианта я не придумал.
[07:42:05] <andrew_b> Чтобы не трокать сам движок поиска.
[07:42:16] <andrew_b> трогать
[07:44:09] <slavazanko> может, всё-таки тронем?
[07:44:17] D-ion вышел из конференции: Replaced by new connection
[07:44:19] D-ion зашёл в конференцию
[07:44:57] <andrew_b> Ну, я уже трогаю.
[07:45:10] <slavazanko> ок
[08:01:09] <andrew_b> С другой стороны, поиск с конвертацией в cp_source, как сейчас, тоже не очень хорошо.
cp_source -- это глобальная переменная, то есть одна на всех. А если у нас открыто несколько редакторов в разных кодировках, то могут возникнуть неожиданности при поиске.
[08:22:18] <andrew_b> http://www.pastebin.ca/2439831
[08:51:54] slavazanko/h вышел из конференции
[08:53:11] slavazanko/h зашёл в конференцию
[10:13:41] <andrew_b> Ну так что?
[13:24:57] <slavazanko/h> я тут. Надо избавляться от cp_source в поисковом движке. передавать отдельно кодировку
[13:25:24] <andrew_b> cp_source там нет.
[13:25:29] <andrew_b> Есть cp_display.
[13:26:23] <slavazanko/h> можно просто оставить функции как есть и создать новые функции, которые примут дополнительные параметры перекодировки
[13:26:40] <andrew_b> Не, не надо.
[14:01:19] imho2 зашёл в конференцию
[14:14:56] imho2 вышел из конференции
[14:45:26] monkey зашёл в конференцию
[15:23:23] iliamaslakov зашёл в конференцию
[15:59:35] iliamaslakov вышел из конференции
[16:54:07] <andrew_b> slavazanko/h: давай за Trac поговорим?
[16:54:08] smind вышел из конференции: Этого участника выгнали из комнаты за то, что он послал сообщение об ошибке другому участнику: service-unavailable
[16:56:19] <andrew_b> Может, стоит снести trac-ticketdelete-plugin, раз это в самом траке есть?
[16:58:25] <andrew_b> И как уменьшить количество регистрирующихся пустышек? Может, капчу каккую-нибудь включить? Их вон сколько в TracSpamFilter.
[16:59:21] <andrew_b> Например, ExpressionCaptcha — captcha in the form of a human readable numeric expression.
[17:10:02] <andrew_b> Оказывается, в AdvancedTicketWorkflowPlugin можно маркировать дубликаты:
http://trac-hacks.org/wiki/AdvancedTicketWorkflowPlugin
См. пример xref в самом конце.
[17:11:58] andrew_b вышел из конференции
[17:41:20] iliamaslakov зашёл в конференцию
[20:48:38] iliamaslakov вышел из конференции
[20:52:38] monkey вышел из конференции
