ChanServ changed the topic of #libreoffice-ru to: Канал поддержки русскоязычных пользователей LibreOffice | Wiki TDF http://tdf.io/wikiru | On-line документация http://tdf.io/helpru | Логи канала: http://goo.gl/gzwdwU
dshil has joined #libreoffice-ru
JohnDoe_71Rus has joined #libreoffice-ru
<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
JohnDoe_71Rus has quit [Quit: KVIrc 5.0.0 Aria http://www.kvirc.net/]
<tagezi> mikekaganski: когда нужно будет начать волноваться? :)
<mikekaganski> ?
<mikekaganski> ох
<tagezi> или я что-то не доделал?
<mikekaganski> пинать надо, а не волноваться
<tagezi> нене, мне не срочно.. слудующий ещё не готов
<tagezi> ну, малоли у тебя времени нет
<tagezi> и потом, ты ведь не один, а за 4 дня, так и висит тишина
<mikekaganski> dshil: стоп, если они bool, то и возвращать должны true, а не TRUE
<tagezi> а может они всегда возвращают фалс, но признаться в этом он себе не может :)
<mikekaganski> :-)
<mikekaganski> tagezi: сейчас перепроверю на свежий мастер, и отправлю
<tagezi> угу
<mikekaganski> tagezi: если не забуду ;-)
<tagezi> а зачем проверять на свежий мастер? что вы там опять натворили?
<mikekaganski> без понятия - мало ли
<tagezi> ну
<dshil> mikekaganski: угу, поправил
<tagezi> ты старший тебе виднее
dshil has quit [Quit: WeeChat 2.2]
JohnDoe_71Rus has joined #libreoffice-ru
tagezi has quit [Remote host closed the connection]
dshil has joined #libreoffice-ru
<dshil> mikekaganski: писал тебе в геррите, видимо, как-то не так писал. https://gerrit.libreoffice.org/#/c/64668/1/vcl/win/gdi/gdiimpl.cxx (в патче надо было ссылку на переменную цикла сделать const, а то не аккуратненько получилось)
<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 доберется до релиза будет открывать за минуту. :)
JohnDoe_71Rus has quit [Quit: KVIrc 5.0.0 Aria http://www.kvirc.net/]
tagezi has joined #libreoffice-ru
<tagezi> mikekaganski: много файлов это больше одного?
<tagezi> я по поводу инклуда :)
<tagezi> inc/
* mikekaganski вспоминает мультфильм про обезьятку и удава
<tagezi> :)
<tagezi> ну, мы помещаетм в inc файлы которые испольщуются в большом количестве файлов :)
<mikekaganski> надо смотреть. Например, если этот файл используется в нескольких связанных файлах в одной директории, и заголовок там же - почему нет
<tagezi> потому что каша начинается типа ../source/basic/blablabla.hxx
<tagezi> а что такое DLL?
<tagezi> мне чего-то кроме динамических библиотек в винде ничего в голову не приходит
<tagezi> что это?
<libreofficeru-tg> <Kompilainenn> mikekaganski: ты откатил криворукие патчи и сделал нормально? :D
<mikekaganski> ну
<libreofficeru-tg> <Kompilainenn> плохо в этом только одно
<libreofficeru-tg> <Kompilainenn> ты один
<libreofficeru-tg> <Kompilainenn> а криворуких, которых никто не контролирует, много
<libreofficeru-tg> <Kompilainenn> блин, вчера Норберт ныл, что все плохо, сегодня Итало влез на любимого коня и заливает, что все хорошо, а МС говнари
<tagezi> МС бесплатно с++ препадаёт, чтобы больше разработчиков ЛО было :)
JohnDoe_71Rus has joined #libreoffice-ru
<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> лять, Хиско из всех вариантов выбрал самый говеный
<tagezi> нихрена эта переменная не нужна
<libreofficeru-tg> <Kompilainenn> mikekaganski: глянь плиз https://bugs.documentfoundation.org/show_bug.cgi?id=121759 и скажи свое мнение
<tagezi> раньше был кусок для эскейп, а теперь мы имеем не нужную переменную и не нужную функцию
<tagezi> вот она, сила документирования :)
<tagezi> хотя я не прав.. ладно, нужно отдахнуть
<mikekaganski> пролистай дальше по ветке
<libreofficeru-tg> <Kompilainenn> как по ветке? там одно сообщение
<libreofficeru-tg> <Kompilainenn> это проблема, которая требует большого UX
<libreofficeru-tg> <Kompilainenn> видать у Хейко UX не такой большой, что он взял и тупо плюнул
<tagezi> толкнёшь его?
<tagezi> а я сегодня пересоберу всё и попробую разобраться с этой переменной
<mikekaganski> мне не нравится просто "ComboBox". тако название может быть в 15 местах в коде. Если можно, сделай BasicComboBox или как-то подобно
<tagezi> так оно в директории basic
<mikekaganski> да, но когда я ищу по имени без пути, например в VS, оно мне вывалит кучу
<tagezi> вот я вас реально не понимаю... либо в коде бордак ещё больше чем "а ну его нахер", либо вы чего-то мудрите, сэр :)
<mikekaganski> здрассте
<tagezi> тогда лучше IDEComboBox
<tagezi> IDE у нас ещё есть?
<mikekaganski> как угодно - лишь бы побольше уникальности в именах файлов
<mikekaganski> к сожалению, нет
<mikekaganski> а питоновский IDE был бы очень кстати
<libreofficeru-tg> <Kompilainenn> А нельзя просто встроенную в питон ИДЕ притащить в ЛО?
<libreofficeru-tg> <Kompilainenn> Там же у них есть некая
JohnDoe_71Rus has quit [Quit: KVIrc 5.0.0 Aria http://www.kvirc.net/]
<tagezi> vim? :)
<libreofficeru-tg> <Kompilainenn> Нет
<libreofficeru-tg> <Kompilainenn> В поставке питона есть что то такое
<tagezi> нету там ничего, кроме питоно-консоли :)
<tagezi> и не нужно нам to` 10 IDE, нужно это до ума довести
<libreofficeru-tg> <Kompilainenn> это что? то, что есть для васика?
<tagezi> mikekaganski: тебе завтра напомнить, если забудешь?
<libreofficeru-tg> <Kompilainenn> во IDLE какая-то
<tagezi> где?
<libreofficeru-tg> <Kompilainenn> в составе Python идет
<libreofficeru-tg> <Kompilainenn> https://ru.wikipedia.org/wiki/IDLE
<tagezi> так она и встроена вроде
<tagezi> по внешнему виду, так точно она :)
<libreofficeru-tg> <Kompilainenn> но ты мне втирал, что там только консоль
<tagezi> ну, так это и есть консоль
<libreofficeru-tg> <Kompilainenn> а в ообще, есть Geany, вот и нечего вымахиваться
<mikekaganski> tagezi: естественно напомни
<tagezi> вообще есть IDE, который и есть IDE, просто там сам sberg ногу сломит, вот никто и не дергает его
<tagezi> mikekaganski: если я сделал ребейс после комита, а потом сделал второй коммит, он первый не отправит в геррит? :)
<mikekaganski> после коммита какого? уже который в мастере?
<tagezi> нет, он ещё не в мастере
<mikekaganski> тогда отправит
<mikekaganski> обновит в геррите
<tagezi> блин, и как мне быть?
<mikekaganski> а новый зависит от старого?
<tagezi> угу
<mikekaganski> тогда отправляй, нехай билдит
<mikekaganski> угу - я это понял, что зависит
<tagezi> да.. первый переименование, а второй с переменной бду развлекаться
<tagezi> так, нужно спать.. уже
<tagezi> завтра буду развлекаться уже..
<tagezi> ночи всем
<libreofficeru-tg> <Kompilainenn> Ппц развлекушечки
libreofficeru-tg has quit [Remote host closed the connection]
libreofficeru-tg has joined #libreoffice-ru