[05:10:44] dkostousov зашёл в конференцию [05:11:02] dkostousov вышел из конференции [06:18:10] andrew_b зашёл в конференцию [07:10:53] il.smind зашёл в конференцию [07:27:52] <andrew_b> il.smind : почему бардак в расположении? [07:28:16] <il.smind > РАССТРЕЛЯТЬ! [07:28:30] <andrew_b> 2120 ты делал? [07:29:41] <il.smind > я [07:30:08] <andrew_b> Почему Owned by: [07:30:14] <il.smind > рифмовать не надо [07:31:44] <andrew_b> А? [07:33:55] <il.smind > готово [07:34:06] <il.smind > по панелям [07:34:16] <il.smind > динамическим [07:34:40] <il.smind > будь добр, согласись на тикет [07:34:48] <andrew_b> Нет. [07:34:53] <il.smind > дополнительный [07:35:21] <il.smind > твой отказ противоречит нашему неофициальному уставу [07:35:52] <andrew_b> Неофициальный устав не догма. [07:36:59] <il.smind > «Пока вы ещё здесь…»
При обзоре кода всегда есть соблазн попросить автора исправить заодно и какие-нибудь другие проблемы. Такой подход может быстро перерасти в полномасштабную операцию по наведению глянца на большом участке кода, хотя изначально автор и проверяющий должны были утвердить лишь небольшое улучшение.
Лучший совет в такой ситуации: дорожить поэтапным улучшением. См. «Лучшее враг хорошего». [07:37:23] <il.smind > Лучшее враг хорошего
Нельзя сделать идеальный патч, который бы разом решил все ваши проблемы. Не гонитесь за идеалом, работайте над поэтапным улучшением. [07:42:18] <Зося Синицкая> RSS: Новости для Midnight Commander • Ticket #2120 (cd to ~ processed incorrectly) reopened http://www.midnight-commander.org/ticket/2120#comment:13 [07:48:49] <il.smind > andrew_b: по 68 [07:49:10] <il.smind > вчера не получилось поговорить [07:50:16] <il.smind > найдёшь время? [07:50:44] <andrew_b> Что ты от меня хочешь? [07:53:46] <il.smind > там кой чего я не понимаю [07:53:59] <il.smind > ща позавтракаю расскажу [08:30:35] <il.smind > andrew_b: я вернулся [08:30:42] <il.smind > сможешь? [08:30:53] <andrew_b> Смогу посмотреть. [08:31:12] <il.smind > собери тогда пока [08:31:21] <il.smind > сейчас точки контрольные скажу [08:53:12] <il.smind > andrew_b: готов? [08:53:25] <andrew_b> Уже нет. [09:05:04] andrew_b вышел из конференции [09:42:07] <slavazanko/w> Ща опять Андрюха в конце дня нарисуется и скажет, что неожиданно наслаждался тишиной в комнате... [09:44:24] andrew_b зашёл в конференцию [09:44:37] <andrew_b> slavazanko/w: Почти. [09:44:43] <slavazanko/w> о :) с возвращением. [09:44:57] <andrew_b> Почему-то опять замерло. [09:45:13] <andrew_b> Кто-то шалит. jabber.ru? [09:45:36] <slavazanko/w> хз. у меня порядок. Правда. я с гмайлового аккаунта сижу [09:45:43] <andrew_b> il.smind : сделай что-нибудь, и я покажу тебу няшку. :) [09:46:43] <il.smind > andrew_b: перезапустил [09:46:46] <il.smind > проверяй [09:47:07] <il.smind > я тоже с гмыла [09:47:19] <andrew_b> Няшка в 2261_multiedit. [09:47:22] <il.smind > НЕБЫЛО НИ ЕДИНОГО РАЗРЫВА [09:47:28] <il.smind > andrew_b: я понял [09:47:32] <il.smind > ща качну [09:47:44] <andrew_b> Пока только одно окно. [09:48:10] <slavazanko/w> il.smind : по 68 у тебя какие трудности? я могу помочь или только Андрюха? [09:48:11] <il.smind > как же я затрахался с рыбой [09:48:26] <il.smind > трудности вот в чём [09:48:37] <andrew_b> il.smind : вот со Славиком поговори, а я на обед. [09:48:39] <il.smind > как только нажимается секд-п [09:48:57] <il.smind > как только нажимается ctrl-g [09:49:10] <il.smind > прекращается подача топлива [09:49:27] <il.smind > а на той стороне нихера не знают о такой подставе [09:50:04] <il.smind > т.е. то что мы говорили про то "сколько получилось считать столько и отправили" [09:50:17] <il.smind > но прерывание идет в другом месте [09:50:31] <il.smind > уже после того как начали слать [09:51:10] <il.smind > т.е. решение в том чтобы сначала дослать порцию а потом уже выходить из [09:51:23] <il.smind > и я не могу понять как этого достичь [09:52:46] <slavazanko/w> укажи на участок кода. где всё это происходит. fish.c:??? [09:52:55] <slavazanko/w> какая строка (строки)? [09:53:21] <il.smind > ща [09:53:25] <slavazanko/w> и ты 68 перебазировал на последний мастер? [09:58:10] <il.smind > да [09:58:22] <il.smind > строки... [09:58:52] <il.smind > fish_get_reply [09:58:59] <il.smind > res = vfs_s_get_line (me, sock, answer, sizeof (answer), '\n'); [09:59:02] <Зося Синицкая> il.smind : '= vfs_s_get_line (me, sock, answer, sizeof (answer), '\n');' не найдено! [09:59:20] <il.smind > после c-g res = 0 [09:59:39] <il.smind > т.е. чтение прекратилось [10:00:37] <slavazanko/w> там проверка есть 202 if (!res) [10:15:39] <slavazanko/w> Илья, ты тут? [10:24:08] <andrew_b> . [10:24:15] <andrew_b> О. Не отвалилось. [10:24:24] <slavazanko/w> :) [10:32:29] slavazanko/h вышел из конференции [10:34:04] slavazanko/h зашёл в конференцию [10:37:30] <andrew_b> slavazanko/w: ну хоть ты посмотри 2261_multiedit. [10:37:44] <slavazanko/w> ок :) Ща. [10:38:03] <slavazanko/w> я всё Илью жду с его 68... но он молчит, значит поезд ушёл, пусть ждёт следующего [10:38:29] <andrew_b> У него статус "Отошёл давно". [10:42:04] <andrew_b> За верхнюю рамку таскаем окно. За правый нижний угол -- изменяем размер. [10:48:50] <slavazanko/w> ахренеть. [10:49:44] <slavazanko/w> по alt+` видны разные редакторы или разные окна в редакторах? [10:49:52] <andrew_b> Редакторы. [10:50:09] <andrew_b> Многооконности в самом редакторе пока нет. [10:50:13] <il.smind > [12:00:37] <slavazanko/w> там проверка есть 202 if (!res) конечно есть [10:50:17] <il.smind > но это не то [10:50:25] <andrew_b> Пока только одно окно. [10:50:50] <andrew_b> С ним надо ещё поработать. [10:51:08] <slavazanko/w> il.smind : чего это "не то"? Оно там именно проверяет на 0. Если 0.то ничего не прочитано [10:51:17] <slavazanko/w> andrew_b: а что ещё осталось? [10:51:54] <andrew_b> Во первых, надо ограничить движение по вертикали. Можно окно услать вверх, а назад не вернуть. [10:52:45] <andrew_b> Во-вторых, загрузку нового файла в окно. Сейчас редактор падает, если сдвинуть окно. [10:53:02] <slavazanko/w> сдвинуть? [10:53:11] <andrew_b> Да. [10:53:39] <andrew_b> меню Файл->Открыть файл... [10:54:03] <andrew_b> Вводим имя. Файл загружается, но если сдвинуть окно, сегфолт. [10:54:23] <slavazanko/w> да [10:55:09] <andrew_b> Ещё на рамке можно разместить кнопки "Закрыть" и "Развернуть". [11:02:56] <andrew_b> . [11:03:11] <slavazanko/w> pong :) [11:03:17] <andrew_b> угу [11:08:10] andrew_b вышел из конференции [11:15:57] <il.smind > погн [11:17:45] <il.smind > andrew_b: собираю [11:18:03] <il.smind > slavazanko/w> il.smind : чего это "не то"? Оно там именно проверяет на 0. Если 0.то ничего не прочитано ты не понимаешь [11:18:09] <il.smind > не надо проверять [11:18:13] <il.smind > надо не прерывать [11:18:26] <il.smind > если прервали отправко то всё! [11:18:36] <il.smind > дальше хоть запроверяйся [11:18:56] <il.smind > сервер будет ждать данных а их никто не пошлёт [11:21:32] <il.smind > andrew_b: ну хочешь мы этот же тикет про панели используем но по отдельности и переоткроем после [11:28:33] <il.smind > andrew_b: Вы тут? [11:31:45] <il.smind > окно не должно ехать если кликнули не в области для захвата [11:41:40] <il.smind > Люди... [11:53:56] andrew_b зашёл в конференцию [11:54:49] <andrew_b> il.smind : куда кликнули? [11:56:05] <il.smind > в текст [11:56:10] <il.smind > для начала пометки [11:56:14] <andrew_b> Да. [11:56:20] <andrew_b> Не должно. [11:56:25] <andrew_b> А оно? [11:56:26] <il.smind > потом потащили курсор за пределы [11:56:33] <il.smind > а оно едет [11:56:36] <andrew_b> А... [11:56:39] <il.smind > а не должно [11:56:45] <il.smind > непорядок [11:56:52] <andrew_b> Куда за пределы? Вверх? [11:56:58] <il.smind > ща [11:57:04] BanderOS зашёл в конференцию [11:57:14] <il.smind > да только вверх [11:57:21] <il.smind > а нет [11:57:40] <il.smind > еще если проехать по нижнеправому углу [11:57:44] <il.smind > то [11:57:56] <il.smind > начинает меняться размер окна [11:57:56] BanderOS вышел из конференции [11:58:43] <andrew_b> Вижу. [11:58:50] <il.smind > и еще ты куда дел мой коммит про [11:59:05] <il.smind > там где статусы на рамках [11:59:18] BanderOS зашёл в конференцию [11:59:20] <il.smind > похоронил? [11:59:22] <andrew_b> Пока выключил. [12:00:13] <andrew_b> Чтобы не мешались. [12:00:23] <il.smind > а... [12:00:45] <il.smind > чем могу помочь то с мультиэдитом? [12:01:06] <andrew_b> Ничем. [12:01:42] <il.smind > но уже всё круто выглядит [12:13:01] BanderOS вышел из конференции [12:21:38] slavazanko/w вышел из конференции [12:25:19] <il.smind > а с 68 кто то поможет? [12:28:25] slavazanko/w зашёл в конференцию [12:31:06] <andrew_b> il.smind : sorry... [12:42:14] <andrew_b> il.smind : итак... [12:42:34] <il.smind > излагать? [12:42:37] <andrew_b> Да. [12:42:50] <il.smind > в третий раз знач... [12:43:17] <il.smind > fish_get_reply [12:43:24] <andrew_b> Глядишь, и самому понятнее станет. [12:43:45] <il.smind > res = vfs_s_get_line (me, sock, answer, sizeof (answer), '\n'); [12:43:48] <Зося Синицкая> il.smind : '= vfs_s_get_line (me, sock, answer, sizeof (answer), '\n');' не найдено! [12:44:02] <il.smind > vfs_s_get_line [12:44:54] <il.smind > direntry.c [12:44:57] <andrew_b> Ну? [12:45:02] <il.smind > vfs_s_get_line (struct vfs_class *me, int sock, char *buf, int buf_len, char term) [12:45:12] <il.smind > 1400 if (read (sock, buf, sizeof (char)) <= 0) 1401 return 0; [12:45:27] <il.smind > как только нажимаем C-g [12:45:43] <il.smind > происходит return 0; [12:45:49] <andrew_b> Да. [12:46:06] <andrew_b> errno устанавливается в EINTR. [12:46:15] <andrew_b> Или как там её.. [12:46:18] <andrew_b> Кстати. [12:46:36] <andrew_b> 1400 if (read (sock, buf, sizeof (char)) <= 0)
Не малова-то ли по чару читать? [12:46:49] <andrew_b> Может, побольше порции сделать? [12:47:45] <andrew_b> il.smind : ну так что дальше return 0? [12:51:34] <andrew_b> Эххх... [12:55:17] <il.smind > этава [12:55:20] <il.smind > вернулся [12:55:36] <il.smind > return 0? значит всё облом [12:55:52] <il.smind > нельза чтобы 0 стрёсся [12:56:23] <il.smind > надо чтобы обязательно дочитало из buf до конца [12:56:30] <andrew_b> Чёйта? [12:56:43] <il.smind > надо как то запретить прерывать на чтении [12:56:44] <andrew_b> read() вполне законно может возвратить 0. [12:56:54] <il.smind > это само собой [12:56:58] <andrew_b> Если достигнут EOF. [12:57:00] <il.smind > я просто говорю что [12:57:22] <il.smind > если пообещали прислать 20 байт [12:57:52] <il.smind > а послали 10 [12:58:08] <il.smind > а после 10-го нажали ctrl-g [12:58:19] <il.smind > и больше ничего не послали [12:58:20] <andrew_b> Ну запрети прерывания в fish_get_reply. [12:58:31] <andrew_b> А потом разреши. [12:58:32] <il.smind > именно там? [12:58:51] <il.smind > а не в vfs_s_get_line ? [12:59:16] <andrew_b> Нет. vfs_s_get_line не только в рыбе используется. [12:59:25] <andrew_b> Можно что-то другое поломать. [13:01:13] <andrew_b> Тут, наверное. 923 if (fish_get_reply (me, SUP.sockr, NULL, 0) != COMPLETE) [13:01:26] <il.smind > не [13:01:32] <il.smind > я всё сделал [13:01:32] <andrew_b> Окружить diable/enable [13:01:41] <il.smind > спасибо тебе друг! [13:01:54] <andrew_b> Не хрустит. [13:02:10] <il.smind > хер там, показалось [13:02:13] <il.smind > еще что то [13:08:38] <andrew_b> Проблема в том, что write() тоже прерывается по SIGINT. Ты можешь не дослать серверу данных. [13:08:44] <andrew_b> Наверное. [13:09:36] <il.smind > скорее так и есть [13:09:45] <il.smind > потому как сейчас вот что [13:09:56] <il.smind > fish_command: send [655360] answer: [### 001] [1] read: [655360], write: 507904 [13:10:17] <il.smind > т.е. недослали [13:11:21] <andrew_b> Перенеси
907 /* reenable SIGINT after fish_command() */ 908 tty_enable_interrupt_key ();
после
910 w_bytes = write (SUP.sockw, buffer, (size_t) r_bytes); [13:12:13] <andrew_b> И после обработку ошибки надо исправить. [13:12:30] <andrew_b> Или не надо? [13:13:28] <andrew_b> С другоя стороны, если канал упал, всё равно надо как-то прерывать. [13:16:09] <il.smind > а можно как то так: прервали, доитали до конца буфера после этого вышли [13:17:45] <il.smind > короче я понял что надо [13:17:55] <il.smind > надо разрешить прерываться везде [13:18:04] <il.smind > кроме записи [13:18:06] <il.smind > но [13:18:26] <andrew_b> Ну вот, не зря три раза объяснял. [13:18:35] <il.smind > если было прерывание то надо дослать как было раньше [13:18:47] <il.smind > из ноля оно дочитывало [13:18:53] <il.smind > помнишь кусок [13:19:11] <il.smind > вот сделать также, но читать не из нуля [13:19:18] <il.smind > а из того же файла [13:19:40] <il.smind > но уже тут запретить прерываться [13:19:50] <il.smind > вот [13:20:18] <il.smind > т.е. если прервали на чтении то попадаем в дочитывание без права прервать [13:20:39] <il.smind > дочитываем и выходим [13:21:29] <il.smind > andrew_b> С другоя стороны, если канал упал, всё равно надо как-то прерывать. оно само [13:21:48] <il.smind > т.к. стрясается ошибка и за ним вылет [13:22:02] <il.smind > в этом плане всё ок [13:22:25] <il.smind > научи дочитать [13:22:44] <andrew_b> ? [13:23:55] <il.smind > ну вот смотри [13:24:41] slavazanko/w вышел из конференции [13:25:29] <il.smind > res = vfs_s_get_line (me, sock, answer, sizeof (answer), '\n'); [13:25:34] <Зося Синицкая> il.smind : '= vfs_s_get_line (me, sock, answer, sizeof (answer), '\n');' не найдено! [13:25:44] <il.smind > рес = 0 [13:25:53] <il.smind > надо еще почитать [13:26:25] <andrew_b> ХЗ [13:28:57] slavazanko/w зашёл в конференцию [13:31:12] <andrew_b> http://www.everyoneweb.com/worldpenissize/ [14:08:54] <slavazanko/w> зачётный линк. Наши сразу принялись считать, на каком белорусы месте... [14:11:04] <andrew_b> :) [15:12:55] <andrew_b> . [15:14:40] <il.smind > ? [15:15:06] <andrew_b> Да думал, у меня опять отвалилось. [15:17:10] <il.smind > я не смог! [15:27:12] andrew_b вышел из конференции: конец рабочего дня [16:40:50] Yury V. Zaytsev зашёл в конференцию [16:41:14] <Yury V. Zaytsev> всем привет [16:41:18] <Yury V. Zaytsev> у меня теоретический вопрос [16:41:35] <Yury V. Zaytsev> какие надо ключи передать, чтобы собираться с дебаг-инфой? [16:42:04] <Yury V. Zaytsev> я помню, что у нас есть --enable-maintainer-mode, но вроде как-то через CFLAGS можно? [16:42:27] <Yury V. Zaytsev> в интернете нет ясности по этому вопросу... кто пишет -g -debug, кто -g3, что бы это значило? [16:44:42] <il.smind > для дебага надо CFLAGS="-g -ggdb3" [16:44:53] <il.smind > для mc_log надо --enable-maintainer-mode [16:45:28] <Yury V. Zaytsev> il.smind : что значит -ggdb3 ? [16:46:01] <il.smind > http://www.midnight-commander.org/wiki/doc/tracingCrash [16:46:31] <il.smind > http://old.nabble.com/difference-between--g,--ggdb-and--ggdb3-td22082494.html [16:46:54] <il.smind > -g produces debugging information in the OS¹s native format (stabs, COFF, XCOFF, or DWARF 2).
-ggdb produces debugging information specifically intended for gdb.
-ggdb3 produces extra debugging information, for example: including macro definitions. -ggdb by itself without specifying the level defaults to -ggdb2 (i.e., gdb for level 2). [16:47:10] <Yury V. Zaytsev> хммм, если у меня эклипс, то у меня dbg? [16:47:19] <il.smind > думаю да [16:47:40] <il.smind > скорее всего там морда к gdb [16:47:46] <Yury V. Zaytsev> т.е. я пишу CFLAGS="-O0 -ggdb3" [16:47:55] <il.smind > вероятность высока но не 100% [16:47:59] <il.smind > Yury V. Zaytsev: да [16:48:23] <il.smind > -O0 вроде там перекрывается [16:48:49] <Yury V. Zaytsev> Осталось только сообразить как его заставить ещё раз configure выполнить [16:48:56] <Yury V. Zaytsev> так сложно всё в этих ваших иде [16:49:05] <il.smind > да [17:02:40] <Yury V. Zaytsev> всё получилось, спасибо [17:02:50] <Yury V. Zaytsev> вернее собирается проект =) [17:03:09] <il.smind > не за что [17:03:22] <il.smind > в эклипсе хочешь потрассировать? [17:03:52] <il.smind > я так понял нам ответят в течении 48 часов? [17:04:00] <Yury V. Zaytsev> да, всё правильно понял [17:04:05] <Yury V. Zaytsev> пока мне ничего не приходило [17:04:09] <il.smind > и 24 уже прошли [17:04:11] <il.smind > да? [17:04:46] <Yury V. Zaytsev> ну почти =) не знаю насколько они пунктуальны [17:04:58] <Yury V. Zaytsev> это дело не суппорта, а маркетинга... [17:06:14] <Yury V. Zaytsev> да, хочу перейти с mc на eclipse для c/c++ проектов, а то после идеи знаешь как-то комплишон нормальный, рефакторинги и т.п. хочется. вот решил хоть собрать пробровать. так всё в этом эклипсе сложно что черт ногу сломит [17:06:55] <Yury V. Zaytsev> там специальный плаггин есть для того, чтобы с автотулс билд-системой эклипс интегрировался [17:07:07] <Yury V. Zaytsev> но его настроить --- это та ещё задача [17:11:38] <il.smind > у меня то писать код хобби, мне мц хватает заглаза [17:12:59] <Зося Синицкая> RSS: Новости для Midnight Commander • Ticket #2515 (Unused variable if without WITH_BACKGROUND) created http://www.midnight-commander.org/ticket/2515 • wtools-without-background.patch attached to Ticket #2515 http://www.midnight-commander.org/attachment/ticket/2515/wtools-without-background.patch [17:47:19] <Yury V. Zaytsev> а у меня каг будто профессия [17:47:31] <Yury V. Zaytsev> проголосуй вот лучше пока андрея нет за однострочник [17:55:13] <Зося Синицкая> RSS: Новости для Midnight Commander • Ticket #2515 (Unused variable if without WITH_BACKGROUND) updated http://www.midnight-commander.org/ticket/2515#comment:1 • Ticket #2515 (Unused variable if without WITH_BACKGROUND) updated http://www.midnight-commander.org/ticket/2515#comment:2 [17:55:36] slavazanko/h вышел из конференции [18:13:11] il.smind зашёл в конференцию [19:40:20] andrew_b зашёл в конференцию [20:05:37] andrew_b вышел из конференции [20:19:09] il.smind вышел из конференции [21:02:45] il.smind зашёл в конференцию [21:11:13] <il.smind > Yury V. Zaytsev: тут? [21:11:48] <Yury V. Zaytsev> пока да [21:12:37] <il.smind > мцо открывается? [21:12:53] <Yury V. Zaytsev> да, но мееедленно [21:13:00] <Yury V. Zaytsev> ты никак голосунуть решил? [21:13:10] <il.smind > нет [21:13:18] <il.smind > я за такое боюсь [21:13:31] <il.smind > есть более грамотные товарищи [21:14:03] <il.smind > еще я не увидел методики проверки [21:15:46] <Yury V. Zaytsev> собрать с without-background и посмотреть что варнинг исчезает [21:16:57] <il.smind > without-background а как? [21:17:01] <il.smind > --without-background [21:17:03] <il.smind > ? [21:20:34] <Yury V. Zaytsev> вроде да [21:21:04] <il.smind > ты то как проверял? [21:23:28] <Yury V. Zaytsev> я проверял методом компиляции в уме [21:23:54] <Yury V. Zaytsev> мне же пришлось патч переделвать, потому, что андрей усправил уже один из этих двух мест в мастере в другом тикете [21:24:17] <Yury V. Zaytsev> поэтому надо что-бы кто-то ещё посмотрел, у меня тперь нет голоса [21:25:13] <il.smind > > я проверял методом компиляции в уме так себе способ... [21:26:10] <Yury V. Zaytsev> ну у кого как, я-то на питоне сефчас пишу, он вообще не компилируется [21:28:37] <il.smind > у меня без этого патча варнингов нет... [21:29:46] <il.smind > --enable-background [21:30:02] <il.smind > надо наверное так --enable-background=no [21:31:56] <Yury V. Zaytsev> --disable-background [21:43:48] <il.smind > вижу [21:44:00] <il.smind > завтра проголосую [21:44:03] il.smind вышел из конференции [23:14:10] Yury V. Zaytsev вышел из конференции
