04:05 <mikekaganski> не понял. Там ещё 6 кусков повторяет твою функцию: 2908-2912, 2923-2931, 3333-3337, 3348-3356, 3594-3598, 3609-3617

04:08 <mikekaganski> нужно только вынести if из функции

04:09 <mikekaganski> и постараться убрать функцию из класса (передавать всё нужное в параметрах)

08:41 <tagezi> ооооо, анализ кода.. а ты уверен что никто не запутается?

08:45 <tagezi> и 2908 это немного другое... если только построить дерево, то есть сделать 2 функции, одна из которых будет вкладываться в эти мелкие места, и в функцию, которая будет объединять большие куски

08:51 <kompi_> mikekaganski: тут?

09:09 <tagezi> ты бы тоже сыном и женой позанимался

09:42 <kompi_> такое ощущение что стили, которые я загрузил из другого документа при импорте бьются

09:43 <kompi_> tagezi: которым из? обоими? я тока и делаю сегодня, что их по разным комнатам развожу, чтобы они не бесились

10:50 <mikekaganski> tagezi: 2908 - это то же самое, где nLeftM и nRightM по нулям, а rEngineWidth не используется. Кстати, зачем тебе отдельные nLeftM и nRightM ?

10:50 <tagezi> всмысле?

10:50 <tagezi> ты уверен, что они не разные?

10:52 <mikekaganski> они могут быть разными, но зачем их передавать в функцию по отдельности, а не одним параметром, как nLeftM + nRightM ,

10:52 <tagezi> а, ты про это

10:53 <tagezi> ну, тут я тупанул

11:04 <tagezi> я бы не стал объединять, оно запутано получиться

11:05 <tagezi> а, не, можно попробовать

11:05 <kompi_> mikekaganski: есть желание потрещать?

11:06 <tagezi> ладно, я попробую сделать

11:06 <tagezi> mikekaganski: объясни мне по поводу класса

11:07 <tagezi> там вроде все функции находятся в ScOutputData

12:48 <mikekaganski> сорри, я тут не всё время дома

14:11 <kompi_> mikekaganski: без проблем

15:25 <tagezi> https://github.com/seanyeh/vibreoffice

15:49 <JohnDoe_71Rus> еще окошки тайтлами замостить

16:29 <kompi_> ну ппц

16:33 <kompi_> tagezi: а не помнишь такую мульку, чтобы в диалоге Настройка стилей отображались параметры, которые наследуются без изменений из родительского стиля

16:34 <kompi_> ну имеется ввиду, как-то выделялись

20:03 <tagezi> mikekaganski: тык

20:03 <tagezi> какой префикс у строковый переменных?

20:04 <tagezi> у OUString

20:04 <tagezi> они как объекст обозначаються?

20:04 <mikekaganski> я использую s или a

20:30 <tagezi> mikekaganski: https://opengrok.libreoffice.org/xref/core/sc/source/ui/view/output2.cxx#2908

20:30 <tagezi> как ты это видишь?

20:30 <tagezi> там нет элса

20:31 <tagezi> то есть, при выполнении с элсом значение будет меняться, вместо того чтобы оставить его темже

20:32 <mikekaganski> Смотри: здесть роль "rNeededPixel" играет nRepeatSize; только сво "else" он получает до ифа

20:34 <mikekaganski> это аналогично тому, если бы сначала присвоить (long) rParam.mpEngine->CalcTextWidth() какой-то другой переменной (nEngineWidth), а потом её значение присвоить либо через трансформацию, либо напрямую в зависимости от условия

20:36 <tagezi> nNeededPixel = nEngineWidth;

20:36 <tagezi> nNeededPixel += nLeftM + nRightM;

20:36 <tagezi> я не вижу ничего подобного

20:37 <tagezi> nRepeatSize это понятно

20:38 <mikekaganski> nLeftM = 0; nRightM = 0

20:39 <mikekaganski> long nRepeatSize = (long) rParam.mpEngine->CalcTextWidth();

20:39 <mikekaganski> if (rParam.mbPixelToLogic)

20:39 <mikekaganski> nRepeatSize = mpRefDevice->LogicToPixel(Size(nRepeatSize,0)).Width();

20:39 <mikekaganski> else

20:40 <mikekaganski> nRepeatSize = nRepeatSize;

20:40 <mikekaganski> nRepeatSize += 0;

20:41 <tagezi> у нас разные версии файла?

20:42 <tagezi> ты мне показываешь начиная с этой строки https://opengrok.libreoffice.org/xref/core/sc/source/ui/view/output2.cxx#2923

20:43 <tagezi> а нет

20:44 <tagezi> да, я понял

20:44 <tagezi> и ты думаешь

20:45 <tagezi> это кто-то поймет?

20:45 <mikekaganski> Понимаешь, мне кажется, если ты находишь общие паттерны, это наоборот станет понятнее. Труднее их найти, а понять готовое, особенно если названия хорошие, проще

20:47 <mikekaganski> заодно проще сопровождать. Например, потом кто-то найдёт ошибку в логике. Фиксить все места, особенно если общность паттерна неочевидна - а найдёт ли он их?

20:48 <tagezi> да, я с тобой согласен, но с названиями я убьюсь

20:48 <mikekaganski> Сделай проще: типа SetEngineTextAndGetWidth, или что-то в этом роде

20:49 <mikekaganski> И хрен с ним, что длинное, зато говорящее

20:49 <tagezi> ну, это имя функции

20:49 <mikekaganski> ну, а остальные оставь как есть

20:49 <tagezi> эм

20:50 <mikekaganski> можем обсудить имена, не проблема. Там и комментарии нужны

20:51 <tagezi> Размет_повторения будет сразу и Размером_локомотива и Высотой_лакомотива :)

20:51 <tagezi> ,

20:51 <tagezi> ?

20:52 <mikekaganski> имена - это одна из двух самых трудных задач в программировании, наряду с очисткой кэша и ошибками смещения на единицу :)

20:53 <tagezi> я знаю, я даже в своих мелких проектах с этим всегда мучаюсь

20:53 <mikekaganski> если ты уберёшь из функции внешний иф, то мы всегда будем инициализировать nEngineWidth, и тогда мы можем вернуться к идее второго патчсета

20:53 <tagezi> а тут большой проекст

20:54 <tagezi> иф я убрал

20:57 <tagezi> да, тогда и алгоритм будет нагляднее

21:02 <tagezi> знаешь что страшно?

21:03 <tagezi> вот я сейчас всё исправлю, а потом все как вспомнят что там всё было скопипасчено, но не доделано, и всё коту под хвост :)

21:20 <mikekaganski> не парься. Я уже не раз на это натыкался. Это норма

21:20 <mikekaganski> :)

21:57 <tagezi> mikekaganski: послал драфт

21:57 <tagezi> без коментариев пока

23:35 <tagezi> так, ладно, больше я не рожу, наверное, уже..

23:36 <tagezi> нужно по поводу переменных подумать.. мне пикселы не нравятся.. и по поводу документации, но это завтра, сейчас уже спать пора