Home
mc-dev
mc-dev@conference.jabber.ru
15.10.2011< ^ >

Тема: http://www.midnight-commander.org | Latest release: 4.8.0-pre2 | Latest stable release: 4.7.5.4
[00:19:23] slavazanko/h вышел из конференции
[00:33:00] slavazanko/h зашёл в конференцию
[03:10:38] * sergem читает проблемы обновления wget-а и думает, как только люди не извращаются, лишь бы не пользоваться нормальным пакетным менеджером с автоматическим разрешением зависимостей...
[03:35:38] alex-vl@jabber.org вышел из конференции
[03:42:34] alex-vl@jabber.org зашёл в конференцию
[05:28:20] alex-vl@jabber.org вышел из конференции: qutIM : Jabber plugin
[07:06:19] andrew_b зашёл в конференцию
[07:56:51] andrew_b установил(а) тему: http://www.midnight-commander.org | Latest release: 4.8.0-pre2 | Latest stable release: 4.7.5.5
[10:00:29] <andrew_b> keasy: насчёт чтение stdin.

В твоём патче есть несколько проверок (filename[0] == '-'). А если имя файла будет начинаться с '-', например,
mcview -file.ooo
[10:02:02] <andrew_b> А проверку из prepend_cwd_on_local() предлагаю перенести в mc_maybe_editor_or_viewer(), в case MC_RUN_VIEWER.
[10:03:56] <andrew_b> Хотя... Можно и оставить.
[10:04:57] <andrew_b> Или перенести? :)
[10:23:07] <keasy> andrew_b: А если имя файла будет начинаться с '-', например,
вроде так небывает :) но если бывает, конечно поправлю
[10:23:52] <andrew_b> Почему не бывает? Мало ли кто как файл назовёт?
[10:24:05] <andrew_b> Формально это не запрещено.
[10:26:33] <keasy> немогу найти POSIX filename спек. ну ладно
[10:27:55] <andrew_b> Насколько я знаю, в имени файла не могут быть только два символа: '/' и '\0'.
[10:29:39] <keasy> да все верно. эх
[10:30:19] <andrew_b> А, вот ещё что...

+ if (view->datasource == DS_STDIN_PIPE) /* stdin can't be "re-opened" */
+ return; /* and we can't do magic toggle without reopen, yet */

Наверное, надо и кнопку F8 запретить.
[10:31:37] <keasy> andrew_b: А проверку из prepend_cwd_on_local()
смотрю, помоему лучше оставить.
[10:31:53] <andrew_b> Чтобы в buttonbar'e не было метки.
[10:32:01] <keasy> Наверное, надо и кнопку F8 запретить.
согласен. а не подскажешь как это сделать?
[10:32:15] <andrew_b> 113 buttonbar_set_label (b, 8, view->magic_mode ? Q_ ("ButtonBar|Raw")
114 : Q_ ("ButtonBar|Parse"), keymap, (Widget *) view);
[10:32:32] <keasy> если не ставить лейбл, ничего не будет? или поставить пустой?
[10:34:04] <andrew_b> if (view->datasource == DS_STDIN_PIPE)
buttonbar_set_label (b, 8, NULL, NULL, NULL);
else
buttonbar_set_label (b, 8, view->magic_mode ? Q_ ("ButtonBar|Raw")
: Q_ ("ButtonBar|Parse"), keymap, (Widget *) view);
[10:34:29] <keasy> ага, ставим пустой, спасибо большое
[10:34:43] <keasy> https://www.midnight-commander.org/ticket/2630
[10:34:53] <keasy> а что насчет этого? это прямо в этомже месте когда
[10:36:08] <andrew_b> Лучше тут. Но надо проверить, если battonbar формируется до того, как будет выставлен datasource, то, возможно, придётся после загрузки принудительно обновить.
[10:36:50] <andrew_b> Как лучше -- надо смотреть.
[10:37:36] <andrew_b> Может, view->dpy_bbar_dirty = TRUE ьудет достаточно, а может, придётся mcview_update() лишний раз дёрнуть.
[10:37:54] <keasy> хорошо, я потестирую
[10:41:33] <keasy> а про #2630 - есть резон эту кнопку переименовать в Refresh для STDIO_PIPE
[10:41:43] <keasy> вроде бы это тоже место кода получится. могу оформить все вместе
[10:42:59] <andrew_b> В один патч всё лепить не надо. Лучше несколько мелких патчей/коммитов.
[10:43:22] <keasy> ну конечно, как другой
[10:44:55] <andrew_b> Для Refresh для STDIO_PIPE нужно отдельное действие. Если в keybind.h подходящего нет, надо будет вводить новое.
[10:45:46] <keasy> а. понял. окей тогда потом
[10:47:40] <andrew_b> Вьювет и сейчас умеет дочитывать из пайпа. Но не автоматически. Надо на End нажимать.
[10:48:01] <andrew_b> s/Вьювет/Вьювер
[10:48:07] <keasy> да, с этим и нет проблем
[10:48:34] <andrew_b> Тогда что рефреш должен делать?
[10:48:41] <keasy> проблема в том, что нажатие на кнопку Parse закрывает файл, а потом открывает снова
[10:48:54] <keasy> с случае с пайпами - это рефреш а не парс :)
[10:49:01] <andrew_b> Так запретили ж.
[10:49:20] <andrew_b> рефреш чего?
[10:49:22] <keasy> только для stdin пайпов. а бывают еще popen
[10:50:07] <andrew_b> Что долен делать рефреш при чтении из stdin?
[10:50:19] <keasy> из стдин ничего. должен быть вообще запрещен.
[10:51:32] <andrew_b> А, это другой тикет.
[10:51:40] <keasy> да :)
[10:52:11] <andrew_b> Протормозил.
[11:29:41] il.smind зашёл в конференцию
[11:32:03] <il.smind > доброе утро
[11:32:48] <andrew_b> il.smind : Какое уж утро? День давно.
[11:32:59] <andrew_b> il.smind : Привет.
[11:33:25] <andrew_b> il.smind : как насчёт https://www.midnight-commander.org/ticket/2626 ?
[11:36:52] <il.smind > а ты его того?
[11:37:24] <andrew_b> ?
[11:40:07] <il.smind > ну то что там я находил
[11:40:13] <il.smind > тэ поправил да?
[11:40:33] <andrew_b> Ты ничего не находил.
[11:41:34] <il.smind > нет?
[11:41:40] <il.smind > это другой тикет
[11:41:42] <il.smind > ?
[11:42:01] <il.smind > ну оно лишний раз эскейпило в комстроке
[11:42:10] <il.smind > другой?
[11:43:21] <andrew_b> Найди ещё раз.
[11:51:13] <il.smind > ок
[11:51:16] <il.smind > ща
[12:12:31] <il.smind > andrew_b: нашел
[12:13:16] <il.smind > лишняя эскейпизация в комстроке любезно включена
[12:13:57] <il.smind > в мастере лишней нет
[12:16:14] <il.smind > andrew_b: ваш ход
[12:22:08] <slavazanko/h> 1551_update_gpl_version
[12:23:39] theMIROn зашёл в конференцию
[12:23:56] <Зося Синицкая> RSS: Новости для Midnight Commander
• 2370_viewer_stdin_fix-2a.patch attached to Ticket #2370
http://www.midnight-commander.org/attachment/ticket/2370/2370_viewer_stdin_fix-2a.patch
[12:34:27] <il.smind > балин уже 20 мин ищу константу для длины utf
[13:00:37] <il.smind > похоже нет такой
[13:04:20] <andrew_b> il.smind : как воспроизвести?
[13:06:35] <il.smind > создай файл wew?ergt&dfds1
[13:07:24] <il.smind > потом в комстроке пиши
ls wew<ESC TAB>
[13:08:23] <il.smind > получается ls wew\\\?ergt\\\&dfds
[13:11:23] <il.smind > в остальных местах всё ок
[13:32:26] il.smind вышел из конференции: Компьютер вошёл в спящий режим
[13:36:44] <andrew_b> il.smind : не воспроизводится.
[13:39:11] <andrew_b> slavazanko/h: mc.spec.in
-License: GPLv2
+License: GPLv3

Наверное, надо GPL3+
[13:53:40] il.smind зашёл в конференцию
[14:19:12] theMIROn вышел из конференции
[14:22:12] <andrew_b> il.smind : проверил в локалях KOI8-R и UTF-8. ls wew\\\?ergt\\\&dfds не получается ни разу.
[14:23:30] <il.smind > andrew_b: во ком строке?
[14:23:40] <andrew_b> Да.
[14:24:14] <il.smind > исключая вариант что ты меня обманываешь...
[14:24:43] <andrew_b> [15:11]<il.smind > балин уже 20 мин ищу константу для длины utf
[15:37]<il.smind > похоже нет такой
MB_LEN_MAX не она?
[14:25:33] <andrew_b> Не, не она.
[14:25:55] <il.smind > она
[14:26:10] <il.smind > ЗАМЕЧАНИЯ
Выражения MB_LEN_MAX и sizeof(wchar_t) абсолютно не связаны друг с другом. В GNU библиотеке libc MB_LEN_MAX обычно равно 6-и, тогда как sizeof(wchar_t) обычно равно 4-м.
[14:26:13] <andrew_b> 30 /* Maximum length of any multibyte character in any locale.
31 We define this value here since the gcc header does not define
32 the correct value. */
33 #define MB_LEN_MAX<---->16
[14:26:23] <andrew_b> 16, а не 6.
[14:27:16] <il.smind > #define MB_LEN_MAX<---->16
они чо психи?!
[14:27:27] <andrew_b> ХЗ.
[14:27:33] <andrew_b> limits.h
[14:27:36] <il.smind > где они насчители еще 10
[14:27:52] <il.smind > шо за буков такой
[14:28:13] <il.smind > ок
[14:28:24] <il.smind > будет у нас своя
[14:29:09] <il.smind > но блин... if (in->charpoint >= MB_LEN_MAX)
[14:29:15] <il.smind > это же я писал
[14:29:28] <andrew_b> MB_LEN_MAX это же не только для UTF-8.
[14:31:00] <il.smind > есть ещё MB_CUR_MAX
[14:31:11] <il.smind > но чо то еще более стрёмно
[14:31:22] <il.smind > так вот
[14:32:04] <il.smind > на счет лишнего эскейпленья
[14:33:20] <il.smind > в мастере давай проверим
[14:33:52] <andrew_b> То же самое.
[14:34:24] <il.smind > wew до чего расширяется?
[14:35:02] <il.smind > у меня получается ls wew\?ergt\&dfds
[14:35:12] <il.smind > в мастере
[14:35:22] <andrew_b> ls wew\?ergt\&dfds1
[14:35:55] <andrew_b> В бранче то же самое.
[14:39:01] ASM вышел из конференции
[14:43:52] <il.smind > в бранче ls wew\\\?ergt\\\&dfds ...
[14:44:02] <il.smind > надо проверять настройки
[14:44:13] <il.smind > тебе к спеху?
[14:44:27] <andrew_b> Нет.
[14:44:33] <il.smind > надо искать что отличается
[14:45:22] <andrew_b> Давай уберём ini.
[14:47:42] <andrew_b> il.smind : убрал. Ничего не изменилось.
[14:47:59] <andrew_b> il.smind : расскажи, как ты создавал файл?
[14:49:25] <il.smind > в mc выполнил >wert\?qwe\&qwe
[14:53:24] <andrew_b> Прикольно. У меня в KOI8 проходит, в UTF-8 нет.
[14:53:40] <andrew_b> $ >wert\?qwe\&qwe
Пропущена команда.
[14:54:03] <andrew_b> В коях созайтся файл.
[14:54:46] <andrew_b> Чёйта?
[14:55:04] <il.smind > не знаю
[14:57:38] <il.smind > но помню в некоторых случаях надо пользователя с родной кодировкой
[14:57:52] <il.smind > чтобы изначально была нужная
[14:57:58] <il.smind > ты же мне говорил
[14:58:30] <il.smind > недостаточно просто сделать LANG=тыры-пыры
[14:58:47] <andrew_b> Причём тут кодировка?
[14:59:02] <andrew_b> У меня даже в шелле не работает.
[15:00:25] <andrew_b> Сработало
echo > wert\?qwe\&qwe
[15:17:43] <il.smind > Прикольно. У меня в KOI8 проходит, в UTF-8 нет.
[15:17:52] <il.smind > я откуда знаю
[16:16:28] slavazanko/h вышел из конференции
[16:31:15] slavazanko/h зашёл в конференцию
[16:54:36] <il.smind > andrew_b: https://www.midnight-commander.org/ticket/2372#comment:7
[16:54:42] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2372 (Editor sometimes shows russian UTF-8 chars as two dots) updated
http://www.midnight-commander.org/ticket/2372#comment:7
[16:55:53] <il.smind > Зося Синицкая: спасибо зося
[16:56:04] <Зося Синицкая> il.smind : =-Oподелись
[17:00:11] <andrew_b> + gchar utf8_buf[3 * 6 + 1];

+ for (i = 0; i < 18; i++)
+ utf8_buf[i] = edit_get_byte (edit, byte_index + i - 12);
+ utf8_buf[18] = '\0';
+
+ valid_char = utf8_buf + 12;
[17:00:40] <andrew_b> + return utf8_buf[11];
[17:02:30] <andrew_b> edit_get_buf_ptr не до конца удалил.
[17:03:08] <andrew_b> И ни одного комментария. Что это за 18, 12, 11...
[17:03:29] <andrew_b> Почему 18, зачем...
[17:03:39] <andrew_b> Сам же вчера спрашивал.
[17:09:48] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2631 ([patch ready] mcedit puts cursor in the middle of multibyte utf8 character) updated
http://www.midnight-commander.org/ticket/2631#comment:1
[18:00:32] <il.smind > andrew_b: надо константу
[18:01:06] <il.smind > ну или MB_LEN_MAX
[18:01:35] <il.smind > как на MB_LEN_MAX ?
[18:01:54] <il.smind > ну и что что 16
[18:01:59] <Зося Синицкая> RSS: Новости для Midnight Commander
• 2370_viewer_stdin_fix-3a.patch attached to Ticket #2370
http://www.midnight-commander.org/attachment/ticket/2370/2370_viewer_stdin_fix-3a.patch
[18:06:12] <il.smind > keasy тык
[18:08:07] <keasy> брык
[18:14:21] <andrew_b> il.smind : 18 = 6*3, да?
[18:14:27] <andrew_b> 12 = 6*2
[18:16:07] <andrew_b> keasy: ящетаю, что
#ifdef HAVE_UNISTD_H
не нужно. У нас много где инклюдится unistd.h без всяких условий.
[18:16:34] <keasy> а, хорошо
[18:32:04] <Зося Синицкая> RSS: Новости для Midnight Commander
• mcedit.less.sh attached to Ticket #2629
http://www.midnight-commander.org/attachment/ticket/2629/mcedit.less.sh
• 2370_viewer_stdin_fix-3b.patch attached to Ticket #2370
http://www.midnight-commander.org/attachment/ticket/2370/2370_viewer_stdin_fix-3b.patch
[19:00:56] andrew_b вышел из конференции
[19:05:11] <il.smind > [20:14:30] <andrew_b> il.smind : 18 = 6*3, да?
[20:14:36] <andrew_b> 12 = 6*2
афтар сказал что да
[19:23:02] slavazanko/h вышел из конференции
[19:25:01] slavazanko/h зашёл в конференцию
[22:32:48] il.smind вышел из конференции
Powered by freQ Powered by Python Valid XHTML 1.0 Transitional Valid CSS!