Skip to content
This repository was archived by the owner on May 30, 2026. It is now read-only.

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

Open
bapho-bush wants to merge 1 commit into
Infactum:masterfrom
bapho-bush:master
Open

Поддержка AttachType и SetLanguageCode#15
bapho-bush wants to merge 1 commit into
Infactum:masterfrom
bapho-bush:master

Conversation

@bapho-bush

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

Copy link
Copy Markdown

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

Closes: #12, closes #13

@Infactum

Infactum commented Mar 30, 2023

Copy link
Copy Markdown
Owner

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

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

Copy link
Copy Markdown
Author

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

Done

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

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

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

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

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

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

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

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

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

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

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