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

Тема: http://www.midnight-commander.org | Latest release: 4.8.1 | Latest stable release: 4.7.5.6
[05:13:00] andrew_b зашёл в конференцию
[06:04:46] <iliamaslakov> >andrew_b: Код исправляй.
что исправлять?
[06:21:57] <andrew_b> Я вчера вечером всё написал.
[06:22:51] <iliamaslakov> нет
[06:23:01] <andrew_b> Лог почитай.
[06:23:16] <iliamaslakov> ага
[06:23:21] <iliamaslakov> вижу
[06:28:57] <iliamaslakov> > andrew_b:
>+ if (tmp->len != 0)
>+ text = g_string_free (tmp, FALSE);
>
>Тут мемлик.
почему?
[06:30:33] <andrew_b> Потому что выше text =
[06:31:34] <iliamaslakov> сделал так
665 if (tmp->len != 0)
666 {
667 g_free (text);
668 text = g_string_free (tmp, FALSE);
669 }
[06:31:43] <andrew_b> Угу.
[06:34:02] <iliamaslakov> >+ *color = EDITOR_BOLD_COLOR;
>В скин?
можно, но и так не плохо
[06:36:57] <iliamaslakov> + char token[highlight_word_length + 1];
вот это я вообще не понял
[06:37:17] <iliamaslakov> highlight_word_length это же переменная
[06:37:53] <iliamaslakov> т.е. каждый раз оно переопределяет размер?
[06:38:04] <iliamaslakov> это вообще нормально?
[06:38:16] <andrew_b> Кстати.
[06:38:23] <andrew_b> Хороший вопрос.
[06:38:55] <andrew_b> У нас -std=c99, в нём, видимо, это можно. Но надо почитать стандарт.
[06:41:20] <iliamaslakov> может переделать?
[06:41:40] <iliamaslakov> чойта мне не нравится такая неопределённость
[06:42:07] <andrew_b> Ну переделай.
[06:42:18] <andrew_b> Вопросов меньше будет.
[06:42:21] <iliamaslakov> я не понимаю где оно будет размещать это дело на всяких других платформах
[06:42:36] <andrew_b> Где-где. В стеке.
[06:42:51] <iliamaslakov> да
[06:42:53] <iliamaslakov> но
[06:43:30] <iliamaslakov> вот в прошлый раз highlight_word_length была равно 10
[06:43:45] <iliamaslakov> а потом стало 100
[06:43:56] <andrew_b> Ну и что?
[06:43:57] <iliamaslakov> оно новое место будет в стеке искать?
[06:44:07] <iliamaslakov> куда 100 влезет?
[06:44:19] <andrew_b> Переделай на хип.
[06:46:16] <iliamaslakov> char token[BUF_MEDIUM + 1];
достаточно будет?
[06:46:57] <andrew_b> Не знаю.
[07:42:48] slavazanko/h зашёл в конференцию
[08:11:37] slavazanko/h вышел из конференции
[08:11:43] slavazanko/h зашёл в конференцию
[08:42:15] IceMan зашёл в конференцию
[08:47:57] <slavazanko/h> всем привет
[08:48:11] <IceMan> привет
[08:48:20] <andrew_b> Привет.
[09:51:00] <andrew_b> И тишина...
[09:51:50] <slavazanko/h> ща... я чуток коммичу в 2421_syntax_occurence_hl
[09:51:57] <slavazanko/h> скоро запушу
[09:55:51] <slavazanko/h> всё, подпушил
[09:59:13] <andrew_b> В exec_highlight_occurences утечка так и осталась.
[10:00:29] <slavazanko/h> точно
[10:00:30] <slavazanko/h> ща
[10:00:48] <andrew_b> Давайте переименуем exec_ в edit_
[10:00:58] <andrew_b> У меня навязчивая идея.
[10:07:15] <slavazanko/h> давайте. не вопрос
[10:08:21] <slavazanko/h> ща сделаю...
[10:13:12] <iliamaslakov> slavazanko/h, ты чо там партизанеж?
[10:13:22] <iliamaslakov> я ж еще не запушил...
[10:14:12] <slavazanko/h> ещё подпушил
[10:14:21] <slavazanko/h> Илья, расслабся и получай удовольствие
[10:14:22] <iliamaslakov> да вижу
[10:14:38] <iliamaslakov> уже в процессе
[10:14:58] <slavazanko/h> Я там убрал статические буфера
[10:15:14] <iliamaslakov> а... я как раз над ниму корпел, ок
[10:15:31] <andrew_b> iliamaslakov: спать надо меньше.
[10:16:59] <iliamaslakov> да с вами "не отворачивайся"...
[10:19:28] <iliamaslakov> + if ( edit_get_byte (edit, byte_index++) != highlight_word[i])
только через мой труп
[10:19:57] <slavazanko/h> что не так?
[10:20:13] <iliamaslakov> всё афигенно
[10:20:34] <iliamaslakov> код должен быть понятным даже мне
[10:20:53] <iliamaslakov> если он не прошел такой фильтр значит код плохой
[10:21:15] <slavazanko/h> подпушил
[10:21:33] <andrew_b> iliamaslakov: это фильтр плохой.
[10:21:38] <iliamaslakov> нет
[10:21:44] <iliamaslakov> фильтр хороший
[10:21:53] <slavazanko/h> ещё раз запуль
[10:21:58] <iliamaslakov> к счастью дело не в фильтре
[10:22:08] <iliamaslakov> а в нашем регламенте
[10:22:50] <iliamaslakov> о!
[10:22:56] <slavazanko/h> ?
[10:23:06] <iliamaslakov> значительно лучше
[10:23:12] <slavazanko/h> ок
[10:23:18] <iliamaslakov> такое трассировать можно без переделки
[10:25:26] <iliamaslakov> засквошу, ты не против?
[10:25:33] <slavazanko/h> да, конечно
[10:25:50] <slavazanko/h> git rebase -i --autosquash ...
[10:26:02] <slavazanko/h> или твой гит не поддерживает автосквоши
[10:26:03] <slavazanko/h> ?
[10:32:52] <iliamaslakov> не знаю, мне проще руками
[10:33:02] <slavazanko/h> там же много
[10:33:09] <slavazanko/h> ну да как знаешь
[10:33:10] <iliamaslakov> я помню ты говорил про fix for
[10:33:16] <iliamaslakov> не много
[10:33:24] <slavazanko/h> git commit --fixup <коммит>
[10:33:48] <iliamaslakov> выделяю по вертикали и вставляю букву f
[10:33:53] <slavazanko/h> а потом автосквошь просто пестня
[10:34:00] <iliamaslakov> дело 10 секунд
[10:34:04] <slavazanko/h> не спеши
[10:34:14] <slavazanko/h> там ещё надо выше одного коммита перенести всё
[10:34:23] <slavazanko/h> а то сейчас лихо в не тот коммит влупишь
[10:34:30] <iliamaslakov> само собой
[10:34:49] <iliamaslakov> перенёс
[10:36:09] <slavazanko/h> голосуем или есть ещё что?
[10:36:24] <iliamaslakov> Maksym Yehorov
Software Engineer at Microsoft
[10:36:47] <slavazanko/h> и чё? наши люди в голливуде
[10:38:23] iliamaslakov вышел из конференции: Replaced by new connection
[10:38:34] iliamaslakov зашёл в конференцию
[10:38:36] <andrew_b> Почему дефаулт в файле keymap в одном коммите, в дефаулт в коде в другом? Где логика, я вас внимательно спрашиваю?
[10:38:47] <iliamaslakov> сейчас я его как автора вписал
[10:39:05] <iliamaslakov> засквошу щас
[10:39:11] <andrew_b> Нихрена подобного, он уже не астор этого патча.
[10:39:28] <andrew_b> Его патч был перепахан.
[10:40:06] <andrew_b> И ещё. Мне дико не нравится такая манера

bindings for support highlight occurences of pattern.

Signed-off-by: Ilia Maslakov < il.smind@gmail.com>

optimization & code cleanup

Signed-off-by: Slava Zanko < slavazanko@gmail.com>
[10:40:47] <andrew_b> Какой смысл разделять комменты, если в коде всё равно не видно, где одно, а где другое?
[10:42:43] <slavazanko/h> да
[10:43:55] <slavazanko/h> http://svalko.org/data/2012_02_27_21_24_img_fotki_yandex_ru_get_5606_138238612_4b_0_6e303_c52382aa_orig.jpg
[10:45:03] <andrew_b> :))
[10:45:23] <slavazanko/h> оказывается, интернациональное явление
[10:45:34] <slavazanko/h> я-то думал, что национальные особенности...
[10:46:24] <andrew_b> Это разговорник для выезжающих за рубеж гопников.
[10:48:05] <iliamaslakov> такое ощущение что тут утечка
[10:48:16] <slavazanko/h> где?
[10:48:29] <iliamaslakov> 1601 char *text = NULL;
1602
1603 (void) edit;
1604 text = input_expand_dialog (_("Highlight occurences"),
1605 _("Text:"), MC_HISTORY_EDIT_OCCURENCES, "");

1626 highlight_word = text;
[10:49:03] <slavazanko/h> ну и?
[10:49:16] <slavazanko/h> перед этим старое значение g_free
[10:49:23] <slavazanko/h> потом присваиваем новое
[10:49:25] <slavazanko/h> всё гуд
[10:49:39] <slavazanko/h> указательна text никуда не теряется
[10:50:17] <slavazanko/h> память по указателю будет освобождена позже. Или при следующем сейнсе подсветки, или при выходе из редактора
[10:51:14] <iliamaslakov> не...
[10:51:18] <iliamaslakov> точно что то не так
[10:51:24] <slavazanko/h> ujdjhb
[10:51:28] <slavazanko/h> тьфу. говори
[10:51:58] <iliamaslakov> по крайней мере edit_highlight_deinit не всегда отрабатывает
[10:52:19] <iliamaslakov> подсвечиваем tmp
[10:52:28] <iliamaslakov> esc
[10:52:41] <iliamaslakov> снова f4
[10:52:55] <iliamaslakov> и видим что подсветка осталась
[10:52:58] <slavazanko/h> f4?
[10:53:07] <slavazanko/h> ну да, подсветка осталась
[10:53:18] <slavazanko/h> потому что text == NULL
[10:53:22] <slavazanko/h> и ничего не изменилось
[10:53:32] <iliamaslakov> ну выходим по esc из редактора
[10:53:43] <iliamaslakov> и снова открываем редактор по f4
[10:54:01] <slavazanko/h> гм.
[10:54:04] <slavazanko/h> странно
[10:54:18] <slavazanko/h> а в куда тогда запихнуть деинициализацию? или пусть так и будет?
[10:54:30] <slavazanko/h> типа. постоянная подсветка между редакторами
[10:54:35] <slavazanko/h> фишка, ёпт
[10:54:35] <andrew_b> Ненене.
[10:54:47] <iliamaslakov> если выходим по f10 то edit_highlight_deinit отрабатывает
[10:54:48] <andrew_b> Закрыли и забыли.
[10:55:00] <slavazanko/h> а что отрабатывает по esc?
[10:55:14] <iliamaslakov> andrew_b, а мне в первую очередь понравилось что подсветка сохранялась
[10:55:19] <slavazanko/h> это диалог сам перехватывает?
[10:55:28] <andrew_b> Да.
[10:55:34] <slavazanko/h> пилять
[10:56:07] <slavazanko/h> editcmd_dialogs.c:
80 dlg_stop (h);
[10:56:16] <slavazanko/h> это второй выход по esc?
[10:56:27] <slavazanko/h> а не
[10:56:33] <slavazanko/h> это для диалогов внутри редактора
[10:56:34] <andrew_b> Короче, предлагаю blocked by 2261.
[10:56:48] <slavazanko/h> Илья взвоет :)
[10:57:05] <iliamaslakov> зачем?
[10:57:45] <andrew_b> Потому что.
[10:57:48] <slavazanko/h> там много осталось?
[10:58:01] <andrew_b> Там более прямо сделано закрытие редактора.
[10:58:22] <slavazanko/h> оно блокедбай от 2608
[10:58:37] <andrew_b> slavazanko/h: там надо доделать скины и переделать последний коммит из-за новой vfs.
[10:58:44] <andrew_b> В 2261.
[10:58:49] <iliamaslakov> да проще потом одну строку перенести
[10:58:49] <slavazanko/h> понял
[10:58:55] <slavazanko/h> подмогу с этим
[10:59:00] <iliamaslakov> чем 2 гда ещё ждать
[10:59:13] <andrew_b> Я и сам справлюсь, надеюсь.
[10:59:35] <andrew_b> iliamaslakov: хватит новых фич для 4.8.2.
[10:59:43] <andrew_b> Одна vfs чего стоит.
[10:59:50] <andrew_b> iliamaslakov: голосуй!
[10:59:52] <iliamaslakov> эта маленькая но приятная
[11:00:31] <andrew_b> Мне сейчас нужно отъехать на часок.
[11:00:53] <iliamaslakov> andrew_b, за что голосовать?
[11:02:24] <slavazanko/h> 2695
[11:03:11] <iliamaslakov> я тестирую
[11:03:22] <iliamaslakov> или хватит уже?
[11:03:31] <slavazanko/h> тестируй
[11:05:52] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2736 (MC deletes the wrong file) updated
http://www.midnight-commander.org/ticket/2736#comment:3
[11:13:36] andrew_b вышел из конференции
[11:20:23] <iliamaslakov> давай деинит в выход из mc сделаем
[11:29:43] <iliamaslakov> slavazanko/h, чо скажешь?
[11:30:04] <slavazanko/h> хз. Стрёмно мне
[11:30:39] <slavazanko/h> чота я , Славик, очкую...
Андрюха придёт - и зарубит все наши планы на корню.
[12:17:15] <iliamaslakov> нас двое
[12:38:53] <slavazanko/h> а нигде нет штатного деинита редактора? Типа, как во вьювере
[12:51:02] <iliamaslakov> edit_clean
[12:51:18] <slavazanko/h> он вызывается при ескейпе?
[12:51:27] <iliamaslakov> в том числе
[12:51:38] <slavazanko/h> гм. Ляпи в туды
[12:53:11] <iliamaslakov> но мне то нравится что подсветка не пропадает
[12:53:28] <slavazanko/h> ляпи опцию
[12:53:53] <slavazanko/h> [editor]
persistent_highlight=true
[12:55:19] <iliamaslakov> у...
[12:55:47] <iliamaslakov> не надо
[12:56:18] <iliamaslakov> вляпил, пусть так
[12:57:08] <slavazanko/h> гуд. Интересно, я могу голосовать или как?
[12:58:58] <iliamaslakov> можешь
[12:59:47] <slavazanko/h> это ты мне в приват пытался?
[12:59:58] <iliamaslakov> да
[13:00:03] <iliamaslakov> и пытаюсь
[13:00:08] <slavazanko/h> оно сказало, что не поддерживается
[13:00:15] <iliamaslakov> ???
[13:00:18] <slavazanko/h> почему-то
[13:00:34] <iliamaslakov> еще раз написал
[13:00:36] <slavazanko/h> I currently only accept messages from people on my contact list - please request my authorization.
[13:01:06] <slavazanko/h> странно, ты у меня вроде в контакт-листе
[13:01:13] <slavazanko/h> наверное, пидгин глючет
[13:01:26] <slavazanko/h> тест словил приватом?
[13:01:28] <Зося Синицкая> slavazanko/h: Две полоски!
[13:01:37] <slavazanko/h> Зося Синицкая: дура, не тебе
[13:01:47] <Зося Синицкая> slavazanko/h: на нах кувалдой
[13:01:57] <slavazanko/h> бля, кто её обучал?
[13:02:41] <iliamaslakov> да
[13:03:05] <slavazanko/h> а я от тебя почему-то не могу получить
[13:06:32] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2737 (modarin256: a new 256-color skin for mc) created
http://www.midnight-commander.org/ticket/2737
• modarin-1.0.tar.gz attached to Ticket #2737
http://www.midnight-commander.org/attachment/ticket/2737/modarin-1.0.tar.gz
[13:36:42] <slavazanko/h> скины подвалило...
[13:37:27] <slavazanko/h> включим в 4.8.2
[14:12:26] IceMan вышел из конференции
[14:37:44] <iliamaslakov> да в 2421 есть проблема надо смотреть,так и не получилось найти косячок
[15:05:13] iliamaslakov вышел из конференции: offline
[15:11:08] IceMan зашёл в конференцию
[15:11:36] <IceMan> Как в MC по Alt+F3 вызывать внешнюю программу?
типа: view %f
[15:22:36] <Зося Синицкая> RSS: Новости для Midnight Commander
• modarin-1.1.tar.gz attached to Ticket #2737
http://www.midnight-commander.org/attachment/ticket/2737/modarin-1.1.tar.gz
[15:45:14] iliamaslakov зашёл в конференцию
[15:46:14] <iliamaslakov> на alt-f3 ничего не навешено
[16:19:05] iliamaslakov вышел из конференции: Machine going to sleep
[16:43:55] IceMan вышел из конференции
[17:42:14] andrew_b зашёл в конференцию
[17:53:59] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2736 (MC deletes the wrong file) updated
http://www.midnight-commander.org/ticket/2736#comment:1
• Ticket #2736 (MC deletes the wrong file) updated
http://www.midnight-commander.org/ticket/2736#comment:4
[18:05:36] <andrew_b> http://week-to-week.livejournal.com/123393.html
[18:09:06] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2608 (Widget enhancement) updated
http://www.midnight-commander.org/ticket/2608#comment:1
• Ticket #2737 (modarin256: a new 256-color skin for mc) updated
http://www.midnight-commander.org/ticket/2737#comment:1
[18:49:22] andrew_b вышел из конференции
[20:04:51] iliamaslakov зашёл в конференцию
[22:19:02] slavazanko/h вышел из конференции
[22:21:06] slavazanko/h зашёл в конференцию
[22:25:22] <iliamaslakov> http://funkyimg.com/u2/1400/028/772928dn_l.png
[22:25:58] <slavazanko/h> что это?
[22:28:11] <iliamaslakov> скин
[22:28:23] <iliamaslakov> который прислали
[22:29:00] <slavazanko/h> неплохо
[22:31:56] <iliamaslakov> да
[22:32:09] <iliamaslakov> очень на dn похоже
[22:34:38] <slavazanko/h> о, наш недавный боян всплыл http://www.linux.org.ru/forum/talks/7467866
[23:01:07] <iliamaslakov> ага
[23:07:26] iliamaslakov вышел из конференции: Machine going to sleep
[23:26:33] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2737 (modarin256: a new 256-color skin for mc) updated
http://www.midnight-commander.org/ticket/2737#comment:2
Powered by freQ Powered by Python Valid XHTML 1.0 Transitional Valid CSS!