mc-dev
mc-dev@conference.jabber.ru
[02:58:20] D-ion зашёл в конференцию
[03:29:57] slavazanko/h вышел из конференции
[03:31:14] slavazanko/h зашёл в конференцию
[04:20:50] slavazanko/h вышел из конференции
[04:21:50] andrew_b зашёл в конференцию
[04:23:32] slavazanko/h зашёл в конференцию
[05:22:04] <smind> андрей!
[05:22:41] <smind> Я так понял, половина патчей в 3051 исправляют фатальные недостатки кода?
[05:25:38] <andrew_b> Что не так?
[05:26:34] <smind> 1475 lc_keys = g_strsplit_set (name, "-+ ", -1);
1476 g_free ((char *) name);
1478 for (p = lc_keys; p != NULL && *p != NULL; p++)
[05:26:52] <smind> не понял чем оно лучше того что было
[05:27:33] <smind> т.е. было
1478 while ((p != NULL) && (*p != NULL))
стало
1478 for (p = lc_keys; p != NULL && *p != NULL; p++)
[05:28:04] <smind> я не против, просто не понял мотива
[05:29:01] <andrew_b> Всё вот это
p = lc_keys;
while ((p != NULL) && (*p != NULL))
{
...
p++;
}
умещается в одной строке
for (p = lc_keys; p != NULL && *p != NULL; p++)
[05:29:19] <smind> это то понятно
[05:29:38] <smind> не понятно зачем
[05:31:30] <smind> https://www.midnight-commander.org/changeset/bbdfbef54686a40c892af454d854501ee7e69825/
клинап говоришь :)
[05:31:39] <smind> ну-ну :)
[05:36:04] <smind> давай я тогда поставлю себе эту версию в качестве основной, иначе не представляю как потенциальные (после оптимизации) косяки выявить
[05:36:23] <andrew_b> Да. Тело функции перенесли внутрь цикла и разделили условия.
[05:36:29] <andrew_b> Да ставь.
[05:43:41] <smind> andrew_b, а у тебя нет под рукой g_string_append_len и g_string_append в сырцах?
[05:44:06] <andrew_b> Есть.
[05:45:10] <smind> ты думаешь замена g_string_append на g_string_append_len даст что то?
[05:45:28] <smind> я у себя нашел сырцы глиба, ща гляну
[05:45:39] <andrew_b> Не надо длину считать.
[05:46:00] <smind> у меня подозрение что длина не считается а берётся из ->len
[05:46:28] <andrew_b> Нет.
[05:47:07] <andrew_b> Какая ->len у const gchar *val?
[05:47:19] <smind> да, согласен с тобой, мой косяк
[05:47:28] <smind> недоглядел
[05:48:00] <smind> там же const gchar а не GString *string
[05:48:22] <smind> снято
[07:44:29] D-ion вышел из конференции: Replaced by new connection
[07:44:33] D-ion зашёл в конференцию
[08:03:41] slavazanko/h вышел из конференции
[08:04:00] slavazanko/h зашёл в конференцию
[08:58:10] ASM вышел из конференции
[08:58:12] ASM зашёл в конференцию
[11:30:55] imho2 зашёл в конференцию
[11:35:22] imho2 вышел из конференции
[13:20:16] <smind> вот времена настали
[13:24:51] andrew_b вышел из конференции
[17:05:08] andrew_b зашёл в конференцию
[18:18:29] andrew_b вышел из конференции
[20:09:02] smind вышел из конференции: Replaced by new connection
[20:09:18] smind зашёл в конференцию
[20:09:34] smind вышел из конференции
[20:10:01] smind зашёл в конференцию
[20:29:03] ASM вышел из конференции
[21:35:49] slavazanko/h вышел из конференции
[21:36:35] slavazanko/h зашёл в конференцию
[21:45:18] slavazanko/h вышел из конференции
[22:18:42] slavazanko/h зашёл в конференцию
