[06:14:36] ASM вышел из конференции [06:14:37] ASM зашёл в конференцию [08:00:20] slavazanko/h зашёл в конференцию [08:15:45] <slavazanko/h> всем привет [08:22:13] <iliamaslakov> привет [08:22:21] <iliamaslakov> поголосовать есть желание? [08:22:28] <slavazanko/h> есть [08:24:40] <iliamaslakov> 2421 [08:34:45] andrew_b зашёл в конференцию [08:42:44] <andrew_b> iliamaslakov: +HighlightOccurences = alt-f Где этот дефолт в коде? [08:44:33] <andrew_b> exec_highlight_occurences Почему перемешаны статические и нестатические функции? В редакторе функции начинаютяс с edit_. [08:44:54] <andrew_b> 672 if (tmp && tmp->len) [08:45:18] <andrew_b> 677 strcpy (highlight_word, text); НИКОГДА не используй strcpy! [08:45:49] <andrew_b> 673 text = tmp->str; 674 g_string_free (tmp, FALSE); [08:45:55] <andrew_b> Плохо. [08:46:10] <andrew_b> 645 ret = !strncmp (token, highlight_word, highlight_word_length); 646 647 if (ret != 0)
Почему не статические? [08:48:18] <andrew_b> Ну и за что голосовать? [08:49:00] <iliamaslakov> ??? [08:49:13] <andrew_b> Код грязен до безобразия. [08:49:26] <andrew_b> 75 int highlight_word_length = 0; [08:49:32] <andrew_b> Почему int? [08:50:30] <iliamaslakov> хорошо, принято, [08:50:49] <iliamaslakov> поправлю кроме strcpy [08:50:57] <iliamaslakov> предложи что то на замену [08:51:05] <andrew_b> strncpy [08:51:06] <iliamaslakov> g_strdup? [08:51:11] <iliamaslakov> а... [08:51:12] <iliamaslakov> ок [08:51:20] <iliamaslakov> тоже поправлю [08:52:31] <iliamaslakov> поясни почему 645 ret = !strncmp (token, highlight_word, highlight_word_length); плох? [08:54:06] <andrew_b> == 0 ? 1 : 0; [08:54:23] <andrew_b> Первый год замужем, что ли? [09:01:48] <iliamaslakov> а... я думал у тебя претензии к логике... [09:02:07] <iliamaslakov> типа математика не идёт [09:04:12] <iliamaslakov> >andrew_b: Почему int? а чем инт не устроил? [09:05:25] <andrew_b> strlen [09:05:38] <andrew_b> И индекс массива. [09:06:04] <slavazanko/h> size_t должен быть [10:00:31] <iliamaslakov> а... [10:29:04] <slavazanko/h> по 2695 остались ещё замечания? [10:30:14] <andrew_b> Тестируем. [10:37:58] iliamaslakov вышел из конференции: Replaced by new connection [10:38:03] iliamaslakov зашёл в конференцию [10:46:18] iliamaslakov вышел из конференции: offline [10:46:23] iliamaslakov зашёл в конференцию [10:51:37] <andrew_b> slavazanko/h: мои коммиты смотрел? [10:51:44] <slavazanko/h> угу [10:56:25] <iliamaslakov> andrew_b, запушил [11:00:35] <iliamaslakov> все что смог поправил [11:51:30] <iliamaslakov> slavazanko/h, Все на выборы! [11:52:02] <slavazanko/h> :) [11:52:57] <andrew_b> + if (ret == 0) + return highlight_word_length; + else + return 0; [11:53:00] <andrew_b> Эх... [11:53:39] <slavazanko/h> return (ret == 0 ) ? highlight_word_length : 0; [11:54:02] <andrew_b> + if (tmp->len != 0) + text = g_string_free (tmp, FALSE); + else + g_string_free (tmp, TRUE);
Ох... [11:54:19] <slavazanko/h> Просто Илья не любит тернарные операторы [11:54:32] <andrew_b> + copylen = MIN ((size_t) (sizeof (highlight_word) - 1), (size_t) (strlen (text))); [11:54:45] <andrew_b> strlen два раза вызываться будет. [11:55:07] <andrew_b> - edit_refresh_cmd (edit);
(сочинение 7-летнего Тараса по теме «Кем я хочу стать когда я буду большим») [13:01:51] <iliamaslakov> т.е. в 2004 это уже был боян.. [13:02:41] <andrew_b> Вряд ли это написал семилетний мальчик. [13:03:03] <andrew_b> В первом классе сочинения не пишут. [13:03:18] <andrew_b> Скорее, лет 10-11. [13:09:03] <iliamaslakov> по моим подсчетам писателю примерно 26-30 лет [13:09:11] <iliamaslakov> может чуть больше [13:10:30] <iliamaslakov> запушил я [13:12:21] <andrew_b> Ну собери всё в кучу, а то эти фиксы смотреть крайне не удобно. [13:18:44] <iliamaslakov> готово [13:19:00] <iliamaslakov> я не сквошил чтобы ты видел что сделано [13:19:18] <iliamaslakov> про дефолт на alt-f [13:19:27] <iliamaslakov> чего сделать то надо ? [13:20:16] <andrew_b> В дефолтах прописать, вестимо. [13:29:29] <iliamaslakov> я видать отупел совсем, а где йто? [13:29:49] <iliamaslakov> keybind-defaults.c ? [13:30:06] <andrew_b> Бинго! [13:54:52] <iliamaslakov> запушил [13:56:21] <andrew_b> В mc-devel@ пришло письмо: More skins? [13:56:50] <iliamaslakov> суть? [13:56:55] <andrew_b> Некто Oliver Lange наделал скинов, хочет ими поделиться, но не знает, куда слать. [13:57:08] <iliamaslakov> добавляется новый или хотят [13:57:13] <andrew_b> I've noticed that there could be a few more 256-color skins shipped with mc, as this feature is now supported. Only two so far..
I've recently created a 256er skin (in 4 variants) and ask myself if you'd be interested in maybe adding a few more skins to future mc releases?
If so, how would i submit skins? I find it a bit complicated to use git or something, maybe an email with a small attached skin.tar.gz or an email with pasted skin contents or some kinda easy-peasy upload URL/form or.. [insert here] [13:57:48] <andrew_b> https://www.midnight-commander.org/ticket/2147 [13:59:08] <andrew_b> В словах Готара есть резон. [13:59:36] <andrew_b> С другой стороны, мы как бы не ответственны за скины в стороннеи репо. [14:03:34] <slavazanko/h> репозиторий скинов у нас давно зрел как идея, откладывался только "на потом" [14:04:12] <slavazanko/h> даже какие-то намёки на html-движок скинов где-то валяется... [14:04:32] <slavazanko/h> надо будет его посмотреть на предмет поддержки 256-ти цветов [14:06:57] <andrew_b> Ответьте человеку-то что-нибудь. [14:11:00] <slavazanko/h> я отвечу [14:27:02] <Зося Синицкая> RSS: Новости для Midnight Commander • Ticket #2147 (create a skin repository) updated http://www.midnight-commander.org/ticket/2147#comment:3 [14:31:02] andrew_b вышел из конференции [15:55:07] IceMan зашёл в конференцию [16:41:33] andrew_b зашёл в конференцию [17:56:57] <andrew_b> - Ты чего такая взвинченная? Что случилось? - Подарила мужу на 23 февраля набор рыболовных блесен. - Ну, все правильно. Он у тебя уже 10 лет каждые выходные на рыбалку уезжает. Чего не так? - Он его в руках вертел, вертел и спрашивает: А что это такое? [17:58:57] <iliamaslakov> )) [17:59:29] <iliamaslakov> andrew_b, голосуем, не расходимся [18:05:28] <andrew_b> Может, хватит фичами накачивать? Может, баги поисправлять? [18:10:13] <slavazanko/h> Илья ещё не все фичи из фара натибрил, рано ещё :) [18:10:29] <slavazanko/h> анекдот классный :) [18:10:43] IceMan вышел из конференции [18:27:15] <andrew_b> iliamaslakov: переименуй exec_ в edit_ [18:27:34] <andrew_b> + if (tmp->len != 0) + text = g_string_free (tmp, FALSE);
Тут мемлик. [18:28:39] <andrew_b> + memset (highlight_word, 0, BUF_MEDIUM);
Эх... [18:41:45] <andrew_b> Это индусский код. [18:42:15] <andrew_b> slavazanko/h: 2736 [18:42:37] <andrew_b> Кстати. [18:43:30] <slavazanko/h> гм ... интересный багн [18:43:44] <Зося Синицкая> RSS: Новости для Midnight Commander • Ticket #2695 (VFS reorganization: the vfs_path_t expansion to 'src/' subdirectory) updated http://www.midnight-commander.org/ticket/2695#comment:8 [18:43:59] <andrew_b> slavazanko/h: ничего интересного. [18:44:53] <andrew_b> WPanel::selected же не меняется. [18:45:09] <slavazanko/h> навскидку: где-то файл берётся по индексу списка (список файлов в панели). [18:45:17] <andrew_b> Файла уже нет, и WPanel::selected кажет на другой файл. [18:46:26] <andrew_b> Я тут недавно ходил на ftp, и перезагрузка каталога перед копированием сильно доставала. [18:46:48] <andrew_b> Жмёшь F5, и он пошёл перечитывать каталог. [18:46:51] <slavazanko/h> это после 2695? [18:47:03] <andrew_b> А файлов много, и скорость небыстрая. [18:47:06] <andrew_b> Не. [18:47:54] <andrew_b> После f856eeb0eac182c2d7e9896b40c5a2fea46d5574 [18:48:27] <andrew_b> Где появилось
+ /* Update panel contents to avoid actions on deleted files */ + if (!panel->is_panelized) { + update_panels (UP_RELOAD, UP_KEEPSEL); + repaint_screen (); + } [18:48:29] <andrew_b> Теперь этого быть не должно. [18:48:55] <andrew_b> Каталог перечитывается, если текущего файла уже нет. [19:05:44] <slavazanko/h> гм. Одно пофиксили, другое вылезло [19:07:56] <andrew_b> ? [19:14:55] <iliamaslakov> andrew_b, голосовать будем? [19:15:15] <iliamaslakov> или дальше ваньку валять будем? [19:15:59] <andrew_b> Код исправляй. [19:16:07] <iliamaslakov> ещё? [19:16:15] <slavazanko/h> :) [19:16:16] <andrew_b> Да. [19:16:22] <iliamaslakov> а ты не говорил что еще надо [19:22:09] <iliamaslakov> я всё поправил еще днём [19:26:23] andrew_b вышел из конференции [19:32:48] <iliamaslakov> slavazanko/h, ну хоть ты [19:36:31] <slavazanko/h> ачоя? С заявленными претензиями согласен. Если трудно, могу помочь исправить [19:44:52] <iliamaslakov> с какими? [19:45:08] <iliamaslakov> суть обозначь [19:46:43] <iliamaslakov> все претензии были устранены, проблемный код переписан, голосуй! [19:51:27] <iliamaslakov> slavazanko/h, и в конце концов кто если не ты [20:17:06] ASM вышел из конференции [20:17:08] ASM зашёл в конференцию [22:18:43] slavazanko/h вышел из конференции [22:21:10] slavazanko/h зашёл в конференцию [22:31:33] slavazanko/h вышел из конференции
