<dshil>
mikekaganski: прочитал в доке про -1, -2, 0, +1, +2 в ревью. Но в итоге не нашел в геррите, где пометить патч, как WIP
<mikekaganski>
в ответе у тебя должны быть +1 и -1
<mikekaganski>
или добавь WIP к commit message
<dshil>
понял, спасибо
<mikekaganski>
dshil: \о/
dshil has quit [Ping timeout: 246 seconds]
dshil has joined #libreoffice-ru
<mikekaganski>
dshil: последний патч немного спорный. Лично я за то, чтобы обозначать конец разделов (пространств, #ifdef...#endif). Но есть те, кто считает это, наоборот, каргокультом.
<mikekaganski>
Так что я бы не стал это делать отдельно, а засунул в какой-то больший патч "прицепом".
<dshil>
mikekaganski: угу, думал засунуть прицепом в патч, который ты как раз отревьюил, но решил делать отдельно, форматирование отдельно, переделка функции отдельно, чтобы не замыливались глаза.
<dshil>
поставлю тогда WIP, подумаю к чему прицепить.
<mikekaganski>
dshil: а по мне, эти три строчки такие ненапряжные, что очень хорошо в тот предыдущий патч впишутся
<mikekaganski>
удалить пустую строку, или добавить коммент к закрытию пространства - это не мешает. Если бы ты менял функциональные строки (даже просто бил на 2 строки) - тогда да.
dshil has quit [Ping timeout: 272 seconds]
<libreofficeru-tg>
<Kompilainenn> mikekaganski: я более внимательно посмотрел на существующий Document converter, он в принципе готов к использованию и так, просто поудалять лишние шаги, добавить поле для маски имени файла и на кнопку Convert назначить более расширенный код
<libreofficeru-tg>
<Kompilainenn> ничего нового городить не нужно
dshil has joined #libreoffice-ru
<dshil>
mikekaganski: если поправил замечание, то просто жму кнопку "Done" ?
<mikekaganski>
не надо ничего
<mikekaganski>
я и сам смотрю :-)
<dshil>
ну на одном я все таки нажал ;)
<mikekaganski>
вот если есть возражение - другое дело
<mikekaganski>
да не проблема, просто лишнее телодвижение ;-)
<mikekaganski>
а чего они возвращают int, а не bool? (не смотрел, может, так надо - ты уже в теме, расскажи)
<mikekaganski>
вообще по замечаниям возражений нет?
<dshil>
int -> т.к. высокоуровневая функция возвращает int (думаю, что legacy и надо bool, надо думать), поэтому низкоуровневые делал по аналогии
<dshil>
по замечаниям - все по делу, только мне "длинные прописи" показались проще для восприятия, чем длиннострочник с условием
<mikekaganski>
просто если у нас их возвращаемые значения не присваиваются в какой-нить инт, а используются в сравнениях - пожалуйста, пользуй bool
<dshil>
обычный bool или у нас есть какой-то свой специальный BOOL? по коду такого еще не видел.
<mikekaganski>
обычный
<dshil>
да, они просто в качестве сравнения используются
<mikekaganski>
насчёт длиннострочника - это да, это дело вкуса. Мне кажется, однострочник яснее показывает "определение", что считатеся, например, DitherColor
<mikekaganski>
раньше у нас несколько своих логических типов было, например, sal_Bool. Последний продолжает использоваться только в C API
<dshil>
сделаю const в другом патче, все равно пока там копаюсь
<mikekaganski>
а когда ты написал в коде, надо это ещё отправить - иначе оно драфт
<mikekaganski>
выходишь в патч, и отвечаешь
<mikekaganski>
иначе нельзя было бы, например, сделать несколько комментариев к разным яастям кода в одном ответе
<dshil>
я тупой, буду знать, сорри
<mikekaganski>
причём тут! просто геррит - это эталон антиюзабилити
<mikekaganski>
и это очень хорошо, что мы используем геррит - всегда можно спокойно вздохнуть: "там ещё хуже!" :-)
<libreofficeru-tg>
<Kompilainenn> ээ
<libreofficeru-tg>
<Kompilainenn> "просто геррит - это эталон антиюзабилити" и " и это очень хорошо, что мы используем геррит" - это два противоположных заявления
<mikekaganski>
почему это?
<JohnDoe_71Rus>
геррит пример что есть ХУЖЕ чем ЛО :)
<libreofficeru-tg>
<Kompilainenn> 1 - геррит говно, 2 - хорошо, что мы юзаем говно
<libreofficeru-tg>
<Kompilainenn> азазаз
<mikekaganski>
это очень помогает восстановить душевный покой, когда юзеры указывают на косяки юзабилити в ЛО
<libreofficeru-tg>
<Kompilainenn> ах, в этом плане
dshil has quit [Quit: Page closed]
<libreofficeru-tg>
<Kompilainenn> хе, Либра 6.3 альфа открывает мою супертяжелую рабочую XLSX таблицу за 15 сек, а Эксель 2010 тот же файл открывает за 45 сек
<JohnDoe_71Rus>
ничего. когда 6.3 доберется до релиза будет открывать за минуту. :)
<tagezi>
mikekaganski: sberg и есть тот самый чувак который ненавидет правки?
UNIm95 has joined #libreoffice-ru
UNIm95 has quit [Client Quit]
<mikekaganski>
да :-)
<mikekaganski>
причём это безумно умный мужик, наверное, самый умный из всех
<tagezi>
да я не спорю, но это не значит, что все остальные такиеже умные, и могут видеть код так же как он :)
<tagezi>
и эти умные мужики, имеют кучу говногода в коде, которого не видят
<tagezi>
вот что мне делать с переменной?
<mikekaganski>
несомненно... но надо вот что учитывать. Его ненависть к "лишним" правкам имеет очень серьёзные основания. Их нужно понять, чтобы говорить предметно
<tagezi>
сейчас она не используется, тоесть нафиг не нужна
<tagezi>
но в этом куске, смысл в ней есть
<mikekaganski>
дело в том, что каждая строка имеет историю в гите. И чем меньше правок в этой строке, тем легче определить, когда и *зачем* она менялась
<mikekaganski>
когда ищется проблема, это бывает важно. Большой рефактор вообще уничтожает историю строки, так что приходится искать, чем она была до рефактора
<tagezi>
но рефакторинг все равно нужно делать, у нас куча не понятных мест
<libreofficeru-tg>
<Kompilainenn> лять, Хиско из всех вариантов выбрал самый говеный