03:23 <tagezi> mikekaganski: а зачем тут явное приведение типа? https://opengrok.libreoffice.org/xref/core/sc/source/core/data/table1.cxx#1322

05:37 <mikekaganski> где приведение?

05:37 <mikekaganski> или там типа явное привИдение?

07:24 <kompi> утра

10:17 <tagezi_> mikekaganski: да, типа :)

10:17 <mikekaganski> tagezi_: а серьёзно - что там за приведение - я не увидел

10:18 <tagezi_> там шаблон https://opengrok.libreoffice.org/xref/core/include/sal/types.h#459

10:18 <mikekaganski> а, ты не про эту строку, а про строки 1332-1333?

10:18 <tagezi_> а тут он используется https://opengrok.libreoffice.org/xref/core/sc/source/core/data/table1.cxx#1332

10:21 <mikekaganski> без понятия на самом деле

10:22 <tagezi_> ну, вот это похоже карго культ какойто :)

10:22 <tagezi_> https://opengrok.libreoffice.org/xref/core/sc/source/core/data/table1.cxx#1352

10:23 <tagezi_> вот тут, почему в скобках это делается понятно, там разные типы

10:24 <tagezi_> хотя это бешенно, оба типа одного типа целочисленные :)

10:27 <tagezi_> а не, разные

10:29 <tagezi> ладно, наверное, это можно оставить в покое.. "работает не трогай"

10:29 <tagezi> мне ветот цыкл нужно только добавить проверку на защищенные ячейки

10:30 <mikekaganski> tagezi: вот тебе пример "работает - не трогай": https://gerrit.libreoffice.org/#/c/43909/2/basic/source/comp/codegen.cxx

10:31 <mikekaganski> здесь косяк, который может дать неправильный результат, закрыли явным приведением и заглушили предупреждение компилятора

10:39 <tagezi> ну.. в данном случае я не знаю как может быть переполнение, количество столбцов и строк фиксированое в программе

10:40 <tagezi> я о идее не могу выйти за диапазон

11:33 <mikekaganski> tagezi: в твоём случае все вроде нормально. Я просто про то, что разные случаются случаи :)

11:44 <tagezi> mikekaganski: слушай.. а если я функцию перенесу в другой класс, мне сильно уши надерут?

11:46 <tagezi> https://opengrok.libreoffice.org/search?project=core&q=SkipCursorHorizontal&defs=&refs=&path=&hist=&type=

11:47 <tagezi> https://opengrok.libreoffice.org/xref/core/sc/source/core/data/table1.cxx#1322

11:47 <tagezi> вот сюда

11:47 <tagezi> ей все равно где жить, а мне не нужно будет с объектами туда сюдакаться :)

11:48 <mikekaganski> не понял. Дай патч плиз

11:48 <tagezi> да я ещё не патчил, я только думаю

11:49 <tagezi> https://opengrok.libreoffice.org/xref/core/sc/source/core/data/table1.cxx#1350 мне вот в этом цыкле, нужно обработать защищенные ячейки

11:50 <tagezi> функция которая их обрабатывает https://opengrok.libreoffice.org/xref/core/sc/source/ui/view/tabview2.cxx#793

11:50 <tagezi> она используется только в одном месте https://opengrok.libreoffice.org/xref/core/sc/source/ui/view/tabview3.cxx#1256

11:52 <tagezi> если я буду обрабатывать защещенные ячейки в том цикле, это сразу решит кучу багов, типа с вылетом курсора за маркерованную область, передвижение между защищенными ячейками

11:59 <mikekaganski> попробуй, потом посмотрим :)

11:59 <tagezi> так.. я не вижу подвоха? :)

12:00 <mikekaganski> нет его

12:01 <mikekaganski> я просто не вижу твоего кода, и не могу вникать пока что

12:02 <tagezi> да, я понял что, на то что я смотрю уже пару недель, ты видишь в первый раз

12:05 <mikekaganski> не в первый, но так и не вникал подробно

12:05 <mikekaganski> открою, погрущу, закрою

12:06 <tagezi> ну, я сделаю патч, заодно мысль до конца оформится

12:07 <tagezi> может Айка и прав был, нужно все разом делать и для таба тоже

12:07 <tagezi> а может нужно наоборот, фукцию ГетНексПозитион перемещать

15:23 <bormant> тэкс, 6.0.0.3 переименовали в релиз

23:42 <tagezi> и всё таки я гений :))