minor style improvements

dependencies.py:
- In a comment, rename "imported modules" to "linked modules"
- Check against "macro" only in one loop, not in both

__main__.py: avoid an odd use of any() where regular `or` does the
trick.

ctypesdescs: avoid unnecessary explicit True/False for statement that
results in a bool already

ctypesgen/__main__.py

@@ -325,7 +325,7 @@ def __call__(self, parser, namespace, values, option_string=None):
 
     args = parser.parse_args(given_argv)
 
-    if not any([args.headers, args.system_headers]):
+    if not (args.headers or args.system_headers):
         raise argparse.ArgumentError("Either --headers or --system-headers required.")
 
     if args.cpp:

ctypesgen/ctypedescs.py

@@ -298,7 +298,7 @@ def __init__(self, tag, attrib, variety, members, src=None):
         self.attrib = attrib
         self.variety = variety  # "struct" or "union"
         self.members = members
-        self.opaque = True if self.members is None else False
+        self.opaque = self.members is None
         self.src = src
 
         if type(self.tag) == int or not self.tag:

ctypesgen/processor/dependencies.py

@@ -18,7 +18,7 @@ def find_dependencies(data, opts):
     typedef_names = {}
     ident_names = {}
 
-    # Start the lookup tables with names from imported modules
+    # Start the lookup tables with names from linked modules
 
     for name in opts.linked_symbols:
         typedef_names[name] = None
@@ -155,12 +155,11 @@ def add_to_lookup_table(desc, kind):
     # Macros are handled differently from everything else because macros can
     # call other macros that are referenced after them in the input file, but
     # no other type of description can look ahead like that.
-
+
+    macros = []
     for kind, desc in data.output_order:
         add_to_lookup_table(desc, kind)
-        if kind != "macro":
-            find_dependencies_for(desc, kind)
+        find_dependencies_for(desc, kind) if kind != "macro" else macros.append(desc)
 
-    for kind, desc in data.output_order:
-        if kind == "macro":
-            find_dependencies_for(desc, kind)
+    for desc in macros:
+        find_dependencies_for(desc, "macro")

tests/conftest.py

@@ -51,7 +51,7 @@ def generate(header, args=[], lang="py", sys_header=False):
     # - The default file encoding seems to be cp1252, which is problematic with special chars (such as the banana in the constants test). Need to specify UTF-8 explicitly. PEP 686 should hopefully improve this.
 
     # Use custom tempfiles scoping so we may retain data for inspection
-    # FIXME can cause confusion with partial test suite runs - static naming by test case would be better, also more descriptive
+    # FIXME can cause confusion with partial test suite runs - static naming by test case would be better, including more descriptive
     global COUNTER; COUNTER += 1
 
     cmdargs = []
@@ -85,9 +85,6 @@ def generate(header, args=[], lang="py", sys_header=False):
         assert False
 
 
-# -- Functions facilitating tests of use of cross inclusion --
-
-
 def generate_common():
     common_lib = "libcommon.dll" if sys.platform == "win32" else "libcommon.so"
     _create_common_files()