Skip to content

[Issue #390]added the client._generate._get_known_symbols #391

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Dec 7, 2022
Merged

[Issue #390]added the client._generate._get_known_symbols #391

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
81b3d18
added the client._generate._get_known_symbols
muddi900 Dec 3, 2022
860332f
fixed the autopep formating from ide and made typing anotiations coma…
muddi900 Dec 5, 2022
df02c66
reverted gitignore as per request and added two space for comments
muddi900 Dec 5, 2022
bd27115
removed gitignore
muddi900 Dec 5, 2022
a4e1c50
removed custom location
muddi900 Dec 6, 2022
6dd9915
fixed weird spacing issue
muddi900 Dec 6, 2022
898b542
autopep format removal
muddi900 Dec 6, 2022
c85fcac
spacing issue fixed again
muddi900 Dec 6, 2022
5a81cab
added space for generics
muddi900 Dec 7, 2022
450bc19
.vscode removed
muddi900 Dec 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,3 +51,7 @@ coverage.xml
docs/_build/


custom location
.vscode


10 changes: 6 additions & 4 deletions comtypes/client/_generate.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
from comtypes.tools import codegenerator, tlbparser

if TYPE_CHECKING:
from typing import Any, Tuple, List, Optional, Union as _UnionT
from typing import Any, Tuple, List, Optional, Dict, Union as _UnionT
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍



logger = logging.getLogger(__name__)
Expand Down Expand Up @@ -247,8 +247,9 @@ def _create_wrapper_module(tlib, pathname):


def _get_known_symbols():
# type: () -> dict[str, str]
known_symbols = {} # type: dict[str, str]
# type: () -> Dict[str,str]

known_symbols = {} # type: Dict[str, str]
for mod_name in (
"comtypes.persist",
"comtypes.typeinfo",
Expand All @@ -259,7 +260,7 @@ def _get_known_symbols():
):
mod = importlib.import_module(mod_name)
if hasattr(mod, "__known_symbols__"):
names = mod.__known_symbols__ # type: list[str]
names = mod.__known_symbols__ # type: List[str]
else:
names = list(mod.__dict__)
for name in names:
Expand All @@ -268,6 +269,7 @@ def _get_known_symbols():

################################################################


if __name__ == "__main__":
# When started as script, generate typelib wrapper from .tlb file.
GetModule(sys.argv[1])