This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	mharbison72	D7761 hgweb: delete a local variable instead of setting to `None`
Closed	mharbison72	D7760 tests: drop unused local variable assignments in linelog
Closed	mharbison72	D7759 ui: delete local variables instead of setting to `None`
Closed	mharbison72	D7758 tests: drop unused local variable assignments in remotefilelog
Closed	mharbison72	D7757 fastannotate: drop unused local variable assignments
Closed	mharbison72	D7756 fsmonitor: drop an unused local variable assignment
Closed	mharbison72	D7755 mq: drop an unused variable assignment
Closed	mharbison72	D7754 hgweb: delete local variable instead of setting it to `None`
Closed	mharbison72	D7753 annotate: avoid using a list comprehension to fill a list with fixed values
Closed	mharbison72	D7752 fsmonitor: drop an unused local variable
Closed	mharbison72	D7751 absorb: avoid using a list comprehension to fill a list with fixed values

Diff 18987

	else:			else:
	raise ValueError(			raise ValueError(
	"unhandled bser opcode 0x%s"			"unhandled bser opcode 0x%s"
	% binascii.hexlify(val_type).decode("ascii")			% binascii.hexlify(val_type).decode("ascii")
	)			)


	def _pdu_info_helper(buf):			def _pdu_info_helper(buf):
	bser_version = -1
	if buf[0:2] == EMPTY_HEADER[0:2]:			if buf[0:2] == EMPTY_HEADER[0:2]:
	bser_version = 1			bser_version = 1
	bser_capabilities = 0			bser_capabilities = 0
	expected_len, pos2 = Bunser.unser_int(buf, 2)			expected_len, pos2 = Bunser.unser_int(buf, 2)
	elif buf[0:2] == EMPTY_HEADER_V2[0:2]:			elif buf[0:2] == EMPTY_HEADER_V2[0:2]:
	if len(buf) < 8:			if len(buf) < 8:
	raise ValueError("Invalid BSER header")			raise ValueError("Invalid BSER header")
	bser_version = 2			bser_version = 2