This is an archive of the discontinued Mercurial Phabricator instance.

Differential D10215

typing: rewrite a conditional assignment to unconfuse pytype
ClosedPublic

Authored by mharbison72 on Mar 13 2021, 2:34 PM.

Download Raw Diff

Details

Reviewers

pulkit

Group Reviewers

hg-reviewers

Commits

rHG86b47ec1960a: typing: rewrite a conditional assignment to unconfuse pytype

Summary

Otherwise, pytype and PyCharm flags it:

File "/mnt/c/Users/Matt/hg/mercurial/localrepo.py", line 2903, in wlock: No attribute 'held' on _weakref.ReferenceType[nothing] [attribute-error]
  In Optional[Union[Any, _weakref.ReferenceType[nothing]]]
File "/mnt/c/Users/Matt/hg/mercurial/localrepo.py", line 2904, in wlock: No attribute 'lock' on _weakref.ReferenceType[nothing] [attribute-error]
  In Optional[Union[Any, _weakref.ReferenceType[nothing]]]

Diff Detail

Repository

rHG Mercurial

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

mharbison72 created this revision.Mar 13 2021, 2:34 PM

Herald added a reviewer: hg-reviewers. · View Herald TranscriptMar 13 2021, 2:34 PM

Herald added a subscriber: mercurial-patches. · View Herald Transcript

pulkit accepted this revision.Mar 18 2021, 3:00 PM

This revision is now accepted and ready to land.Mar 18 2021, 3:00 PM

mharbison72 added a commit: rHG86b47ec1960a: typing: rewrite a conditional assignment to unconfuse pytype.Mar 18 2021, 5:06 PM

Closed by commit rHG86b47ec1960a: typing: rewrite a conditional assignment to unconfuse pytype (authored by mharbison72). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			mercurial/localrepo.py (2 lines)

Status	Author	Revision
Closed	mharbison72	D10215 typing: rewrite a conditional assignment to unconfuse pytype
Closed	mharbison72	D10214 typing: add assertions to localrepo.py to appease pytype
Closed	mharbison72	D10213 obsutil: maintain a homogenous list when computing successors
Closed	mharbison72	D10212 typing: add an assertion to verify.py to appease pytype
Closed	mharbison72	D10211 typing: add an assertion to mercurial/hgweb/webcommands.py to help pytype
Closed	mharbison72	D10210 crecord: null out the curses attribute with `None` on failure to import
Closed	mharbison72	D10209 typing: fix directives mangled by black
Closed	mharbison72	D10208 typing: disable module attribute warnings for properly conditionalized code
Closed	mharbison72	D10207 typing: disable import error warnings that are already handled
Closed	mharbison72	D10206 typing: disable a few errors when accessing Windows specific attributes
Closed	mharbison72	D10205 typing: disable a few errors calling py3.7+ functions in resourceutil.py
Closed	mharbison72	D10204 wsgicgi: switch the default PATH_INFO back to str
Closed	mharbison72	D10203 crecord: convert an instance of bytes to str
Closed	mharbison72	D10202 crecord: change str literals to byte literals
Closed	mharbison72	D10201 bisect: replace a missing method on the list primitive
Closed	mharbison72	D10183 typing: disable a module-attr warning in the worker module's py2 code
Closed	mharbison72	D10182 wireprotoserver: convert ErrorResponse to bytes
Closed	mharbison72	D10181 verify: convert an exception to bytes before logging
Closed	mharbison72	D10180 typing: add an assertion to the upgrade engine to help pytype
Closed	mharbison72	D10179 subrepo: handle unexpected file types from git gracefully
Closed	mharbison72	D10178 patch: convert a UI message to bytes when editing a patch
Closed	mharbison72	D10177 merge: force an exception message to bytes before printing as a warning
Closed	mharbison72	D10176 hg: convert an exception to bytes in the repo creation exception handler
Closed	mharbison72	D10175 typing: add an assertion instead of blacklisting mercurial/extensions.py
Closed	mharbison72	D10174 debug: convert a few exceptions to bytes before wrapping in another error
Closed	mharbison72	D10173 grep: convert an exception to bytes for a warning message
Closed	mharbison72	D10172 morestatus: convert a UI message about merge conflicts to bytes
Closed	mharbison72	D10171 changegroup: convert a warning message to bytes
Closed	mharbison72	D10170 branchmap: force Exception to bytes before logging
Closed	mharbison72	D10169 nodemap: convert error message to bytes
Closed	mharbison72	D10168 mail: convert SMTPException to bytes before passing to error.Abort()
Closed	mharbison72	D10167 typing: switch an argument type to the generic form
Closed	mharbison72	D10166 typing: ensure that error.Abort is given bytes
Closed	mharbison72	D10165 typing: fix a suppression directive that was mangled by black formatting

Diff 26508

mercurial/localrepo.py

	def wlock(self, wait=True):			def wlock(self, wait=True):
	"""Lock the non-store parts of the repository (everything under			"""Lock the non-store parts of the repository (everything under
	.hg except .hg/store) and return a weak reference to the lock.			.hg except .hg/store) and return a weak reference to the lock.

	Use this before modifying files in .hg.			Use this before modifying files in .hg.

	If both 'lock' and 'wlock' must be acquired, ensure you always acquires			If both 'lock' and 'wlock' must be acquired, ensure you always acquires
	'wlock' first to avoid a dead-lock hazard."""			'wlock' first to avoid a dead-lock hazard."""
	l = self._wlockref and self._wlockref()			l = self._wlockref() if self._wlockref else None
	if l is not None and l.held:			if l is not None and l.held:
	l.lock()			l.lock()
	return l			return l

	# We do not need to check for non-waiting lock acquisition. Such			# We do not need to check for non-waiting lock acquisition. Such
	# acquisition would not cause dead-lock as they would just fail.			# acquisition would not cause dead-lock as they would just fail.
	if wait and (			if wait and (
	self.ui.configbool(b'devel', b'all-warnings')			self.ui.configbool(b'devel', b'all-warnings')

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	26329		Mar 13 2021, 2:34 PM	★	★
Diff 2	26508	rHG86b47ec1960abd22143665bcd84f5d28390f70aa	Mar 13 2021, 2:09 AM	★	★