This is an archive of the discontinued LLVM Phabricator instance.

[include-fixer] Don't run the script if clang-include-fixer binary is not valid.
Needs ReviewPublic

Authored by hokein on May 20 2016, 6:20 AM.

This revision needs review, but there are no reviewers specified.

Summary

Also show a better error message.

hokein retitled this revision from to [include-fixer] Don't run the script if clang-include-fixer binary is not valid..

hokein updated this object.

hokein added a reviewer: ioeric.

Path

Size

include-fixer/

tool/

8 lines

Show All 31 Lines	def main():
parser = argparse.ArgumentParser(		parser = argparse.ArgumentParser(
description='Vim integration for clang-include-fixer')		description='Vim integration for clang-include-fixer')
parser.add_argument('-db', default='yaml',		parser.add_argument('-db', default='yaml',
help='clang-include-fixer input format.')		help='clang-include-fixer input format.')
parser.add_argument('-input', default='',		parser.add_argument('-input', default='',
help='String to initialize the database.')		help='String to initialize the database.')
args = parser.parse_args()		args = parser.parse_args()

		# Check whether the path of clang-include-fixer binary is valid.
		try:
		invocation = [binary, '-help']
		subprocess.check_output(invocation)
		except:
		print >>sys.stderr, "Unable to find clang-include-fixer binary."
		return

# Get the current text.		# Get the current text.
buf = vim.current.buffer		buf = vim.current.buffer
text = '\n'.join(buf)		text = '\n'.join(buf)

# Call clang-include-fixer.		# Call clang-include-fixer.
command = [binary, "-stdin", "-db="+args.db, "-input="+args.input,		command = [binary, "-stdin", "-db="+args.db, "-input="+args.input,
vim.current.buffer.name]		vim.current.buffer.name]
p = subprocess.Popen(command,		p = subprocess.Popen(command,
Show All 17 Lines