1 files changed, 49 insertions, 18 deletions

diff --git a/lldb/scripts/verify_api.py b/lldb/scripts/verify_api.py
index e636cdda649..630247d9331 100755
--- a/lldb/scripts/verify_api.py
+++ b/lldb/scripts/verify_api.py
@@ -7,8 +7,10 @@ import os.path
 import re
 import sys
 
-def extract_exe_symbol_names (arch, exe_path, match_str):
-    command = 'dsymutil --arch %s -s "%s" | grep "%s" | colrm 1 69' % (arch, exe_path, match_str)
+
+def extract_exe_symbol_names(arch, exe_path, match_str):
+    command = 'dsymutil --arch %s -s "%s" | grep "%s" | colrm 1 69' % (
+        arch, exe_path, match_str)
     (command_exit_status, command_output) = commands.getstatusoutput(command)
     if command_exit_status == 0:
         if command_output:
@@ -19,27 +21,55 @@ def extract_exe_symbol_names (arch, exe_path, match_str):
         print 'error: command failed with exit status %i\n    command: %s' % (command_exit_status, command)
     return list()
 
+
 def verify_api(all_args):
     '''Verify the API in the specified library is valid given one or more binaries.'''
     usage = "usage: verify_api --library <path> [ --library <path> ...] executable1 [executable2 ...]"
-    description='''Verify the API in the specified library is valid given one or more binaries.
-    
+    description = '''Verify the API in the specified library is valid given one or more binaries.
+
     Example:
-    
+
         verify_api.py --library ~/Documents/src/lldb/build/Debug/LLDB.framework/LLDB --arch x86_64 /Applications/Xcode.app/Contents/PlugIns/DebuggerLLDB.ideplugin/Contents/MacOS/DebuggerLLDB --api-regex lldb
     '''
-    parser = optparse.OptionParser(description=description, prog='verify_api',usage=usage)
-    parser.add_option('-v', '--verbose', action='store_true', dest='verbose', help='display verbose debug info', default=False)
-    parser.add_option('-a', '--arch', type='string', action='append', dest='archs', help='architecure to use when checking the api')
-    parser.add_option('-r', '--api-regex', type='string', dest='api_regex_str', help='Exclude any undefined symbols that do not match this regular expression when searching for missing APIs.')
-    parser.add_option('-l', '--library', type='string', action='append', dest='libraries', help='Specify one or more libraries that will contain all needed APIs for the executables.')
+    parser = optparse.OptionParser(
+        description=description,
+        prog='verify_api',
+        usage=usage)
+    parser.add_option(
+        '-v',
+        '--verbose',
+        action='store_true',
+        dest='verbose',
+        help='display verbose debug info',
+        default=False)
+    parser.add_option(
+        '-a',
+        '--arch',
+        type='string',
+        action='append',
+        dest='archs',
+        help='architecure to use when checking the api')
+    parser.add_option(
+        '-r',
+        '--api-regex',
+        type='string',
+        dest='api_regex_str',
+        help='Exclude any undefined symbols that do not match this regular expression when searching for missing APIs.')
+    parser.add_option(
+        '-l',
+        '--library',
+        type='string',
+        action='append',
+        dest='libraries',
+        help='Specify one or more libraries that will contain all needed APIs for the executables.')
     (options, args) = parser.parse_args(all_args)
-    
+
     api_external_symbols = list()
     if options.archs:
         for arch in options.archs:
             for library in options.libraries:
-                external_symbols = extract_exe_symbol_names(arch, library, "(     SECT EXT)");
+                external_symbols = extract_exe_symbol_names(
+                    arch, library, "(     SECT EXT)")
                 if external_symbols:
                     for external_symbol in external_symbols:
                         api_external_symbols.append(external_symbol)
@@ -52,16 +82,17 @@ def verify_api(all_args):
         print "API symbols:"
         for (i, external_symbol) in enumerate(api_external_symbols):
             print "[%u] %s" % (i, external_symbol)
-    
+
     api_regex = None
     if options.api_regex_str:
         api_regex = re.compile(options.api_regex_str)
-    
-    for arch in options.archs:        
+
+    for arch in options.archs:
         for exe_path in args:
             print 'Verifying (%s) "%s"...' % (arch, exe_path)
             exe_errors = 0
-            undefined_symbols = extract_exe_symbol_names(arch, exe_path, "(     UNDF EXT)");
+            undefined_symbols = extract_exe_symbol_names(
+                arch, exe_path, "(     UNDF EXT)")
             for undefined_symbol in undefined_symbols:
                 if api_regex:
                     match = api_regex.search(undefined_symbol)
@@ -79,6 +110,6 @@ def verify_api(all_args):
                 print 'error: missing %u API symbols from %s' % (exe_errors, options.libraries)
             else:
                 print 'success'
-        
+
 if __name__ == '__main__':
-    verify_api(sys.argv[1:])
\ No newline at end of file
+    verify_api(sys.argv[1:])