Skip to content

Conversation

bapho-bush
Copy link

@bapho-bush bapho-bush commented Mar 28, 2023

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

Closes: #12, closes #13

@Infactum
Copy link
Owner

Infactum commented Mar 30, 2023

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

Removes rarely used headers in windows.h.
@bapho-bush
Copy link
Author

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

Done

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

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

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

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

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

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

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GetAttachedInfo из 8.3.21 SetUserInterfaceLanguageCode из 8.3.21

2 participants