Use list literals

This commit is contained in:
Adam Turner 2025-01-03 20:56:48 +00:00
parent 546dd50c41
commit 20c1c7fc1c
5 changed files with 27 additions and 49 deletions

View File

@ -708,8 +708,7 @@ class EpubBuilder(StandaloneHTMLBuilder):
Subelements of a node are nested inside the navpoint. For nested nodes Subelements of a node are nested inside the navpoint. For nested nodes
the parent node is reinserted in the subnav. the parent node is reinserted in the subnav.
""" """
navstack: list[NavPoint] = [] navstack: list[NavPoint] = [NavPoint('dummy', 0, '', '', [])]
navstack.append(NavPoint('dummy', 0, '', '', []))
level = 0 level = 0
lastnode = None lastnode = None
for node in nodes: for node in nodes:

View File

@ -141,8 +141,7 @@ class Epub3Builder(_epub_base.EpubBuilder):
The difference from build_navpoints method is templates which are used The difference from build_navpoints method is templates which are used
when generating navigation documents. when generating navigation documents.
""" """
navstack: list[NavPoint] = [] navstack: list[NavPoint] = [NavPoint('', '', [])]
navstack.append(NavPoint('', '', []))
level = 0 level = 0
for node in navnodes: for node in navnodes:
if not node['text']: if not node['text']:

View File

@ -657,8 +657,7 @@ class ASTBinOpExpr(ASTBase):
return hash((self.exprs, self.ops)) return hash((self.exprs, self.ops))
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.exprs[0])]
res.append(transform(self.exprs[0]))
for i in range(1, len(self.exprs)): for i in range(1, len(self.exprs)):
res.extend(( res.extend((
' ', ' ',
@ -699,8 +698,7 @@ class ASTAssignmentExpr(ASTExpression):
return hash((self.exprs, self.ops)) return hash((self.exprs, self.ops))
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.exprs[0])]
res.append(transform(self.exprs[0]))
for i in range(1, len(self.exprs)): for i in range(1, len(self.exprs)):
res.extend(( res.extend((
' ', ' ',
@ -872,8 +870,7 @@ class ASTParameters(ASTBase):
return self.args return self.args
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = ['(']
res.append('(')
first = True first = True
for a in self.args: for a in self.args:
if not first: if not first:
@ -1345,8 +1342,7 @@ class ASTDeclaratorPtr(ASTDeclarator):
) )
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = ['*'] res = ['*', transform(self.attrs)]
res.append(transform(self.attrs))
if len(self.attrs) != 0 and (self.restrict or self.volatile or self.const): if len(self.attrs) != 0 and (self.restrict or self.volatile or self.const):
res.append(' ') res.append(' ')
if self.restrict: if self.restrict:
@ -1620,8 +1616,7 @@ class ASTTypeWithInit(ASTBase):
return self.type.get_id(version, objectType, symbol) return self.type.get_id(version, objectType, symbol)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.type)]
res.append(transform(self.type))
if self.init: if self.init:
res.append(transform(self.init)) res.append(transform(self.init))
return ''.join(res) return ''.join(res)
@ -1699,8 +1694,7 @@ class ASTMacro(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.ident)]
res.append(transform(self.ident))
if self.args is not None: if self.args is not None:
res.append('(') res.append('(')
first = True first = True
@ -1826,8 +1820,7 @@ class ASTEnumerator(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.name)]
res.append(transform(self.name))
if len(self.attrs) != 0: if len(self.attrs) != 0:
res.extend((' ', transform(self.attrs))) res.extend((' ', transform(self.attrs)))
if self.init: if self.init:

View File

@ -1314,8 +1314,7 @@ class ASTBinOpExpr(ASTExpression):
return hash((self.exprs, self.ops)) return hash((self.exprs, self.ops))
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.exprs[0])]
res.append(transform(self.exprs[0]))
for i in range(1, len(self.exprs)): for i in range(1, len(self.exprs)):
res.extend(( res.extend((
' ', ' ',
@ -1804,8 +1803,7 @@ class ASTTemplateArgs(ASTBase):
)) ))
return ''.join(res) return ''.join(res)
res = [] res = ['I']
res.append('I')
if len(self.args) > 0: if len(self.args) > 0:
for a in self.args[:-1]: for a in self.args[:-1]:
res.append(a.get_id(version)) res.append(a.get_id(version))
@ -2172,8 +2170,7 @@ class ASTParametersQualifiers(ASTBase):
return ''.join(a.get_id(version) for a in self.args) return ''.join(a.get_id(version) for a in self.args)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = ['(']
res.append('(')
first = True first = True
for a in self.args: for a in self.args:
if not first: if not first:
@ -2485,8 +2482,7 @@ class ASTDeclSpecs(ASTBase):
def get_id(self, version: int) -> str: def get_id(self, version: int) -> str:
if version == 1: if version == 1:
res = [] res = [self.trailingTypeSpec.get_id(version)]
res.append(self.trailingTypeSpec.get_id(version))
if self.allSpecs.volatile: if self.allSpecs.volatile:
res.append('V') res.append('V')
if self.allSpecs.const: if self.allSpecs.const:
@ -2689,9 +2685,8 @@ class ASTDeclaratorNameParamQual(ASTDeclarator):
def get_type_id(self, version: int, returnTypeId: str) -> str: def get_type_id(self, version: int, returnTypeId: str) -> str:
assert version >= 2 assert version >= 2
res = [] res = [self.get_ptr_suffix_id(version)]
# TODO: can we actually have both array ops and paramQual? # TODO: can we actually have both array ops and paramQual?
res.append(self.get_ptr_suffix_id(version))
if self.paramQual: if self.paramQual:
res.extend(( res.extend((
self.get_modifiers_id(version), self.get_modifiers_id(version),
@ -2834,8 +2829,7 @@ class ASTDeclaratorPtr(ASTDeclarator):
return self.next.require_space_after_declSpecs() return self.next.require_space_after_declSpecs()
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = ['*'] res = ['*', transform(self.attrs)]
res.append(transform(self.attrs))
if len(self.attrs) != 0 and (self.volatile or self.const): if len(self.attrs) != 0 and (self.volatile or self.const):
res.append(' ') res.append(' ')
if self.volatile: if self.volatile:
@ -2866,8 +2860,7 @@ class ASTDeclaratorPtr(ASTDeclarator):
res.append(self.next.get_ptr_suffix_id(version)) res.append(self.next.get_ptr_suffix_id(version))
return ''.join(res) return ''.join(res)
res = [self.next.get_ptr_suffix_id(version)] res = [self.next.get_ptr_suffix_id(version), 'P']
res.append('P')
if self.volatile: if self.volatile:
res.append('V') res.append('V')
if self.const: if self.const:
@ -2949,8 +2942,7 @@ class ASTDeclaratorRef(ASTDeclarator):
return self.next.require_space_after_declSpecs() return self.next.require_space_after_declSpecs()
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = ['&'] res = ['&', transform(self.attrs)]
res.append(transform(self.attrs))
if len(self.attrs) != 0 and self.next.require_space_after_declSpecs(): if len(self.attrs) != 0 and self.next.require_space_after_declSpecs():
res.append(' ') res.append(' ')
res.append(transform(self.next)) res.append(transform(self.next))
@ -3565,8 +3557,7 @@ class ASTTypeWithInit(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.type)]
res.append(transform(self.type))
if self.init: if self.init:
res.append(transform(self.init)) res.append(transform(self.init))
return ''.join(res) return ''.join(res)
@ -3601,8 +3592,7 @@ class ASTTypeUsing(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.name)]
res.append(transform(self.name))
if self.type: if self.type:
res.extend((' = ', transform(self.type))) res.extend((' = ', transform(self.type)))
return ''.join(res) return ''.join(res)
@ -3745,8 +3735,7 @@ class ASTClass(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.attrs)]
res.append(transform(self.attrs))
if len(self.attrs) != 0: if len(self.attrs) != 0:
res.append(' ') res.append(' ')
res.append(transform(self.name)) res.append(transform(self.name))
@ -3804,8 +3793,7 @@ class ASTUnion(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.attrs)]
res.append(transform(self.attrs))
if len(self.attrs) != 0: if len(self.attrs) != 0:
res.append(' ') res.append(' ')
res.append(transform(self.name)) res.append(transform(self.name))
@ -3908,8 +3896,7 @@ class ASTEnumerator(ASTBase):
return symbol.get_full_nested_name().get_id(version) return symbol.get_full_nested_name().get_id(version)
def _stringify(self, transform: StringifyTransform) -> str: def _stringify(self, transform: StringifyTransform) -> str:
res = [] res = [transform(self.name)]
res.append(transform(self.name))
if len(self.attrs) != 0: if len(self.attrs) != 0:
res.extend((' ', transform(self.attrs))) res.extend((' ', transform(self.attrs)))
if self.init: if self.init:
@ -4211,8 +4198,7 @@ class ASTTemplateParams(ASTBase):
def get_id(self, version: int, excludeRequires: bool = False) -> str: def get_id(self, version: int, excludeRequires: bool = False) -> str:
assert version >= 2 assert version >= 2
res = [] res = ['I']
res.append('I')
res.extend(param.get_id(version) for param in self.params) res.extend(param.get_id(version) for param in self.params)
res.append('E') res.append('E')
if not excludeRequires and self.requiresClause: if not excludeRequires and self.requiresClause:

View File

@ -495,9 +495,10 @@ class DefinitionParser(BaseParser):
self.pos = pos self.pos = pos
header = 'Error in postfix expression,' header = 'Error in postfix expression,'
header += ' expected primary expression or type.' header += ' expected primary expression or type.'
errors = [] errors = [
errors.append((e_outer, 'If primary expression')) (e_outer, 'If primary expression'),
errors.append((e_inner, 'If type')) (e_inner, 'If type'),
]
raise self._make_multi_error(errors, header) from e_inner raise self._make_multi_error(errors, header) from e_inner
# and now parse postfixes # and now parse postfixes