[05:02:08] andrew_b зашёл в конференцию [05:58:59] iliamaslakov вышел из конференции: Replaced by new connection [05:59:03] iliamaslakov зашёл в конференцию [06:08:14] <iliamaslakov> andrew_b, Здравствуй! [06:10:13] <andrew_b> Привет. [06:11:37] <iliamaslakov> что вы вчера с диалогом нашли? [06:11:45] <iliamaslakov> как этот косяк увидеть [06:12:52] <andrew_b> C-x, c Up/Down [07:24:04] andrew_b вышел из конференции [07:38:43] <iliamaslakov> а, понял [08:18:45] <slavazanko/w> всем привет [08:21:47] <iliamaslakov> slavazanko/w, Привет! [08:23:26] <slavazanko/w> я поспал и во сне подумал насчёт "вернуть не нулл а структуру в которой vpath->path == NULL". Чего-то оно мне не нравится. лучше возврашать NULL. а в коде явно проверять, был ли vpath равен NULL или нет. так оно правильнее [08:48:31] andrew_b зашёл в конференцию [08:52:05] <iliamaslakov> хорошо [08:52:37] <iliamaslakov> тоже думал об этом [08:53:55] <slavazanko/w> ну тады голосуй [09:02:47] <iliamaslakov> а ты запушил? [09:03:50] <slavazanko/w> ну... бранч вроде опубликован [09:03:56] <iliamaslakov> да [09:04:08] <slavazanko/w> ну и головсуй [09:04:14] <slavazanko/w> тьфу. Голосуй [09:04:14] <iliamaslakov> но там нет вот этого >а в коде явно проверять, был ли vpath равен NULL или нет. так оно правильнее [09:04:20] <slavazanko/w> как нету? [09:04:25] <iliamaslakov> а где? [09:04:29] <slavazanko/w> в src/edit/editcmd.c [09:05:15] <iliamaslakov> 129 if (filename_vpath == NULL) немного не оно [09:05:27] <slavazanko/w> чойта? [09:06:10] ASM вышел из конференции [09:06:11] ASM зашёл в конференцию [09:06:51] <iliamaslakov> vpath не равен NULL != filename_vpath == NULL) [09:08:47] <slavazanko/w> не понял. это откуда? [09:54:23] IceMan зашёл в конференцию [10:12:44] <iliamaslakov> slavazanko/w, чего не понял? [10:13:07] <slavazanko/w> > vpath не равен NULL != filename_vpath == NULL) это откуда? [10:13:10] <iliamaslakov> ты написал лучше возврашать NULL. а в коде явно проверять, был ли vpath равен NULL или нет. так оно правильнее [10:13:31] <slavazanko/w> ну да. В бранче естьи то, и другое [10:13:35] <iliamaslakov> а предлагаешь голосовать за 129 if (filename_vpath == NULL) 130 return 0; [10:13:53] <slavazanko/w> ну и? [10:14:21] <iliamaslakov> но filename_vpath == NULL не эквивалентно "лучше возврашать NULL. а в коде явно проверять, был ли vpath равен NULL" [10:14:58] <iliamaslakov> filename_vpath == NULL это проверка до вызова функции [10:15:00] IceMan вышел из конференции [10:15:32] <iliamaslakov> а проверка "явно проверять, был ли vpath равен NULL" это проверка после вызова функции [10:15:42] <iliamaslakov> т.е. это пока еще не реализовано [10:16:17] <slavazanko/w> а надо в том отдельно взятом коде? в editcmd.c:129 [10:16:27] <slavazanko/w> посмотри внимательно [10:16:34] <slavazanko/w> я там нигде не получаю vfs_path_t [10:16:41] <slavazanko/w> то есть vfs_path_element_t [10:16:49] <slavazanko/w> я получаю сразу char * [10:16:59] <slavazanko/w> который получается из vfs_path_get_by_index (filename_vpath, 0)->path [10:17:09] <iliamaslakov> ок, ща запуши и обсудим [10:17:16] <slavazanko/w> если vfs_path_get_by_index() возвратит NULL, то я про это никогда не узнаю [10:17:32] <slavazanko/w> если, конечно, не введу дополнительную переменную [10:18:39] <slavazanko/w> vfs_path_element_t path_element = vfs_path_get_by_index (filename_vpath, 0); const char *start_filename;
if (path_element == NULL) return 0; start_filename = path_element->path; [10:18:51] <slavazanko/w> проще в том куске кода проверить сразу vpath [10:19:28] <iliamaslakov> если, конечно, не введу дополнительную переменную [13:18:40] slavazanko/w: vfs_path_element_t path_element = vfs_path_get_by_index (filename_vpath, 0); const char *start_filename;
именно про это я и говорю [10:19:37] <slavazanko/w> ты этого хочешь? [10:19:47] <iliamaslakov> так велит мне совесть [10:20:43] <andrew_b> Ну давайте на каждый чих по переменной. [10:21:20] <iliamaslakov> я не за переменнуя на каждый чих [10:21:20] <slavazanko/w> iliamaslakov: ну смотри коммит нумер 847a5b92719ff9531075e3d431dd32ed3971357e устроит? [10:21:38] <slavazanko/w> как по мне - это ишлишне [10:21:44] <slavazanko/w> излишне [10:21:52] <iliamaslakov> я про проверить результаты работы функции ибо она крайне не безопасна в данном виде [10:22:14] <slavazanko/w> результаты проверяются до вызова функции [10:22:23] <slavazanko/w> вернее, входные параметры [10:22:26] <slavazanko/w> это тоже нормально [10:22:29] <iliamaslakov> но у нас этого нет [10:22:33] <slavazanko/w> здрасти. [10:22:35] <iliamaslakov> это нормально [10:22:42] <slavazanko/w> if (filename_vpath == NULL) return 0; [10:22:42] <iliamaslakov> но у нас этого нет [10:22:49] <iliamaslakov> это 1 фикс [10:22:50] <slavazanko/w> это появилось [10:22:52] <slavazanko/w> да [10:23:04] <iliamaslakov> а у нас таких мест 130+ [10:23:10] <slavazanko/w> мдым? [10:23:16] <iliamaslakov> а проверка только в 3-х [10:23:31] <slavazanko/w> в трёх, где vpath может быть NULL [10:23:47] <slavazanko/w> в остальных случаях vpath будет заполнен [10:23:49] <iliamaslakov> где он точно может быть NULL [10:24:07] <slavazanko/w> если не так, то проверка появится в 4-х случаях из 130+ [10:24:12] <iliamaslakov> .т.к. мы увидели это по свалившемуся mc [10:24:15] <slavazanko/w> д [10:24:17] <slavazanko/w> да [10:24:30] <iliamaslakov> а в остальных случаях ты предлагаешь на слово верить [10:24:41] <slavazanko/w> да [10:24:49] <iliamaslakov> я тебе конечно верю [10:25:00] <iliamaslakov> ты на стороне добра [10:26:37] <iliamaslakov> просто в данном случае хотелось бы не верить а исключить возможность [10:28:14] <slavazanko/w> если честно, то мне лениво делать это [10:28:29] <iliamaslakov> я сделаю, если вы не против [10:28:49] <iliamaslakov> если нет противопоказаний со стороны быстродействия [10:32:13] <slavazanko/w> делай [12:09:55] <iliamaslakov> шизануться завал на работе, продохнуть не дают, сделаю чуть попозжа [13:10:06] <iliamaslakov> slavazanko/w, make[1]: Entering directory `/home/projects/mc.old/lib' CC libmc_la-lock.lo warning: unknown warning option '-Wmissing-parameter-type' [-Wunknown-warning-option] warning: unknown warning option '-Wunused-result' [-Wunknown-warning-option] lock.c:123:19: error: assigning to 'vfs_path_element_t' from incompatible type 'const vfs_path_element_t *' vpath_element = vfs_path_get_by_index (fname_vpath, 0); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ lock.c:124:23: error: invalid operands to binary expression ('vfs_path_element_t' and 'void *') if (vpath_element == NULL) ~~~~~~~~~~~~~ ^ ~~~~ lock.c:128:27: error: member reference type 'vfs_path_element_t' is not a pointer; maybe you meant to use '.'? elpath = vpath_element->path; ~~~~~~~~~~~~~^~ . 2 warnings and 3 errors generated. [13:10:09] <iliamaslakov> украл у тебя [13:10:20] <iliamaslakov> почему ругает? [14:19:00] andrew_b вышел из конференции [15:07:32] <iliamaslakov> slavazanko/w, СЛАВ! [15:07:41] <slavazanko/w> ? [15:07:59] <iliamaslakov> ну скажи же ты мне, что не так я написал [15:08:22] <slavazanko/w> всё так. git pull скажи [15:08:28] <iliamaslakov> --- a/src/diffviewer/ydiff.c +++ b/src/diffviewer/ydiff.c @@ -161,8 +161,14 @@ rewrite_backup_content (const vfs_path_t * from_file_name_vpath, const char *to_ char *contents; gsize length; const char *from_file_name; + vfs_path_element_t vpath_element;
- from_file_name = vfs_path_get_by_index (from_file_name_vpath, -1)->path; + vpath_element = vfs_path_get_by_index (from_file_name_vpath, -1); + + if (vpath_element == NULL) + return FALSE; + + from_file_name = vpath_element->path; [15:09:06] <iliamaslakov> ругает так [15:09:19] <iliamaslakov> ydiff.c:166:19: error: assigning to 'vfs_path_element_t' from incompatible type 'const vfs_path_element_t *' vpath_element = vfs_path_get_by_index (from_file_name_vpath, -1); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ydiff.c:168:23: error: invalid operands to binary expression ('vfs_path_element_t' and 'void *') if (vpath_element == NULL) ~~~~~~~~~~~~~ ^ ~~~~ ydiff.c:171:35: error: member reference type 'vfs_path_element_t' is not a pointer; maybe you meant to use '.'? from_file_name = vpath_element->path; ~~~~~~~~~~~~~^~ [15:09:29] <slavazanko/w> (17:08:22) slavazanko/w: всё так. git pull скажи [15:09:47] <slavazanko/w> там увидишь :) [15:09:54] <iliamaslakov> ах ты злодюн! [15:10:10] <slavazanko/w> я такой. да [17:32:42] slavazanko/h зашёл в конференцию [17:39:43] <iliamaslakov> slavazanko/h, [17:39:52] <slavazanko/h> ? [17:40:01] <iliamaslakov> твоя помощь надоть [17:40:06] <iliamaslakov> НалУдерж="97544" НалПеречисл="52" [17:40:06] <slavazanko/h> ? [17:40:14] <slavazanko/h> чо? [17:40:18] <iliamaslakov> надо сделать НалУдерж="97544" НалПеречисл="97544" [17:40:31] <iliamaslakov> это написано на одной строке [17:40:46] <iliamaslakov> awk? [17:41:00] <iliamaslakov> sed? [17:41:05] <slavazanko/h> sed [17:41:08] <slavazanko/h> ща [17:41:54] <slavazanko/h> sed 's/НалПеречисл="52"/НалПеречисл="97544"/' [17:42:16] <iliamaslakov> издиваисси? [17:42:52] <slavazanko/h> нет. Вроде то, что тебе надо [17:42:53] <iliamaslakov> НалУдерж="YYYYY" НалПеречисл="ZZZZZ" [17:43:00] <iliamaslakov> было [17:43:09] <iliamaslakov> стать должно [17:43:22] <iliamaslakov> НалУдерж="YYYYY" НалПеречисл="YYYYY" [17:43:29] <slavazanko/h> а. так бы сразу [17:43:50] <iliamaslakov> там как то номер поля надо задать [17:45:35] <slavazanko/h> sed '/НалУдерж="\(.*\)" НалПеречисл=".*"/НалУдерж="\1" НалПеречисл="\1"/' [17:50:05] <iliamaslakov> cat 1.xml |sed '/nalu="\(.*\)" nalp=".*"/nalu="\1" nalp="\1"/' sed: -e expression #1, char 27: extra characters after command [17:52:04] <iliamaslakov> slavazanko/h, чагось делать то? [17:54:55] <slavazanko/h> sed 's/n...
s забыл я [17:55:38] <iliamaslakov> Дай я тебя облабызаю [17:55:45] andrew_b зашёл в конференцию [17:55:52] <iliamaslakov> всё зупа гут [17:57:17] <andrew_b> iliamaslakov: почувствуй разницу + vfs_path_element_t vpath_element; + vfs_path_element_t *vpath_element; [17:57:34] <iliamaslakov> да давно уде [18:07:02] <iliamaslakov> кадаж я домой то пойду... [18:07:08] <iliamaslakov> а эта... [18:07:46] <iliamaslakov> как <><><><><> заменить на <> <> <> <> <> [18:09:34] <andrew_b> s/>/>\n/g [18:10:06] <andrew_b> echo '<><><><><>' | sed -e 's/>/>\n/g' [18:10:55] <iliamaslakov> /g жадный? [18:11:04] <slavazanko/h> да [18:11:45] <andrew_b> g -- это все вхождения в строке, а не только первое. [18:13:45] <iliamaslakov> ...sed 's/>/>\n/g' >NO_NDFL2_1122_1122_1102016594112250001_20120313_1F077D2C-C24E-4340-AAD0-CA4 sed: -e expression #1, char 2: unterminated `s' command [18:14:25] <iliamaslakov> \> ? [18:14:49] <iliamaslakov> нет [18:18:59] <iliamaslakov> победил [18:26:39] andrew_b вышел из конференции [21:47:51] slavazanko/h вышел из конференции
