mc-dev
mc-dev@conference.jabber.ru
[00:02:54] slavazanko/h вышел из конференции
[02:08:35] imho2 вышел из конференции
[04:19:45] andrew_b зашёл в конференцию
[07:11:04] slavazanko/h зашёл в конференцию
[08:15:06] iliamaslakov зашёл в конференцию
[08:45:48] <andrew_b> slavazanko: ping
[08:49:06] <andrew_b> slavazanko/h: ping
[08:53:54] <slavazanko> тут
[08:54:21] <andrew_b> Есть время поговорить?
[08:54:26] <slavazanko> да
[08:54:55] <andrew_b> В клинапе 04e51ba3f8e9954a6bdddb9a714404a7405918a8.
Я откатил некоторые твои изменения. Мне кажется, они не эквивалентны.
[09:05:49] iliamaslakov вышел из конференции
[09:06:00] <slavazanko> блин, там и раньше была лажа. Смысл там такой: надо манипулировать двумя параметрами после вызова fcntl: кодом возврата из функции и переменной errno.
если fcntl вернула -1, то надо проверить, стал ли erroo == EINTR (типа, ресурс продолжает быть заблокированным после таймаута). Если это так, то надо ещё раз вызвать. Если же код возврата -1, но errno не равно EINTR
[09:06:27] <slavazanko> значит, произошла какая-то другая ошибкаи нужно выйти с закрытием хендла
[09:07:03] <andrew_b> Где-то пропущено "не".
[09:07:13] <andrew_b> если fcntl вернула -1
Если же код возврата -1
[09:08:15] <andrew_b> Если же код возврата _не_ -1, то это нормальное значение, и ничего проверять не надо.
[09:08:45] <andrew_b> errno нужно проверять тогда и только тогда, когда функция возвращает ошибку.
[09:08:51] <slavazanko> int ret;
while ((ret = fcntl()) == -1 && errno == EINTR);
if (ret == -1 && errno != EINTR)
{
close ();
return NULL;
}
[09:08:54] <slavazanko> ка-то так
[09:10:17] <andrew_b> Если ты считаешь, что это баг, напиши в bug-gnulib@gnu.org. Этот код я брал оттуда.
[09:10:30] <andrew_b> Мне кажется, что бага нет.
[09:11:21] <andrew_b> А бля, я ещё неправильно закоммитил.
[09:11:28] <andrew_b> Забыл условие в цикле.
[09:12:01] <slavazanko> бывает. А мои изменения были ошибочными, ты правильно сделал, что откатил. Извиняюсь
[09:15:55] <andrew_b> Второй вопрос.
https://www.midnight-commander.org/ticket/3110
Эгмонт там всё расписал, я с ним согласен. wontfix?
[09:17:29] <slavazanko> да
[09:22:17] <andrew_b> Кто голосует вторым за клинап?
[09:37:52] <andrew_b> Может, наконец-то релиз не этой неделе?
[09:42:45] <slavazanko> Да, давайте релиз.
я проголосовал
[09:46:14] <andrew_b> Тогда я вливаю клинап.
[09:48:05] <andrew_b> И закрываю блокируемые тикеты.
[09:48:12] <slavazanko> угу
[09:58:41] <andrew_b> Готово.
Я сейчас соберу себе мастер в качестве рабочего.
Надо бы пару дней его потестить.
[10:04:56] <andrew_b> А бля.
[10:05:04] <andrew_b> Накосячил.
[12:06:28] <slavazanko> исправил косяки?
[12:06:45] <andrew_b> Сразу же.
[12:06:57] <slavazanko> ок. Тоже рабочим собираю
[13:25:58] andrew_b вышел из конференции
[16:01:03] Зоська Синицкая вышел из конференции
[16:18:13] Зоська Синицкая зашёл в конференцию
[16:26:02] andrew_b зашёл в конференцию
[16:28:43] <andrew_b> http://i.voffka.com/archives/bq-j.jpg
[16:49:33] slavazanko/h вышел из конференции
[16:54:05] slavazanko/h зашёл в конференцию
[16:54:23] D-ion вышел из конференции: Replaced by new connection
[16:54:23] D-ion зашёл в конференцию
[17:02:29] imho2 зашёл в конференцию
[17:12:15] imho2 вышел из конференции
[17:56:36] andrew_b вышел из конференции
[18:14:09] slavazanko вышел из конференции
[18:15:40] slavazanko зашёл в конференцию
[18:29:13] iliamaslakov зашёл в конференцию
[19:18:08] iliamaslakov вышел из конференции
[19:34:55] slavazanko/h вышел из конференции
[19:35:37] slavazanko/h зашёл в конференцию
[19:47:57] slavazanko/h вышел из конференции
[19:51:52] slavazanko/h зашёл в конференцию
