Python binding: converging 3 opt. args into list of flags

python/dbgtest.py

@@ -1,6 +1,6 @@
 import r2pipe
 
-r2=r2pipe.open("/bin/ls", False, False, True)
+r2=r2pipe.open("/bin/ls", ["-nd"])
 for a in range(1,10):
 	regs = r2.cmdj("drj")
 	print "0x%x  0x%x"%(regs['rip'],regs['rsp'])

python/r2pipe/__init__.py

@@ -68,7 +68,7 @@ def version():
 class open:
 	"""Class representing an r2pipe connection with a running radare2 instance
 	"""
-	def __init__(self, filename='', writeable=False, bininfo=True, debug=False,):
+	def __init__(self, filename='', flags=[]):
 		"""Open a new r2 pipe
 		The 'filename' can be one of the following:
 
@@ -79,9 +79,8 @@ def __init__(self, filename='', writeable=False, bininfo=True, debug=False,):
 
 		Args:
 			filename (str): path to filename or uri
-			writeable (bool): if True opens the file in read-write
-			bininfo (bool): if True loads the info from the binary
-			debug (bool): if True opens in debugger (r2 -d)
+			flags (list of str): arguments, either in comapct form
+				("-wdn") or sepparated by commas ("-w","-d","-n")
 		Returns:
 			Returns an object with methods to interact with r2 via commands
 		"""
@@ -129,12 +128,7 @@ def __init__(self, filename='', writeable=False, bininfo=True, debug=False,):
 		else:
 			self._cmd = self._cmd_process
 			cmd = ["radare2", "-q0", filename]
-			if writeable:
-				cmd = cmd[:1] + ["-w"] + cmd[1:]
-			if debug:
-				cmd = cmd[:1] + ["-d"] + cmd[1:]
-			if not bininfo:
-				cmd = cmd[:1] + ["-n"] + cmd[1:]
+			cmd = cmd[:1] + flags + cmd[1:]
 			self.process = Popen(cmd, shell=False, stdin=PIPE, stdout=PIPE)
 			self.process.stdout.read(1) # Reads initial \x00