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
VitaliyAnderson has quit [Read error: Connection reset by peer]
JohnDoe_71Rus has joined #libreoffice-ru
<tagezi> mikekaganski: объяснишь мне это? https://opengrok.libreoffice.org/xref/core/sw/qa/extras/ww8export/ww8export.cxx#53
<mikekaganski> давай, в чём вопрос?
<tagezi> если 120 >= 0 то константа, оначе таже константа
<mikekaganski> ? почему 120?
<mikekaganski> и соответственно, почему константа
<mikekaganski> TWIP - это пакраметр макроса
<tagezi> пакрамент?
<mikekaganski> параметр :)
<mikekaganski> а, он только в одном месте использован, вижу
<mikekaganski> ну, сделали общий алгоритм, а использовали только в одном месте
<mikekaganski> и да, там это вычисляется при компиляции и подставляется результат
<tagezi> функцию делать такойже?
<mikekaganski> наверное, не надо это менять. Я не посмотрел, что это юниттесты, не основной код
<tagezi> эм
<mikekaganski> хотя макросы - всегда зло :)
<tagezi> а юнит тесты не нужно чистить?
<mikekaganski> нужно, просто у меня к ним менее строгие внутренние убеждения :)
<tagezi> это таже программа, только она отвечает за работоспособность кодв другой программы
<tagezi> так что она должна быть сделана более идеально, чем основная :)
<mikekaganski> сделай такой же
<mikekaganski> :)
<tagezi> понял
<tagezi> нужно будет разобраться с юнит тестами
<tagezi> а возвращать что? лонг или инт?
<mikekaganski> sal_uInt32
<tagezi> угу, вижу
<mikekaganski> Всё-таки при умножении там оставь L
<tagezi> так я приму параметр как лонг
<mikekaganski> ну да, почему нет
<tagezi> и тогда L yt ye;yf
<tagezi> не нужна*
<mikekaganski> да, верно
<tagezi> спасибо :)
<mikekaganski> нет
<tagezi> что нет? :)
<mikekaganski> так, всё ты правильно говоришь, не обращай на меня внимания. Нужно кофе пить
<JohnDoe_71Rus> кофе вредно. может лучше свежий сок?
<mikekaganski> :) это предрассудки :)
<mikekaganski> кофе исключили из потенциальных канцерогенов
<tagezi> JohnDoe_71Rus: мне врач прописалл по утрам пить, чтобы давление нормализовать
<tagezi> так что не нужно туту ляля :)
<JohnDoe_71Rus> из канцерогенов может и да, а на сердечко действует
<JohnDoe_71Rus> тут по ящику показывали про "кофе с собой" стаканы на 0,5L а кофе там до 60мл. хотя у меня отец на такую же кружку 3 ложки растворимого сыпал
<tagezi> и потом, фины пьют кофе каждый день литрами, одна из самый пьющий кофе наций на земле, и живут охренеть как долго :)
<JohnDoe_71Rus> забавно, люблю шоколад, и шоколад мокко. а в жидком виде ни кофе ни какаву не могу пить
<JohnDoe_71Rus> tagezi: они все северные долго живут, половину жизни криозаморозка :)
kompi has joined #libreoffice-ru
VitaliyAnderson has joined #libreoffice-ru
<tagezi> mikekaganski: почему макросы это плохо?
<mikekaganski> ооооо... это целая песня. :) там и религиозные доводы, и рациональные...
<tagezi> этоже как инлайн функция, фактически
<tagezi> просто кусок кода который подставляется куда нужно
<mikekaganski> да. Но поскольку это препроцессируемый кусок кода, то зачастую трудно дебагить тамошние проблемы. Поэтому по крайней мере там, где есть эквивалентные решения без макросов, лучше ими не пользоваться.
<tagezi> ясно
<tagezi> mikekaganski: такой коментарий имеет смысл? Now nTwip always = 120. It need to understand why there is choose. And if it does not needed to some supposed tasks, it needs to be simplify.
<tagezi> к функции
<mikekaganski> а зачем?
<tagezi> там не нужен выбор
<mikekaganski> функция понятна. Она была использована один раз, но если понадобится снова в этом модуле, будет использована снова
<tagezi> сейчас в коде там можно сделать константу
<tagezi> ну, оно уже 6 лет в таком состоянии
<mikekaganski> ну и ладно :) это нормальное состояние. Если код не вызывает непонимания, то не принципиально.
<mikekaganski> Я зачастую не использую просто число 1234, если оно получается в результате каких-то вычислений, привожу их, чтобы понимали, откуда я это взял
<tagezi> я вижу только ожно определение макроса, кстати
<tagezi> mikekaganski: кстати, а каолан с самого начала проекта чтоли?
<mikekaganski> да
<tagezi> у него есть коменты за 2001 год
<tagezi> охренеть, так он код на изусть знает
<tagezi> 16 лет в одном проекте, он за место гита может работать :)
<kompi> он соло написал же либру
<kompi> мы же уже договорились
<JohnDoe_71Rus> да проще, либру написал тот кто сделал форк с опена
<tagezi> mikekaganski: я думаю оно было просто тупо скопипасчено туда
<mikekaganski> о, всё верно
<mikekaganski> так что можно просто инклюдить это :)
<tagezi> да, оно и ругается, что две одинаковых функции
<tagezi> зачем вообще был макрос то тогда?
<mikekaganski> вот и нафиг его
<tagezi> ладно, убедил, седом все замены не сделать, всеравно придется руками лазить и думать
<tagezi> mikekaganski: а можно прогонять тесты только из оного модуля?
<mikekaganski> да
<tagezi> как?
<mikekaganski> сек
<mikekaganski> make CppunitTest_sc_ucalc
<mikekaganski> там имя mk-файла соответствующего модуля
<mikekaganski> соответственно, есть такой CppunitTest_sc_ucalc.mk
<mikekaganski> или cd sw && make - всё в этом модуле, включая тесты
<tagezi> последне говорит что нет задач
<mikekaganski> хм
<mikekaganski> попробуй make sw
<tagezi> тоже самое
<mikekaganski> ты из корня?
<tagezi> да.. это я попробовал в первую очередь
<mikekaganski> странно
<tagezi> [BIN] top level modules: sw
<tagezi> [LOC] top level modules: sw
<tagezi> [ALL] top level modules: build-non-l10n-only build-l10n-only
<tagezi> make: Цель «all» не требует выполнения команд.
<mikekaganski> Может, оно в этом виде не делает тесты? попробуй make sw.chaek
<mikekaganski> check
<tagezi> угу, работает, спасибо
<tagezi> а make CppunitTest_sw_* просто тупо собирает все тесты :)
<tagezi> mikekaganski: а при вычитании int приводиться в лог?
<mikekaganski> а, это ты про мой вчерашний коммент?
<mikekaganski> там всё непросто
bormant has joined #libreoffice-ru
<mikekaganski> при вычитании беззнакового целого из целого со знаком сначала целое со знаком приводится к беззнаковому, потом производится вычитание, потом результат приводится к типу результата...
<tagezi> тут везде всё не просто :)
<mikekaganski> :)
<mikekaganski> почему c/c++ не любят
<tagezi> потому что процессоры стали быстрые
<tagezi> почему на асме не пишут? :)
<tagezi> потому что скорость разработки важнее быстродействия
<tagezi> почему оно тут безнаковое?
<tagezi> sal_uLong nWW8BrushStyle
<mikekaganski> наверное, потому что в спецификации это беззнаковое целое
<tagezi> да, нашё определение
<tagezi> значит получается что из знакового лонг вычитается беззнаковый лонг лонг
<tagezi> а я пытаюсь заменить на инт
<tagezi> mikekaganski: и где тут камень?
<mikekaganski> нигде, я тебе написал вчера, что у тебя всё нормально
<tagezi> инт приведётся к беззнаковому инту или лонгу
<mikekaganski> > [00:46:18]mikekaganski: ww8par6.cxx - у тебя всё правильно, но блин хотел бы я понять, что там творится
<tagezi> ну, я уверен, что если бы был реальный косяк, ты бы мне сказал
<tagezi> я просто понять хочу :)
<tagezi> ясно, ты не в курсе :)
<mikekaganski> я тоже понять хочу :) код, что называется, попахивает.
<mikekaganski> tagezi: слушай, а в wrtww8gr.cxx недоступна эта convertTwipToMm100?
<mikekaganski> чтобы не плодить одно и то же
<mikekaganski> Set_UInt16(pArr, msword_cast<sal_uInt16>(convertTwipToMm100(aGrTwipSz.Width())));
<tagezi> mikekaganski: не понял
<tagezi> второй патч не полный
<mikekaganski> это как раз тот случай, когда у нас пятьсот способов сделать одно и то же
<tagezi> третий смотри
<mikekaganski> wrtww8gr.cxx
<tagezi> понял тебя
<mikekaganski> frmpage.cxx - мы вчера говорили насчёт умножения на 100, ты проверил, что это безопасно убирать у сотки L
<mikekaganski> ?
<tagezi> не знаю.. там тип Size, и я не нахожу определения
<mikekaganski> tagezi: include/tools/gen.hxx
<mikekaganski> безопасно: там и так лонг
<tagezi> спасибо
* tagezi хрен нашёл бы
<mikekaganski> у тебя vim?
<tagezi> угу
<tagezi> о что?
<tagezi> а*
<mikekaganski> спроси у vmiklos насчёт IDE-интеграции его
<mikekaganski> он тогда, возможно, ьудет нормально помогать с поиском определений
<tagezi> в вики написано.. мне нужно только кланг настроить
<tagezi> а, и тагс тоже :)
<tagezi> mikekaganski: про Set_UInt16(pArr, msword_cast<sal_uInt16>(convertTwipToMm100(aGrTwipSz.Width())));
<tagezi> я не понимаю как это должно работать
<mikekaganski> почему? мы переводили твипы в мм100 через домножение на константу, а у нас есть для этого функция
<mikekaganski> если эта функция здесь доступна, почему ей не воспользоваться?
<tagezi> а
<tagezi> ну, можно добавить в заголовок и тогда использовать функцию
<mikekaganski> да, проверь, если она доступна так, то не добавляй заголовок
<mikekaganski> добавь если недоступна
<tagezi> слушай, не матерится
<tagezi> mikekaganski: есть ещё что-нибудь?
<mikekaganski> вроде всё
<tagezi> ладно, тогда собираю с тестами и публикую, пусть дженкинс проверяет
<tagezi> mikekaganski: кого там в ревьюеры поставить, или ты сам комит сделаешь, если всё хорошо?
<mikekaganski> поставь mst
<mikekaganski> только после успешной сборки дженкинсом
<tagezi> mikekaganski: а кто это?
<mikekaganski> Michael Stahl
<tagezi> логово медведей
<tagezi> mikekaganski: https://gerrit.libreoffice.org/#/c/46073/ а зачем Миклос ставит себе 2 плюсика?
<mikekaganski> чтобы никто не парился проверкой его патча
<mikekaganski> он сам его запульнёт после дженкинса
<tagezi> тоесть он просто хочет чтобы дженкинс проверил
<mikekaganski> да
<tagezi> блин, обсуждали обсуждали, а изменений только в обном файле
<tagezi> жжжж
<mikekaganski> ?
<tagezi> да долго обсуждали равное, сейчас комит делаю, а изменён только один файл
<mikekaganski> разве?????
<tagezi> ну вот да
<tagezi> https://gerrit.libreoffice.org/#/c/46041/3..4/sw/source/filter/ww8/wrtww8gr.cxx
<mikekaganski> ничё подобного. Может ты --amend забыл?
<tagezi> остальное проверка, и обучение студента :)
<kompi> tagezi: ты был прав, дизайнеров хрен убедишь
<kompi> долбоящер Хейко WFM, а Юсуф отмахнулся, что это дубликат
<tagezi> они живут в своём личном мире, у них своя концепция: я этим не пользуюсь - это никому не нужно
<kompi> при этом достаточно много запросов на эту тему висит, штуки 4 наверное, а они все не могут сделать несчастный один элемент контекстного меню
<mikekaganski> kompi: а что за баг?
<tagezi> да, 50 изменений в 27 файлах, это многовато как-то по ощущениям
<kompi> tagezi: пусть проверяют ленивцы эти
<tagezi> ну, у девов нормально работы.. бубли вон, второй или третий день не можут проверить
<tagezi> она ожно замечание высказала, и молчит
<kompi> mikekaganski: баг логичный
<kompi> mikekaganski: я не хочу писать текст в фигуре, я хочу его настроить для шаблона (ну как пример)
<kompi> а диалог текст спрятан по хитрому
<kompi> mikekaganski: то, что сейчас видно в логе геррита по коммитам, это уже в 6.1 идет?
<mikekaganski> да, если в мастере
<kompi> я хз где там
<JohnDoe_71Rus> 6.0 уже запечатали?
JohnDoe_71Rus has quit [Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org]
<kompi> уже бета 2
kompi has quit [Quit: Page closed]
<tagezi> почему висит?
<tagezi> сколько у тебя локальных веток в проекте открыто? :))
<mikekaganski> когда как. Я многие убиваю, если надо - восстанавливаю из геррита
<tagezi> а этот патч почему висит?
<tagezi> сегодня дженкинс тормазной какой-то
<mikekaganski> он вообще последнее время жутко перегружен
<mikekaganski> там даже поменяли логику, он теперь автоматом отменяет билды старых версий при поступлении новой, и сначала билдит лин, и только при успехе вин - чтобы ускорить
kompi has joined #libreoffice-ru
<kompi> люблю такие документы, кривая таблица в таблице в таблице
<mikekaganski> хм :) третий уровень вложенности не вижу.
JohnDoe_71Rus has joined #libreoffice-ru
<kompi> mikekaganski: а таблицу без половины полей видишь?
<kompi> не полей, а границ
<mikekaganski> да, я полагал, что там просто отключены границы
kompi has quit [Quit: Page closed]
<mikekaganski> kompi: в дев: "[19:22:28]Francesco_G: I'm wotking on this: https://bugs.documentfoundation.org/show_bug.cgi?id=108865"
bormant_ has joined #libreoffice-ru
<JohnDoe_71Rus> http://www.opennet.ru/opennews/art.shtml?num=47706 вот как надо
<tagezi> mikekaganski: это должно, наверное, развивать память. Если сделал патч, который месяц никто не просматривает, то забудешь вообще что там было
bormant_ has quit [Quit: Leaving]
kompi has joined #libreoffice-ru
kompi has quit [Ping timeout: 260 seconds]
mikekaganski has quit [Read error: Connection reset by peer]
mikekaganski has joined #libreoffice-ru
ChanServ has quit [shutting down]
ChanServ has joined #libreoffice-ru
JohnDoe_71Rus has quit [Quit: KVIrc 4.9.2 Aria http://www.kvirc.net/]