Я не вижу никаких побочных проблем такого ново-введения, а удобство повышает значительно.
Собственно вот. Мож пригодится.
Не знаю куда сунуть: предложение добавить функционал
Posted: Wed 09 Dec, 2015 01:16
by JouriM
ПС, Забыл совсем самое главное
Огромная благодарность за код и комплект, который мало того что собирается "из коробки", так еще и работает потом
По текущим временам - это почти на подвиг похоже
Обычно даже в "модных" проектах приходится сутками выпиливать из чугунного куба хрустальный шар...
А тут все собралось (ну кроме завязки на вижуал в syslog.hpp-е, но это мелочь), за час все добавил, отладил и оно работает
Спасибо!
Не знаю куда сунуть: предложение добавить функционал
Posted: Wed 09 Dec, 2015 01:54
by zg
зачем? за день на основе d_usercontrol пишется реализация с куда большим функционалом в плане раскраски.
Не знаю куда сунуть: предложение добавить функционал
Posted: Wed 09 Dec, 2015 02:59
by JouriM
Затем чтобы не терять день ВСЕМ
Тем более что большинству писать что-то сложнее 2х строк влом
Будет готовое - будут пользовать
Не знаю куда сунуть: предложение добавить функционал
Posted: Wed 09 Dec, 2015 11:07
by Shmuel
JouriM wrote: Не знаю куда сунуть: предложение добавить функционал
На багтрекере есть раздел Wishes. Там заводят новый тикет и прикладывают код (если есть) в виде патча.
Не знаю куда сунуть: предложение добавить функционал
Не знаю куда сунуть: предложение добавить функционал
Posted: Wed 09 Dec, 2015 16:16
by DrKnS
Попридираюсь немного.
JouriM wrote:позволит из текущих плагинов (любых)
Текущим это не особо поможет, т. к. управляющие символы сами себя не добавят. static bool NullToEmptyHilight( MenuItemEx& mi, const wchar_t *s )
NullToEmpty нужна для защиты от нулевого указателя (к сожалению, в некоторых местах API он трактуется как пустая строка). Давайте не смешивать мух с котлетами. rc.reserve( wcslen(s)+5 );
Что такое 5 и зачем резервировать больше, чем в исходной строке, если больше там не будет? if ( *s == L'\r' || *s == L'\n' )
Выбор переносов строк в качестве управляющих символов для подсветки выглядит, мягко говоря, неожиданно. if ( rc.length() >= rc.max_size() ) rc.reserve( rc.length()+10 );
Такой reserve может превратить экспоненциальное количество реаллокаций в линейное. К счастью, он никогда не вызовется, т. к. условие невыполнимо.
Ну и как уже подмечено, лучше диффы на багтрекер.
Предложение добавить функционал подсветки частей строк в меню
Posted: Wed 09 Dec, 2015 16:21
by HaRT
DrKnS wrote: превратить экспоненциальное количество реаллокаций в линейное
Может всё же логарифмическое в линейное?
Предложение добавить функционал подсветки частей строк в меню
Posted: Wed 09 Dec, 2015 16:48
by DrKnS
HaRT wrote: Может всё же логарифмическое в линейное?
Конечно логарифмическое, это рост экспоненциальный.
Благодарю за бдительность.
Предложение добавить функционал подсветки частей строк в меню
Ты серьезно считаешь что кому-то будет проще вынуть какой-то недокументированный класс на С, собрать его и воткнуть.... куда, в скрипт на луа из 10 строк?
Писание плагина на С это несравнимо более титаническая работа в сравнению с втыканием каких-то спецсимволов в строчку скрипта
Я думал что это самоочевидно
Ну и такая мелочь, например, как то что текущий plugin.hpp не соберется ничем кроме гнуси и силанговых наследников
ПС: Мне, в сущности, плевать
Препирания о видении мира художником мне надоели еще 10 лет назад (или когда там это было)
Сунул сюда а не молча исправил 10 строк в локальном ФАРе только потому, что не думал что вообще какие-то проблемы возникнут из-за 100 байт и 10 строк которые абсолютно ничему не мешают.
Воткнете в ФАР - хорошо, значит не зря потратит на писанину сюда больше чем на исправление кода
Не воткнете - не расстроюсь
Предложение добавить функционал подсветки частей строк в меню
Posted: Wed 09 Dec, 2015 17:34
by JouriM
DrKnS wrote:Такой reserve может превратить экспоненциальное количество реаллокаций в линейное
Я именно сюда и кинул т.к. vector<> я вчера написал впервые в жизни, а чтобы узнать методы string их пришлось гуглить...
Это функциональная болванка
Если она написана криво то исправить ее дело минуты
Предложение добавить функционал подсветки частей строк в меню
Posted: Wed 09 Dec, 2015 17:42
by HaRT
JouriM wrote: Если она написана криво то исправить ее дело минуты
Так за чем же дело стало?
Вам написали, что там не так, — казалось бы, берёте и за минуту исправляете. Или Вы ожидали, что это кто-то будет делать за Вас?
Предложение добавить функционал подсветки частей строк в меню
Posted: Wed 09 Dec, 2015 23:16
by JouriM
HaRT wrote:
JouriM wrote: Если она написана криво то исправить ее дело минуты
Так за чем же дело стало?
Вам написали, что там не так, — казалось бы, берёте и за минуту исправляете. Или Вы ожидали, что это кто-то будет делать за Вас?
Ты чего-то с чем-то перепутал. МНЕ ничего за меня делать не нужно. Я за себя уже все сделал.
Я предложил вариант для всех. РАБОЧИЙ. Если он нарушает чьето чувство прекрасного - то я тут при чем?