[07:21:39] <andrew_b> О. [07:21:59] <andrew_b> НАдо было просто .join, а не .rejoin [07:25:50] <il.smind> ) [07:26:11] <il.smind> [09:26:03] <Ilia Maslakov> .rejoin [09:26:03] <mc_log> фиг [07:26:29] <il.smind> [09:26:22] <Ilia Maslakov> .rejoin mc-dev@conference.jabber.ru mc_log [09:26:22] <mc_log> фиг [07:27:05] <il.smind> andrew_b: прокачай меня до влядельца [07:27:16] <andrew_b> Я не знаю как. [07:27:43] <il.smind> контекстное меню на мне [07:27:56] <il.smind> Изменить ранг [07:28:15] <il.smind> выбрать "Владелец" [07:29:20] <andrew_b> У меня нету владельца. Есть модератор и администратор. [07:42:17] <il.smind> странно [07:46:17] il.smind вышел из конференции [08:24:05] il.smind зашёл в конференцию [08:24:21] il.smind зашёл в конференцию [08:25:16] <il.smind > il.smind: +1 [08:25:18] <Зося Синицкая> il.smind : Вы изменили карму il.smind до 0. Следующий раз можно изменить через: 12:00:00 [08:25:34] <il.smind> il.smind : +1 [08:25:34] <Зося Синицкая> il.smind: Вы изменили карму il.smind до +1. Следующий раз можно изменить через: 01:00:00 [08:25:52] il.smind вышел из конференции [08:25:56] <il.smind> ) [08:26:28] <il.smind> andrew_b: появились приглашения у всех [08:26:35] <il.smind> в контакте [08:30:38] <andrew_b> Да? [08:57:46] iNode вышел из конференции [08:58:32] iNode зашёл в конференцию [09:24:09] slavazanko/h зашёл в конференцию [09:25:26] <slavazanko/h> всем привет [09:26:12] <andrew_b> slavazanko/h: ! [09:31:28] <il.smind> slavazanko/h: тут? [09:31:40] <slavazanko/h> тут [09:31:45] <il.smind> хорошо [09:31:47] <slavazanko/h> я тебя овнером сделал [09:31:55] <il.smind> у меня всё [09:32:11] <il.smind> :) [09:32:28] <slavazanko/h> не понял :) [09:32:39] <il.smind> я хотел попросить [09:32:44] <il.smind> но ты уже сделал [09:33:02] <slavazanko/h> а :) [09:33:15] il.smind зашёл в конференцию [09:33:50] <il.smind > пока неудачники [09:33:52] il.smind вышел из конференции [09:33:55] slavazanko/h вышел из конференции [09:34:35] <andrew_b> Шото я не понял. [09:34:37] <il.smind> slavazanko/w: у меня еще просьба [09:35:00] <il.smind> andrew_b: да это я виртуала своего привел :) [09:35:09] <andrew_b> Дубля? [09:36:15] <il.smind> ну это почта только для всяких подтверждений от организаций [09:36:47] <andrew_b> Не понял. [09:39:23] <andrew_b> Причём тут почта? [09:41:05] <il.smind> с il.smind я пишу везде а il.vmind только для определённых задач [09:41:24] <il.smind> а экаунт он привязан к почте [09:41:40] <andrew_b> А... [09:41:50] <andrew_b> Не заметил v. [09:41:52] <il.smind> вот я второму дал прав чтобы он меня потом если что [09:42:46] <andrew_b> Кстате, я типа закончил с переименованием. [09:43:36] <andrew_b> Можно пообсуждать. [09:50:05] slavazanko/h зашёл в конференцию [09:53:40] <il.smind> ща я тут по телефону дотреплюсь... [10:03:09] <slavazanko/h> andrew_b: что за переименование? DEV_simplify_keybindings? [10:25:34] <andrew_b> Да. [10:25:42] <andrew_b> il.smind: там у тебя это... [10:25:48] <andrew_b> Туннель опять... [10:27:35] <il.smind> а щас? [10:27:46] <andrew_b> Минуту назад не было. [10:28:17] <andrew_b> Полилось. [10:28:26] <il.smind> угу... [10:55:00] <andrew_b> $git log --pretty=oneline 8d26abd53^..HEAD | wc -l 95
Наверное, надо немного подсквошить, да? :) [10:55:31] <andrew_b> В итоге надо будет все переименования слить в один коммит. [10:56:09] <il.smind> 95
понтанулся да? :) [10:56:18] <andrew_b> А то ребейз будет выглядить ужасно. [10:56:46] <andrew_b> il.smind: ну я там по чуть-чуть переименовывал, без всякой системы. [10:57:19] <andrew_b> Коммиты мелкие, поэтому их так много. [11:14:28] BanderOS зашёл в конференцию [11:27:28] BanderOS вышел из конференции [11:28:31] BanderOS зашёл в конференцию [11:31:09] BanderOS вышел из конференции [11:35:33] BanderOS зашёл в конференцию [11:36:47] BanderOS вышел из конференции [11:39:12] BanderOS зашёл в конференцию [11:42:55] BanderOS вышел из конференции [11:45:33] BanderOS зашёл в конференцию [11:48:18] BanderOS вышел из конференции [12:05:01] <il.smind> andrew_b: итак [12:05:23] <il.smind> засквошишь? [12:05:32] <il.smind> в штук 10 [12:05:37] <andrew_b> Однозначно. [12:05:45] <andrew_b> Но не сейчас. [12:05:55] <andrew_b> Сейчас это в ткннель не пролезет. [12:06:05] <slavazanko/h> мой бранч кому-нить интересен? [12:06:13] <il.smind> slavazanko/h: да [12:06:21] <andrew_b> slavazanko/h: у тебя масса бранчей. [12:06:21] <il.smind> очень интересен [12:06:24] <slavazanko/h> а почему интересуетесь? [12:06:27] <andrew_b> Какой из? [12:06:33] <slavazanko/h> библиотека [12:06:37] <il.smind> про разделение [12:06:38] <slavazanko/h> 2501 [12:06:42] <il.smind> я так понимаю [12:06:48] <slavazanko/h> угу [12:06:58] <andrew_b> Да, надо смотреть. [12:08:12] <slavazanko/h> после него можно и VFS колупать спокойно - юнит-тесты свабодна компиляццо и проверяюццо [12:08:56] <andrew_b> Кстате. [12:09:08] <andrew_b> А что ты там за тесты написал? [12:09:25] <andrew_b> А то я не шарю. [12:10:04] <il.smind> да шо за тесты? [12:10:17] <il.smind> они нам mc не заразят вирусами? [12:10:25] <andrew_b> slavazanko/h: да, ещё, у Ильи там что-то не компилилось. [12:10:35] <andrew_b> У пятницу. [12:10:40] <andrew_b> Ночью. [12:10:41] <il.smind> дада [12:10:44] <slavazanko/h> тесты, которые тестируют VFS. базовые функции пока что. По мере углуюления в изучение VFS буду писать тесты посложнее. [12:10:50] <slavazanko/h> что не компилилось? [12:11:43] <andrew_b> http://www.midnight-commander.org/chatlogs/mc-dev@conference.jabber.ru/2011/02/18.html#21:54:25 [12:14:10] <slavazanko/h> ну хз. Сейчас скомпилил - всё гуд. Без сучка и задоринки [12:14:44] <andrew_b> Да, у меня тоже. [12:17:46] <slavazanko/h> а что скажет виновник торжества? [12:22:15] <andrew_b> mc_event_destroy_value [12:22:21] BanderOS зашёл в конференцию [12:22:30] <andrew_b> Мож напрямую использовать g_tree_destroy? [12:25:53] <slavazanko/h> там не совсем GTree
Попробую объяснить. Группы событий являются GTree каждое имя в группе тоже входит в GTree
То есть, события - это дерево деревьев
Но вот уже содержимое событий (каллбэки) - это простые списки GPtrArray. Вот они и чистятся отдельно [12:26:31] <andrew_b> +static void +mc_event_destroy_value (gpointer data) +{ + GTree *mc_event_keylist = (GTree *) data; + g_tree_destroy (mc_event_keylist); +} [12:26:46] BanderOS вышел из конференции [12:26:53] <slavazanko/h> а [12:27:03] <slavazanko/h> я перепутал с mc_event_group_destroy_value [12:27:21] <slavazanko/h> ну можно и просто g_tree_destroy, да [12:27:30] BanderOS зашёл в конференцию [12:27:58] <andrew_b> + if (event_group == NULL) + return; + + g_tree_remove (event_group, (gconstpointer) event_name);
+ if (mc_event_grouplist != NULL && event_group_name != NULL) + g_tree_remove (mc_event_grouplist, (gconstpointer) event_group_name); [12:31:46] <slavazanko/h> ну.. это моя наверное извращённая логика :) Я привык, что сначала отсекаю все невалидные варианты. а потом в остатке работаю с заведомо валидными условиями в функции... [12:32:29] <andrew_b> Ну да. [12:32:39] <andrew_b> Но тут всего две строки. [12:32:48] <slavazanko/h> ок. поменяю :) [12:34:05] <slavazanko/h> готово. Пока не пушу - потом по окончании ревьюва сделаю один большой фикс-коммит [12:34:36] <andrew_b> - exit (1); + exit (EXIT_FAILURE); [12:35:32] <slavazanko/h> готово [12:35:51] <andrew_b> + if (!mc_event_deinit (&error)) + { + fprintf (stderr, _("Failed while close:\n%s\n"), error->message); + g_error_free (error); + exit (1); + } + mc_config_deinit_config_paths ();
Может, тут не выходить сразу, а таки сделать деинициализацию путей? [12:37:38] <andrew_b> a66cc976236ce09386cb0634ea6e047f41d9f517 [12:37:40] <slavazanko/h> готово [12:38:00] <andrew_b> Вот видишь, что получается. Ты определил структуру три раза. [12:39:00] <andrew_b> +vfs_stamp_create (struct vfs_class *vclass, vfsid id) { - struct vfs_class *nvfs, *n2vfs, *n3vfs; - vfsid nvfsid, n2vfsid, n3vfsid; + struct vfs_class *nvfs; + vfsid nvfsid; + struct + { + struct vfs_class *vclass; + vfsid id; + gboolean ret; + } event_data = { vclass, id, FALSE };
+check_other_panel_timestamp (const gchar * event_group_name, const gchar * event_name, + gpointer init_data, gpointer data) +{ + struct event_data_t + { + struct vfs_class *vclass; + vfsid id; + gboolean ret; + } *event_data = (struct event_data_t *) data; [12:39:09] <slavazanko/h> да. Для этого я хотел отдельный файл lib/event_types.h завести... [12:40:04] <il.smind> slavazanko/h: отвлекают сильно [12:40:16] <slavazanko/h> биатлонишь? [12:40:56] <andrew_b> Надо бы что-то более-менее универсальное. Типа { void *pdata1; void *pdata2; void *pdata2; void *pdata4; gint64_t idata1; gint64_t idata2; gint64_t idata3; gint64_t idata4; } [12:41:05] <andrew_b> Коряво, конечно. [12:41:25] <andrew_b> Но заводить свою структуру на каждое событие тоже не фонтан. [12:41:49] <il.smind> slavazanko/h: а слушай слав [12:42:00] <il.smind> давно еще вот чего хотел сказать [12:42:12] <il.smind> ты эта [12:42:51] <il.smind> иногда делаешь коммиты в которых натрагиваются разные логически не смязанные коммиты [12:43:29] <slavazanko/h> да [12:43:34] <il.smind> и твой коммит их связывает так что дальше без напильника никак [12:43:41] <slavazanko/h> да [12:43:50] <il.smind> и мне потом очень не очень это дело разгребать [12:44:11] <il.smind> вот хотел попросить [12:44:38] <il.smind> если ты меня еще хотябы капельку любишь, не делать этого [12:44:47] <andrew_b> :) [12:44:59] <andrew_b> Какие подробности всплывают... [12:45:05] <slavazanko/h> ты про фикс- коммиты? Они же того... по результатам ревьюва. Эти коммиты потом git reset HEAD^ и потом вручную нужные файлы git add и разбивать по фикс-коммитам [12:45:31] <slavazanko/h> юзай gcc и не буду такие коммиты делать :) [12:51:22] <andrew_b> print_vfs_message и mc_event_add (MCEVENT_GROUP_CORE, "vfs_print_message", print_vfs_message, NULL, NULL); надо бы внести под #if ENABLE_VFS. [12:51:59] <andrew_b> Без VFS вроде не используется. [12:52:28] <andrew_b> Или использууется/ [12:52:30] <andrew_b> ? [12:52:39] <slavazanko/h> нет [12:53:49] <andrew_b> Похоже что таки да. [12:54:01] <andrew_b> vfs_s_open [12:55:33] <slavazanko/h> значит, не трогаю [12:57:25] <andrew_b> Короче, надо убрать и скомпилить с --disable-vfs. [12:59:16] <andrew_b> + if (error == NULL) + { + if (mc_config_deprecated_dir_present ()) + { + mc_config_migrate_from_old_place (&error);
Слепи в одно условие. [13:02:13] <andrew_b> Ещё такой вопрос. Память, аллокированная в mc_global, похоже, не освобождается. sysconfig_dir там, share_data_dir всякие. [13:02:15] <slavazanko/h> готово [13:02:30] <slavazanko/h> освобождается [13:02:38] <andrew_b> Где? [13:03:02] <slavazanko/h> lib/mcconfig/paths.c:263 [13:03:18] <andrew_b> А, да. Виноват. [13:10:47] <andrew_b> Пока у меня одна претензия -- это независимые определения структур на передеющей и приёмной стороне ивента. [13:11:54] <slavazanko/h> предлагаю всё-таки на каждый эвент по структуре. И в отдельный файл. [13:13:07] <slavazanko/h> в отдельный хидер определения структур, имею ввиду [13:13:19] <andrew_b> Это очень не красиво. [13:13:42] <slavazanko/h> зато практично. и для Swig будет удобно. [13:14:14] <slavazanko/h> Swig - генерилка врапперов для различных других языков - перл, питон, пхп, ... [13:14:51] <andrew_b> Как гонять ивенты, например, между ядром и плагинами, если неизвестны типы? [13:15:10] <andrew_b> Хотя... [13:15:27] <andrew_b> Что-то я не то говорю. [13:15:40] <slavazanko/h> быват [13:15:48] <il.smind> slavazanko/h> ты про фикс- коммиты? Они же того... по результатам ревьюва. Эти коммиты потом git reset HEAD^ это если разные файлы [13:16:07] <il.smind> а у меня и в одном файле разные по смыслу бывают [13:16:48] <slavazanko/h> Уболтал. В следующий раз, если оно у меня не собираеццо, я просто буду тебе об этом говорить. А дальше разгребай сам что и где :) [13:38:19] BanderOS вышел из конференции [13:42:27] qub зашёл в конференцию [13:42:36] BanderOS зашёл в конференцию [13:45:36] BanderOS вышел из конференции [16:52:05] Зося Синицкая зашёл в конференцию [16:52:05] slavazanko/h зашёл в конференцию [16:52:05] qub зашёл в конференцию [16:52:05] slavazanko/w зашёл в конференцию [16:52:05] iNode зашёл в конференцию [16:52:05] mc_log зашёл в конференцию