- if (edit_insert_file (edit, edit->filename) == 0) - { - edit_clean (edit); - return 1; - } + (void) edit_insert_file (edit, edit->filename); [14:00:48] <andrew_b> Возвращай -1 и проверяй. [14:00:52] <il.smind> тут как раз не надо [14:01:09] <il.smind> ну ок [14:01:19] <il.smind> но просто 2 варианта [14:01:31] <il.smind> допустим читаем из пустого файла [14:01:39] <il.smind> редактор надо открыть [14:01:40] <andrew_b> Как не надо? edit_load_file() [14:01:57] <andrew_b> Если не смогли файл загрузить? [14:02:01] <il.smind> читаем из потока начитали с ошибкой [14:02:54] <il.smind> тоже по идее открываем с пустотой [14:03:29] <il.smind> эдит_лоад там сам генерит ошибку в другом месте [14:03:43] <andrew_b> slavazanko/w: запушь. [14:04:13] <il.smind> edit_insert_file просто вставляет начитанное [14:04:37] <il.smind> от того что он ничего не начитал ничего не зависит [14:05:22] <andrew_b> Блин. edit_load_file возвращает 0 в двух случаях: когда не смогли открыть файл, или когда смогли, но оно пустой. [14:05:41] <andrew_b> Это разные ситуации и обрабатывать их нужно по-разному. [14:06:08] <andrew_b> Если файл пустой -- это не ошибка. [14:06:22] <andrew_b> А когда не смогли отурыть файло -- ошибка. [14:06:23] <il.smind> > или когда смогли, но оно пустой. я говорю о том что это вообще не ошибка [14:06:39] <andrew_b> Это не ошибка. [14:06:57] <slavazanko/w> щапушил [14:07:01] <slavazanko/w> запушил [14:07:14] <andrew_b> il.smind: Но когда возвратился 0, нельзя понять, что случилось. [14:07:24] <il.smind> ок [14:07:26] <il.smind> ща [14:09:02] <andrew_b> slavazanko/w: я комент к последнему коммиту попаравлю. [14:09:10] <slavazanko/w> ок [14:11:21] iNode зашёл в конференцию [14:12:04] slavazanko зашёл в конференцию [14:12:08] slavazanko вышел из конференции [14:12:44] <iNode> Всем привет и с наступившим новым годом. [14:12:58] <slavazanko/w> Привет, Стас. И тебя также. [14:13:02] <andrew_b> Привет. [14:14:48] <andrew_b> slavazanko/w: как насчёт заголовка панели? [14:16:34] <slavazanko/w> думаю. как это сделать более-менее универсально. оно происходит в src/filemanager/panel.c:1083 panel_correct_path_to_show (WPanel * panel) там надо будет вставить распознавалку, что вошли в extfs... ну и потом добавлять последний элемент пути, то есть, имя архива [14:16:36] <il.smind> какими судьбами? [14:17:16] <andrew_b> Для fish, например, уже добавляется. [14:18:04] <andrew_b> Для ftp тоже. [14:18:29] <andrew_b> Можно, чтобы заголовок формировала сама vfs. [14:18:44] <andrew_b> extfs [14:18:52] <andrew_b> tar, cpio [14:19:07] <andrew_b> Нужен метод отдельный. [14:22:19] <andrew_b> slavazanko/w: можешь зайти по ftp или fish куда-нибудь? [14:22:47] <slavazanko/w> это не заголовок, на самом деле, а название vfs [14:23:22] <slavazanko/w> формировать силами самого модуля vfs? гм... что-то в этом есть. Ща попробую покопать в этом направлении [14:23:34] <andrew_b> Зайди в какую-нибудь vfs, хоть в любой архив. [14:23:49] <slavazanko/w> /utar:// [14:23:56] <andrew_b> Без разницы. [14:24:02] <slavazanko/w> зашёл [14:24:16] <andrew_b> Тепеь в комстроке набери cd. [14:24:34] <andrew_b> Ничего не происходит. [14:24:44] <slavazanko/w> гм. [14:24:44] <andrew_b> В хоум не переходим. [14:25:22] <andrew_b> Если ты даже выйдешь из vfs по .., то cd так и не начинает работать. [14:25:43] <slavazanko/w> ща [15:19:19] il.smind вышел из конференции [15:22:28] <andrew_b> slavazanko/w: @@ -2425,6 +2425,9 @@ do_enter_on_file_entry (file_entry * fe) } . /* Check if the file is executable */ + if (!is_exe (fe->st.st_mode)) + return 0; + ok = (is_exe (fe->st.st_mode) && if_link_is_exe (full_name_vpath, fe)); vfs_path_free (full_name_vpath); if (!ok)
is_exe() два раза вызывается. Когда попадаем на reеurn 0, получем лик: не хватает vfs_path_free (full_name_vpath); [15:22:55] <andrew_b> Шо ж я мимо клавиш попадаю... [15:23:53] <slavazanko/w> угу. Нашл коммит. в котором переход в хомяк сломан. 4309b5ef14cd3d3a5c791e77436039ec5a9d69ab [15:23:56] <slavazanko/w> ща поправлю. Заодно и лик [15:24:14] il.smind зашёл в конференцию [15:25:00] <andrew_b> -Changed do_cd() and do_panel() functions +Changed do_cd() and do_panel_cd() functions [15:35:07] il.smind вышел из конференции [16:25:27] il.smind зашёл в конференцию [16:25:58] <il.smind> . [16:27:08] <andrew_b> il.smind: what? [16:29:24] <il.smind> так, тест [16:30:44] <andrew_b> Пройден. [16:54:19] <il.smind> запушил [17:32:52] <il.smind> не смотри & голосуй [17:35:39] <andrew_b> - edit_insert_file (edit, tmp); + (void) edit_insert_file (edit, tmp); Зачем? [17:38:47] <il.smind> ну мы же везде меняли тогда [17:39:05] <il.smind> чтобы конпилятор ублажить [17:41:02] <andrew_b> Не везде. [17:41:56] <il.smind> поясни [17:41:59] <andrew_b> А там, где у функций есть атрибут, забыл как называется. Что-то типа "не игнорировать возвращаемое значение". [17:42:13] <il.smind> почему зднсь не надо? [17:42:14] <andrew_b> Это к glibc в основном относится. [17:42:38] <il.smind> т.е. убрать? [17:42:55] <andrew_b> Да. [17:43:26] <il.smind> ок [17:46:28] <il.smind> очень запушил [18:08:07] <il.smind> есть чо? [18:09:14] andrew_b вышел из конференции [18:38:57] il.smind вышел из конференции [21:03:21] slavazanko/h зашёл в конференцию