addin-template icon indicating copy to clipboard operation
addin-template copied to clipboard

Поддержка AttachType и SetLanguageCode

Open bapho-bush opened this issue 1 year ago • 2 comments

Плюс добавление опции, чтоб исключить редко используемые заголовки из windows.h

Closes: #12, closes #13

bapho-bush avatar Mar 28 '23 11:03 bapho-bush

Лишние изменения из дифа не плохо бы убрать для начала. Именно в шаблон WIN32_LEAN_AND_MEAN кажется лишним. Собраться же можно и без этого дефайна. Кому надо станет - сам выставит. Зачем способ подключения компоненты выносить на уровень cmake? Выбрать какой-то вариант по умолчанию и разрешить переопределять. С учетом того, что их всего два - я бы просто булевый флаг сделал "изолировано / нет". Насчет SetLanguageCode - внимательно не смотрел, как это изменение сочетается с тем, что компонента может подключаться как к "старой" платформе, так и к новой.

Infactum avatar Mar 30 '23 07:03 Infactum

Лишние изменения из дифа не плохо бы убрать для начала.

Done

Именно в шаблон WIN32_LEAN_AND_MEAN кажется лишним. Собраться же можно и без этого дефайна. Кому надо станет - сам выставит.

А мне кажется не лишнее. Штука, кмк, довольно неочевидная. И влияет лишь на хедеры, которые редко используются. Классическое плюсовое "не используй то, что тебе не нужно".

Зачем способ подключения компоненты выносить на уровень cmake? Выбрать какой-то вариант по умолчанию и разрешить переопределять. С учетом того, что их всего два - я бы просто булевый флаг сделал "изолировано / нет".

Три ведь. Я вариант eCanAttachIsolated понял как "подключается только изолированно".

Допустимые значения:

eCanAttachNotIsolated = 1 - подключение к процессу платформы, eCanAttachIsolated = 2 - подключение к отдельному хост-процессу, eCanAttachAny = 3 - любое подключение.

В целом, на уровне шаблона просто потому что захотелось так. Показалось, что это удобнее.

Насчет SetLanguageCode - внимательно не смотрел, как это изменение сочетается с тем, что компонента может подключаться как к "старой" платформе, так и к новой.

Есть мысли как потестить?

bapho-bush avatar Jan 05 '24 23:01 bapho-bush