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

[07:21:33] andrew_b зашёл в конференцию
[07:21:33] il.smind зашёл в конференцию
[07:21:33] Зося Синицкая зашёл в конференцию
[07:21:33] slavazanko/w зашёл в конференцию
[07:21:33] iNode зашёл в конференцию
[07:21:33] mc_log зашёл в конференцию
Тема: http://www.midnight-commander.org | Latest stable release: 4.7.5.1
[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);

Проверка не нужна.
[12:29:38] <slavazanko/h> ок, поправил
[12:30:06] <andrew_b> +void
+mc_event_group_del (const gchar * event_group_name)
+{
+
+ if (mc_event_grouplist == NULL || event_group_name == NULL)
+ return;
+
+ g_tree_remove (mc_event_grouplist, (gconstpointer) event_group_name);
+}

А можно тут без return?

+ 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_current_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;


+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 зашёл в конференцию
Тема: http://www.midnight-commander.org | Latest stable release: 4.7.5.1
[16:57:01] il.smind зашёл в конференцию
[17:55:26] qub вышел из конференции
[18:11:43] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2501 ('lib' directory sources should not depend on the 'src' directory) updated
http://www.midnight-commander.org/ticket/2501#comment:1
• Ticket #2501 ('lib' directory sources should not depend on the 'src' directory) updated
http://www.midnight-commander.org/ticket/2501#comment:2
[18:46:24] andrew_b зашёл в конференцию
[19:01:46] slavazanko/h вышел из конференции
[19:19:15] slavazanko/h зашёл в конференцию
[19:21:54] slavazanko/h вышел из конференции
[19:26:32] andrew_b вышел из конференции
[19:57:41] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2507 (makefile.syntax colors not completely honored) created
http://www.midnight-commander.org/ticket/2507
• ScreenShot148.png attached to Ticket #2507
http://www.midnight-commander.org/attachment/ticket/2507/ScreenShot148.png
[20:43:13] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2507 (makefile.syntax colors not completely honored) updated
http://www.midnight-commander.org/ticket/2507#comment:1
[22:13:29] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2501 (sources in 'lib' directory should be independ to sources in 'src' …) updated
http://www.midnight-commander.org/ticket/2501#comment:1
• Ticket #2501 (sources in 'lib' directory should be independ to sources in 'src' …) updated
http://www.midnight-commander.org/ticket/2501#comment:2
• Ticket #2501 (sources in 'lib' directory should be independ to sources in 'src' …) updated
http://www.midnight-commander.org/ticket/2501#comment:3
Powered by freQ Powered by Python Valid XHTML 1.0 Transitional Valid CSS!