This is an archive of the discontinued LLVM Phabricator instance.

Python 2/3 compat - config parser
ClosedPublic

Authored by serge-sans-paille on Dec 3 2018, 2:44 AM.

Reviewers

joerg
michaelplatings

Commits

tools/scan-view/share/ScanView.py
25	except ImportError:

Thanks @michaelplatings for the review o/

LGTM

This revision is now accepted and ready to land.Dec 13 2018, 2:36 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

tools/

scan-view/

share/

7 lines

utils/

check_cfc/

7 lines

Context not available.
	import itertools	import itertools

		michaelplatingsUnsubmitted Done Reply Inline Actions except ImportError: michaelplatings: except ImportError:
	import Reporter	import Reporter
	import ConfigParser	try:
		import configparser
		except ImportError:
		import ConfigParser as configparser

	###	###
	# Various patterns matched or replaced by server.	# Various patterns matched or replaced by server.
Context not available.
	self.load_config()	self.load_config()

	def load_config(self):	def load_config(self):
	self.config = ConfigParser.RawConfigParser()	self.config = configparser.RawConfigParser()

	# Add defaults	# Add defaults
	self.config.add_section('ScanView')	self.config.add_section('ScanView')
Context not available.

Context not available.
	import subprocess	import subprocess
	import sys	import sys
	import tempfile	import tempfile
	import ConfigParser	try:
		import configparser
		except ImportError:
		import ConfigParser as configparser
	import io	import io

	import obj_diff	import obj_diff
Context not available.
	for c in checks:	for c in checks:
	default_config += "{} = false\n".format(c)	default_config += "{} = false\n".format(c)

	config = ConfigParser.RawConfigParser()	config = configparser.RawConfigParser()
	config.readfp(io.BytesIO(default_config))	config.readfp(io.BytesIO(default_config))
	scriptdir = get_main_dir()	scriptdir = get_main_dir()
	config_path = os.path.join(scriptdir, 'check_cfc.cfg')	config_path = os.path.join(scriptdir, 'check_cfc.cfg')
Context not available.