Skip to content

Fix typos, unknown utf8-chars, and capitalize GitHub in docs#2495

Merged
Avasam merged 15 commits intomhammond:mainfrom
Avasam:Fix-typos,-unknown-utf8-chars,-and-capitalize-GitHub-in-docs
Jun 16, 2025
Merged

Fix typos, unknown utf8-chars, and capitalize GitHub in docs#2495
Avasam merged 15 commits intomhammond:mainfrom
Avasam:Fix-typos,-unknown-utf8-chars,-and-capitalize-GitHub-in-docs

Conversation

@Avasam
Copy link
Copy Markdown
Collaborator

@Avasam Avasam commented Mar 15, 2025

No description provided.

Comment thread com/TestSources/PyCOMTest/PyCOMTest.idl Outdated
Comment thread com/TestSources/PyCOMTest/PyCOMTest.idl Outdated
@Avasam
Copy link
Copy Markdown
Collaborator Author

Avasam commented Mar 15, 2025

The usage of ® when mentioning DirectShow, Microsoft or Win32 is very sparse, is it better to just omit it?
image

Comment thread win32/src/win32event.i
Comment on lines -87 to +89
#define EVENT_MODIFY_STATE EVENT_MODIFY_STATE // Enables use of the event handle in the SetEvent and ResetEventfunctions to modify the events state.
#define EVENT_MODIFY_STATE EVENT_MODIFY_STATE // Enables use of the event handle in the SetEvent and ResetEvent functions to modify the event's state.

#define SYNCHRONIZE SYNCHRONIZE // Windows NT only:Enables use of the event handle in any of the wait functionsto wait for the events state to be signaled.
#define SYNCHRONIZE SYNCHRONIZE // Windows NT only: Enables use of the event handle in any of the wait functions to wait for the event's state to be signaled.
Copy link
Copy Markdown
Collaborator Author

@Avasam Avasam Mar 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't figure out what the other were supposed to be here. Probably special spaces.

@Avasam Avasam force-pushed the Fix-typos,-unknown-utf8-chars,-and-capitalize-GitHub-in-docs branch from ed57c12 to 320fce4 Compare April 29, 2025 22:57
@Avasam Avasam merged commit cfd3d86 into mhammond:main Jun 16, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants