<mikekaganski>
чтобы изменить gerrit change, нужно туда отправить снова патч, у которого строка Change-Id в commit message такая же. Никаких форспушей не надо.
<mikekaganski>
Всё отслеживается черед change id
<dshil>
change-id соот-т патчу, не коммиту, так?
<mikekaganski>
патчу
<mikekaganski>
Убери обещание "It's only a preparation before fixing the whole stuff with manual memory management in the GDI related code"
<mikekaganski>
зачем оно?
<mikekaganski>
изменение рационально само по себе, безо всякий условностей
<mikekaganski>
честно сказать, я бы pActSysColor = pFirstSysColor = nullptr перенёс прямо под SysColorEntry* pEntry = pFirstSysColor;
<mikekaganski>
о, Ноэль тоже одобрям
<dshil>
сек, сейчас отвечу на все твои комментарии по порядку
<libreofficeru-tg>
<Kompilainenn> хе и tagezi в этом баге вджобывал
<mikekaganski>
ну, @Kompilainenn, теперь тебе в этом диалоге печати пахать-неперепахать.
<dshil>
Патчей будет несколько и я думал, как читающему коммиты проследить всю логику.
<dshil>
Подумал о том, что можно заложить эту логику в коммиты. С другой стороны после
<dshil>
место, вынес все отвлеченные разговоры в багзилу, в коммитах оставил только то,
<dshil>
твоего замечания Я еще раз подумал. В итоге согласен, коммиты совсем не то
<dshil>
что касается самих изменений.
<dshil>
упс, простите, думал weechat умеет вставлять multiline-ы, в итоге разбилось на несколько сообщений.
<mikekaganski>
насчёт stupid - эт ты погорячился. Патчи могут быть mechanical, но никак не stupid :-)
<mikekaganski>
что касается unique_ptr vs shared_ptr, тут надо смотреть. Всё зависит от lifecycle management
<mikekaganski>
конечно, если всё просто, то unique_ptr покатит. Но, боюсь, там будет не всегда "вызови функцию, передай в неё HDC, дождись завершения, продолжи"
<dshil>
не, stupid они еще как могут быть, вот мой первый патч очень даже stupid :)
<dshil>
unique_ptr vs shared_ptr да, там не везде ложится, но я буду править именно те места, где легко ложится, над другими буду думать.
<mikekaganski>
так, насчёт переноса обнуления вверх - что ты думаешь? отправлять патч так, или подумаешь про это?
<dshil>
первое, что смутило, если delete зафэилится каким-то образом, то указатели на начало и текущий элемент уже будут инвалидированны. С другой стороны, если delete зафэилится и мы каким-то боком опять попробуем освободить этот
<dshil>
список, то это ни к чему хорошему не приведет.
<mikekaganski>
dshil: если будет фейл на освобождении первого элемента - *возможно*, какой-то смысл и был бы. Но даже частично проведённая деструкция первого, уже не говоря про успех первой операции, требуют очистки
<mikekaganski>
худшее здесь - утечка при ошибке (а работа корректная, поскольку нам *надо* чистить). А худшее в нынешнем случае - повторная деструкция и сегфолт
<dshil>
mikekaganski: угу, согласен про повторную деструкцию. Перенес.
<dshil>
остался последний орг вопрос. В геррите указано, что parent not current - rebase possible. Хотя я забирал последний мастер.
<mikekaganski>
ну и что? :-) только за сегодняшнее утро уже нападало 32 коммита
<mikekaganski>
начиная с 7 утра
<mikekaganski>
(по москве, сорри)
<mikekaganski>
в твоём случае ничего страшного
<dshil>
не, имел ввиду, что прям забрал мастер -> ребейз -> пуш в геррит -> все равно не up to date. сейчас еще раз попробую.
<dshil>
о, мне все время не везло и кто-то все время пушил в мастер в этот момент, теперь я попал в окно, когда никто не пушит :)
<mikekaganski>
Это про импорт, и про стили. Не знаю, какая мета туда подходит. В общем, наверное, лучше импорт DOC (не DOCX)
<libreofficeru-tg>
<Kompilainenn> как умнее МЕТА назначить? наверное все же DOC-Table
<mikekaganski>
да
<libreofficeru-tg>
<Kompilainenn> изначально это же просто таблица в DOC, она же в самом DOC не во фрейме?
<mikekaganski>
да
<libreofficeru-tg>
<Kompilainenn> спасибо
<mikekaganski>
прополка и пересадка? вроде зима на дворе, а ты садово-огородничаешь
<libreofficeru-tg>
<Kompilainenn> не понял про огород
<mikekaganski>
ну, как же: берёшь баги, аккуратно пересаживаешь в нужные меты...
<libreofficeru-tg>
<Kompilainenn> =)
<libreofficeru-tg>
<Kompilainenn> мне нравится порядок, раз уж завели какую-то структуру, то и надо все, что можно структурировать
<libreofficeru-tg>
<Kompilainenn> тем более, что это просто, не то, что код писать и ломать голову, как назвать переменную и почему хидер, схх и ui для ОДНОГО диалога имеют РАЗНЫЕ названия
<libreofficeru-tg>
<Kompilainenn> на самом деле облако тэгов было бы просто супер в багзилле
<libreofficeru-tg>
<Kompilainenn> одна сущность, которая могла бы заменить кучу полей в баге
<libreofficeru-tg>
<Kompilainenn> mikekeganski: есть документ writer, в нем фрейм, во фрейме контрол
<libreofficeru-tg>
<Kompilainenn> у контрола признак печатать стот Да, у фрейма - Нет
<libreofficeru-tg>
<Kompilainenn> конрол должен печататься в таком случае?
<mikekaganski>
нет
<mikekaganski>
как "crash" по-русски?
tagezi has joined #libreoffice-ru
<libreofficeru-tg>
Nanciscor (@Nanaciscor) has joined the Telegram Group!
<libreofficeru-tg>
<Kompilainenn> ломать
<libreofficeru-tg>
<Kompilainenn> упал-сломался
<mikekaganski>
имеется ввиду "caused a crash"
<libreofficeru-tg>
<Kompilainenn> вызвало сбой, говорит нам ГТ
<libreofficeru-tg>
<Kompilainenn> я бы добавил "фатальный"
<mikekaganski>
сбой - как-то мягковато
<mikekaganski>
а... возможно
<libreofficeru-tg>
<Kompilainenn> фатальный сбой
<libreofficeru-tg>
<Kompilainenn> тебе для чего?
<mikekaganski>
для перевода :-)
<libreofficeru-tg>
<Kompilainenn> для перевода чего и для кого?
<mikekaganski>
попросил коллега для макстора
<libreofficeru-tg>
<Kompilainenn> пиши фатальный сбой, не парься
<mikekaganski>
ок
<libreofficeru-tg>
<Kompilainenn> блин, есть МЕТА (CPU-AT-100%) - [META] Bugs that cause CPU to run at 100%
<libreofficeru-tg>
<Kompilainenn> я в шоке
<libreofficeru-tg>
<Kompilainenn> mikekaganski: математические формулы в ворде 2007 и более новых - это OLE объект ?
<mikekaganski>
нет, там свой язык mathml
<libreofficeru-tg>
<Kompilainenn> я понимаю, что язык свой
<libreofficeru-tg>
<Kompilainenn> но вставлен он в документ не как OLE?
<libreofficeru-tg>
<Kompilainenn> эм
<libreofficeru-tg>
<Kompilainenn> давай я по другом спрошу
<libreofficeru-tg>
<Kompilainenn> это считается объектом?
<libreofficeru-tg>
<Kompilainenn> такми же, как shape
<mikekaganski>
я не знаю, как это считает ворд, но с большой вероятностью подозреваю, что да
VitaliyAndersonI has joined #libreoffice-ru
VitaliyAnderson has quit [Ping timeout: 264 seconds]
tagezi has quit [Ping timeout: 246 seconds]
dshil has joined #libreoffice-ru
<dshil>
mikekaganski: jenkins +1
<mikekaganski>
Спасибо!
<dshil>
и тебе спасибо за оперативное участие!
dshil has quit [Quit: WeeChat 2.2]
dshil has joined #libreoffice-ru
dshil has quit [Ping timeout: 272 seconds]
<libreofficeru-tg>
<Kompilainenn> ыыы, нашел в спаме
<libreofficeru-tg>
<Kompilainenn> по первому впечатлению он излишне здоровый
<libreofficeru-tg>
<Kompilainenn> особенно это заметно по длине листбоксов
<mikekaganski>
ну, ... возможно.
<mikekaganski>
кстати, чекбокс "превью" я бы перетащил вправо возле More Options, и скрывал всю панель по его отключению
<mikekaganski>
и хэлп туда же
<libreofficeru-tg>
<Kompilainenn> нуу, это вотчина Хиско
<mikekaganski>
тож верно
<libreofficeru-tg>
<Kompilainenn> я так понимаю, они взяли готовый мокап и просто его реализовали
<mikekaganski>
но мнение-то никто не мешает озвучить :-)
<libreofficeru-tg>
<Kompilainenn> никто
<libreofficeru-tg>
<Kompilainenn> епта, Норберт опять ноет на @libreoffice
<mikekaganski>
а, вот что я заметил: меняю ориентацию - и в подлежащем документе она тоже прямо тут же менятеся
<mikekaganski>
где?
<mikekaganski>
и чего у него за проблема?
<libreofficeru-tg>
<Kompilainenn> у него проблема, что ЛИбра не развивается так быстро, как ему хочется, имеет много багов и медленно и неправильно работает с чужими форматами
<libreofficeru-tg>
<Kompilainenn> и МСО лучше да
<libreofficeru-tg>
<Kompilainenn> втопку его
<libreofficeru-tg>
<Kompilainenn> он каждый квартал начинает свои унылые завывания
<libreofficeru-tg>
<Kompilainenn> хе, ты во Writer менял ориентацию в диалоге печать?
<libreofficeru-tg>
<Kompilainenn> это реальная бага=)
<mikekaganski>
ага
<libreofficeru-tg>
<Kompilainenn> ладно, я Хиско спросил, чо мы делаем с этим делом, ответит я тогда буду писать баги
<mikekaganski>
ок
<mikekaganski>
ты пока пиши себе заметки
<mikekaganski>
"In Writer, printer page orientation is now synchronized with document orientation. That means that if you select page orientation in print dialog, the document will adapt to the new orientation. There is no need to close the print dialog, go to Format->Page, set the orientation you want to print and then reopen print dialog"
<mikekaganski>
нифига себе фича
<mikekaganski>
это надо будет выжечь
<mikekaganski>
конечно, жаль труд, человек старался, тут надо бы по рукам дать ментору и UX
<libreofficeru-tg>
<Kompilainenn> ай лол, это ФИЧА?