This is an archive of the discontinued LLVM Phabricator instance.

Paths

[lit] Remove unused variable in googletest format
ClosedPublic

Authored by modocache on Oct 9 2016, 1:51 PM.

Reviewers

Commits

Summary

prefix is written to but never read.

Build Status

Buildable 293
Build 293: arc lint + arc unit

modocache retitled this revision from to [lit] Remove unused variable in googletest format.

modocache updated this object.

modocache added a subscriber: llvm-commits.

• ddunbar edited edge metadata.

This revision is now accepted and ready to land.Oct 9 2016, 5:03 PM

Path

Size

utils/

lit/

formats/

1 line

Show First 20 Lines • Show All 46 Lines • ▼ Show 20 Lines	def getGTestTests(self, path, litConfig, localConfig):
continue		continue

if 'Running main() from gtest_main.cc' in ln:		if 'Running main() from gtest_main.cc' in ln:
# Upstream googletest prints this to stdout prior to running		# Upstream googletest prints this to stdout prior to running
# tests. LLVM removed that print statement in r61540, but we		# tests. LLVM removed that print statement in r61540, but we
# handle it here in case upstream googletest is being used.		# handle it here in case upstream googletest is being used.
continue		continue

prefix = ''
index = 0		index = 0
while ln[index2:index2+2] == ' ':		while ln[index2:index2+2] == ' ':
index += 1		index += 1
while len(nested_tests) > index:		while len(nested_tests) > index:
nested_tests.pop()		nested_tests.pop()

ln = ln[index*2:]		ln = ln[index*2:]
if ln.endswith('.'):		if ln.endswith('.'):
▲ Show 20 Lines • Show All 77 Lines • Show Last 20 Lines