Skip to content

Commit

Permalink
Merge pull request #9182 from tk0miya/refactor_autodoc
Browse files Browse the repository at this point in the history
refactor: autodoc: Remove backslashes from the tail of line
  • Loading branch information
tk0miya committed May 9, 2021
2 parents 98c756b + 80fb2c0 commit b88187d
Showing 1 changed file with 15 additions and 16 deletions.
31 changes: 15 additions & 16 deletions sphinx/ext/autodoc/__init__.py
Expand Up @@ -389,8 +389,8 @@ def parse_name(self) -> bool:
# functions can contain a signature which is then used instead of
# an autogenerated one
try:
explicit_modname, path, base, args, retann = \
py_ext_sig_re.match(self.name).groups()
matched = py_ext_sig_re.match(self.name)
explicit_modname, path, base, args, retann = matched.groups()
except AttributeError:
logger.warning(__('invalid signature for auto%s (%r)') % (self.objtype, self.name),
type='autodoc')
Expand All @@ -412,8 +412,8 @@ def parse_name(self) -> bool:

self.args = args
self.retann = retann
self.fullname = (self.modname or '') + \
('.' + '.'.join(self.objpath) if self.objpath else '')
self.fullname = ((self.modname or '') +
('.' + '.'.join(self.objpath) if self.objpath else ''))
return True

def import_object(self, raiseerror: bool = False) -> bool:
Expand Down Expand Up @@ -825,8 +825,9 @@ def document_members(self, all_members: bool = False) -> None:
if self.objpath:
self.env.temp_data['autodoc:class'] = self.objpath[0]

want_all = all_members or self.options.inherited_members or \
self.options.members is ALL
want_all = (all_members or
self.options.inherited_members or
self.options.members is ALL)
# find out which members are documentable
members_check_module, members = self.get_object_members(want_all)

Expand All @@ -842,8 +843,7 @@ def document_members(self, all_members: bool = False) -> None:
classes.sort(key=lambda cls: cls.priority)
# give explicitly separated module name, so that members
# of inner classes can be documented
full_mname = self.modname + '::' + \
'.'.join(self.objpath + [mname])
full_mname = self.modname + '::' + '.'.join(self.objpath + [mname])
documenter = classes[-1](self.directive, full_mname, self.indent)
memberdocumenters.append((documenter, isattr))

Expand Down Expand Up @@ -1968,8 +1968,8 @@ def document_members(self, all_members: bool = False) -> None:
pass

def get_real_modname(self) -> str:
return self.get_attr(self.parent or self.object, '__module__', None) \
or self.modname
real_modname = self.get_attr(self.parent or self.object, '__module__', None)
return real_modname or self.modname

def get_module_comment(self, attrname: str) -> Optional[List[str]]:
try:
Expand Down Expand Up @@ -2034,8 +2034,7 @@ class MethodDocumenter(DocstringSignatureMixin, ClassLevelDocumenter): # type:
@classmethod
def can_document_member(cls, member: Any, membername: str, isattr: bool, parent: Any
) -> bool:
return inspect.isroutine(member) and \
not isinstance(parent, ModuleDocumenter)
return inspect.isroutine(member) and not isinstance(parent, ModuleDocumenter)

def import_object(self, raiseerror: bool = False) -> bool:
ret = super().import_object(raiseerror)
Expand Down Expand Up @@ -2470,8 +2469,8 @@ def import_object(self, raiseerror: bool = False) -> bool:
return ret

def get_real_modname(self) -> str:
return self.get_attr(self.parent or self.object, '__module__', None) \
or self.modname
real_modname = self.get_attr(self.parent or self.object, '__module__', None)
return real_modname or self.modname

def should_suppress_value_header(self) -> bool:
if super().should_suppress_value_header():
Expand Down Expand Up @@ -2572,8 +2571,8 @@ def document_members(self, all_members: bool = False) -> None:
pass

def get_real_modname(self) -> str:
return self.get_attr(self.parent or self.object, '__module__', None) \
or self.modname
real_modname = self.get_attr(self.parent or self.object, '__module__', None)
return real_modname or self.modname

def add_directive_header(self, sig: str) -> None:
super().add_directive_header(sig)
Expand Down

0 comments on commit b88187d

Please sign in to comment.