<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, если оно получается в результате каких-то вычислений, привожу их, чтобы понимали, откуда я это взял
<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?
<tagezi>
сколько у тебя локальных веток в проекте открыто? :))
<mikekaganski>
когда как. Я многие убиваю, если надо - восстанавливаю из геррита
<tagezi>
а этот патч почему висит?
<tagezi>
сегодня дженкинс тормазной какой-то
<mikekaganski>
он вообще последнее время жутко перегружен
<mikekaganski>
там даже поменяли логику, он теперь автоматом отменяет билды старых версий при поступлении новой, и сначала билдит лин, и только при успехе вин - чтобы ускорить