<mikekaganski>
ну какая разница какая последовательность, если конечный результат тот же?
<tagezi>
нет, конечный результат будет другой
<mikekaganski>
объясни
<tagezi>
аа, да..это было в другой версии файла.. я её разложил, теперь это не важно
<tagezi>
да, тогда можно вынести под иф, и работать быстрее будет и понятнее смотрится
<tagezi>
это всё? :)
<mikekaganski>
мне кажется, скобки вокруг nLeftM + nRightM лишние. А так - всё :)
<mikekaganski>
ну, комментарии, конечно
<tagezi>
эм..
<tagezi>
вообще читается лучше, и помоему ничему не мешает
<tagezi>
разве нет?
<mikekaganski>
ок; а пикселы почему не нравятся? функция возвращает размер в единицах editEngine (которые используются в данный момент), а ещё - в пикселах
<tagezi>
ну вот, у меня и возникает вопрос, всегдали это пикселы?
<tagezi>
если нет, то это сбивает с толку
<tagezi>
лучше тогда rNeededWidth
<tagezi>
она и для пиксилов подходит и для абстрактной единицы
<tagezi>
и отсюда пикселы убрать nAddWidthPixels
<tagezi>
ещё вопрос вот с этой строкой SetEngineTextAndGetWidth( rParam, aCellStr, nRepeatSize, ( nLeftM + nRightM ) );
<tagezi>
может вместо ( nLeftM + nRightM ) поставить 0
<mikekaganski>
а, конечно!
<tagezi>
это к чему было? :)
<mikekaganski>
[09:39:04]tagezi: может вместо ( nLeftM + nRightM ) поставить 0
<tagezi>
хорошо
<mikekaganski>
я проглядел
<mikekaganski>
смотри: у нас если там используется rParam.mbPixelToLogic, везде выше пикселы преобразуются в логические зазмеры
<mikekaganski>
а здесь в этом случае мы возвращаем назад в пикселы
<mikekaganski>
а если rParam.mbPixelToLogic не используется, то всё и так в пикселах
<tagezi>
понятно, значит пикселы
<mikekaganski>
да, ещё
<mikekaganski>
const long nSumM -> long nSumM
<mikekaganski>
это же не ссылка
<tagezi>
но это константа
<tagezi>
нет?
<mikekaganski>
можно и оставить
<mikekaganski>
но обычно это в параметрах излишнее
<mikekaganski>
загромождает объявление
<mikekaganski>
функции
<tagezi>
ну, как скажешь, тебе виднее
<tagezi>
mikekaganski: вот смотри, эта функция высчитывает ширину прямоугольник с текстом, так?
<tagezi>
а что такое Engine в данном контексте?
<tagezi>
этоже не мотор, в прямоугольнике текста нет моторов
<tagezi>
что она вообще возвращает?
<tagezi>
Я так понимаю, что это служебная переменая, для хранения временных результатов, а смасл у неё Средство, Инструмент
tagezi has quit [Remote host closed the connection]
tagezi has joined #libreoffice-ru
<mikekaganski>
по-русски? не задумывался. Я её рассматриваю именно как механизм
<tagezi>
он логн (число), она не делает операции :)
<mikekaganski>
?
<tagezi>
механизм - это набор элементов, которые предназначены для получения какого-либо результата
<tagezi>
то есть, там кроме штук, есть ещё действия, которые используют штуки для производства результата
<mikekaganski>
а ScFieldEditEngine - это не то?
<tagezi>
в функции, она выступает как переменная, для хранения промежуточного результата
<mikekaganski>
нет, она принимает текст, хранит внутри себя форматирование, шрифт, и на этой основе даёт размеры
<tagezi>
да, она от туда приходит, но я не понимаю, что там происходит... там в параметрах конструктора вызывается функция, и дальше я теряю нить
<mikekaganski>
это как раз механизм
<JohnDoe_71Rus>
blackbox
<tagezi>
mikekaganski: кроме как к функции, ещё коментарии нужны?
<mikekaganski>
наверное, нет
<mikekaganski>
прикольно. я задался посмотреть, как используется mpEngine
<mikekaganski>
в конструкторе инициализируется нулём
<mikekaganski>
а потом везде используется без проверки
<mikekaganski>
полагается на правильную инициализацию клиентом
<mikekaganski>
хреновый дизайн
<mikekaganski>
я бы понял, если бы это была структура... а то класс
<mikekaganski>
я про DrawEditParam, если что :)
tagezi has quit [Quit: No Ping reply in 180 seconds.]
<kompi_>
tagezi: руководитель подразделения в большой компании сколько получает?
<mikekaganski>
tagezi: // Returns the required engine width for the string;
<tagezi>
mikekaganski: это где?
<mikekaganski>
// return the width in pixels by reference
<mikekaganski>
tagezi: комментарии - к функции
<mikekaganski>
и код там же
<tagezi>
там одно значение возвращается как результат функции, а второе изменяется по ссылке
<mikekaganski>
ну да
<kompi_>
Строитель – 3475 - это рядовой строитель!
<kompi_>
а у меня меньше 1000 евро сейчас
<kompi_>
так сравнения ваши это так
<tagezi>
строители не знаю, но если ты зарабатываешь 1500,то половину тратишь на квартиру, если не в ипотеке, а если в ипотеке, то 3/4
<tagezi>
+бензин 1,6 евро литр
<kompi_>
ты мне только что сказал, что 280 евро
<kompi_>
откуда половина взялась?
<tagezi>
а если больше 1600, то там налог начинает расти бешенно.. у меня знакомый получает 4000, 2 отдает налогами, 600 за квартиру и 200 на бензин
<tagezi>
электричество 200, симать 400-600
<tagezi>
500+200
<tagezi>
700, почти половина.. это тут в мухосрани
<tagezi>
в Хелсинки ещё круче
<tagezi>
mikekaganski: ну я и написал подробно, чтобы было понятно что куда возвращается
<tagezi>
mikekaganskiты думаешь это слишком много?
<mikekaganski>
нет
<mikekaganski>
renuts
<mikekaganski>
наоборот мало
<mikekaganski>
// Sets a needed width for string and renuts it by reference
<mikekaganski>
sets куда? returns by reference куда?
<mikekaganski>
// Sets rNeededPixel to needed width for string
<mikekaganski>
njulf gjyznyj
<mikekaganski>
тогда понятно
<mikekaganski>
returns needed width in current units; sets rNeededPixel to needed width in pixels
<mikekaganski>
потому что возвращает - это по return, и там не по ссылке
<tagezi>
rParam.mpEngine->SetText( OUString( rSetString ) ) - я упустил
<mikekaganski>
ок. И можно выдавать
<tagezi>
он блокируется предыдущим патчем, нужно тот соммитить в мастер, и тогда этот уже ребейзить и посылать
<kompi_>
изыски в большом проекте
<tagezi>
я потому и просил право симмитить, чтобы вот как с прошлым патчем не было, эйка сказал что поправить, я поправил, а он это только завтра вечером толкнёт
<kompi_>
не дадут
<tagezi>
да нужно было год назад было просить, тогда были патчи мои, а теперь один ответ, у тебя мало патчей
<tagezi>
за последние три месяца
<kompi_>
я прав оказался что ли?
<kompi_>
ну и не спеши никуда
<tagezi>
даже Маркус тут написал.. видимо ему ненакого было желч слить