*** This commit represents a complete reformatting of the LLDB source code

*** to conform to clang-format’s LLVM style.  This kind of mass change has
*** two obvious implications:

Firstly, merging this particular commit into a downstream fork may be a huge
effort.  Alternatively, it may be worth merging all changes up to this commit,
performing the same reformatting operation locally, and then discarding the
merge for this particular commit.  The commands used to accomplish this
reformatting were as follows (with current working directory as the root of
the repository):

    find . \( -iname "*.c" -or -iname "*.cpp" -or -iname "*.h" -or -iname "*.mm" \) -exec clang-format -i {} +
    find . -iname "*.py" -exec autopep8 --in-place --aggressive --aggressive {} + ;

The version of clang-format used was 3.9.0, and autopep8 was 1.2.4.

Secondly, “blame” style tools will generally point to this commit instead of
a meaningful prior commit.  There are alternatives available that will attempt
to look through this change and find the appropriate prior commit.  YMMV.

llvm-svn: 280751

1 files changed, 59 insertions, 35 deletions

diff --git a/lldb/packages/Python/lldbsuite/test/macosx/indirect_symbol/TestIndirectSymbols.py b/lldb/packages/Python/lldbsuite/test/macosx/indirect_symbol/TestIndirectSymbols.py
index f8b884003ad..e717e2ecdd9 100644
--- a/lldb/packages/Python/lldbsuite/test/macosx/indirect_symbol/TestIndirectSymbols.py
+++ b/lldb/packages/Python/lldbsuite/test/macosx/indirect_symbol/TestIndirectSymbols.py
@@ -3,13 +3,14 @@
 from __future__ import print_function
 
 
-
-import os, time
+import os
+import time
 import lldb
 from lldbsuite.test.decorators import *
 from lldbsuite.test.lldbtest import *
 from lldbsuite.test import lldbutil
 
+
 class TestIndirectFunctions(TestBase):
 
     mydir = TestBase.compute_mydir(__file__)
@@ -30,60 +31,83 @@ class TestIndirectFunctions(TestBase):
         target = self.dbg.CreateTarget(exe)
         self.assertTrue(target, VALID_TARGET)
 
-        self.main_source_spec = lldb.SBFileSpec (self.main_source)
+        self.main_source_spec = lldb.SBFileSpec(self.main_source)
 
-        break1 = target.BreakpointCreateBySourceRegex ("Set breakpoint here to step in indirect.", self.main_source_spec)
+        break1 = target.BreakpointCreateBySourceRegex(
+            "Set breakpoint here to step in indirect.", self.main_source_spec)
         self.assertTrue(break1, VALID_BREAKPOINT)
 
-        break2 = target.BreakpointCreateBySourceRegex ("Set breakpoint here to step in reexported.", self.main_source_spec)
+        break2 = target.BreakpointCreateBySourceRegex(
+            "Set breakpoint here to step in reexported.", self.main_source_spec)
         self.assertTrue(break2, VALID_BREAKPOINT)
 
         # Now launch the process, and do not stop at entry point.
-        process = target.LaunchSimple (None, None, self.get_process_working_directory())
+        process = target.LaunchSimple(
+            None, None, self.get_process_working_directory())
 
         self.assertTrue(process, PROCESS_IS_VALID)
 
         # The stop reason of the thread should be breakpoint.
-        threads = lldbutil.get_threads_stopped_at_breakpoint (process, break1)
+        threads = lldbutil.get_threads_stopped_at_breakpoint(process, break1)
         if len(threads) != 1:
-            self.fail ("Failed to stop at breakpoint 1.")
+            self.fail("Failed to stop at breakpoint 1.")
 
         thread = threads[0]
 
-        # Now do a step-into, and we should end up in the hidden target of this indirect function.
+        # Now do a step-into, and we should end up in the hidden target of this
+        # indirect function.
         thread.StepInto()
         curr_function = thread.GetFrameAtIndex(0).GetFunctionName()
-        self.assertTrue (curr_function == "call_through_indirect_hidden", "Stepped into indirect symbols.")
-
-        # Now set a breakpoint using the indirect symbol name, and make sure we get to that:
-        break_indirect = target.BreakpointCreateByName ("call_through_indirect");
-        self.assertTrue (break_indirect, VALID_BREAKPOINT)
-
-        # Now continue should take us to the second call through the indirect symbol:
-
-        threads = lldbutil.continue_to_breakpoint (process, break_indirect)
-        self.assertTrue (len(threads) == 1, "Stopped at breakpoint in indirect function.")
+        self.assertTrue(
+            curr_function == "call_through_indirect_hidden",
+            "Stepped into indirect symbols.")
+
+        # Now set a breakpoint using the indirect symbol name, and make sure we
+        # get to that:
+        break_indirect = target.BreakpointCreateByName("call_through_indirect")
+        self.assertTrue(break_indirect, VALID_BREAKPOINT)
+
+        # Now continue should take us to the second call through the indirect
+        # symbol:
+
+        threads = lldbutil.continue_to_breakpoint(process, break_indirect)
+        self.assertTrue(
+            len(threads) == 1,
+            "Stopped at breakpoint in indirect function.")
         curr_function = thread.GetFrameAtIndex(0).GetFunctionName()
-        self.assertTrue (curr_function == "call_through_indirect_hidden", "Stepped into indirect symbols.")
+        self.assertTrue(
+            curr_function == "call_through_indirect_hidden",
+            "Stepped into indirect symbols.")
 
         # Delete this breakpoint so it won't get in the way:
-        target.BreakpointDelete (break_indirect.GetID())
+        target.BreakpointDelete(break_indirect.GetID())
 
-        # Now continue to the site of the first re-exported function call in main:
-        threads = lldbutil.continue_to_breakpoint (process, break2)
+        # Now continue to the site of the first re-exported function call in
+        # main:
+        threads = lldbutil.continue_to_breakpoint(process, break2)
 
-        # This is stepping Into through a re-exported symbol to an indirect symbol:
+        # This is stepping Into through a re-exported symbol to an indirect
+        # symbol:
         thread.StepInto()
         curr_function = thread.GetFrameAtIndex(0).GetFunctionName()
-        self.assertTrue (curr_function == "call_through_indirect_hidden", "Stepped into indirect symbols.")
-
-        # And the last bit is to set a breakpoint on the re-exported symbol and make sure we are again in out target function.
-        break_reexported = target.BreakpointCreateByName ("reexport_to_indirect");
-        self.assertTrue (break_reexported, VALID_BREAKPOINT)
-
-        # Now continue should take us to the second call through the indirect symbol:
-
-        threads = lldbutil.continue_to_breakpoint (process, break_reexported)
-        self.assertTrue (len(threads) == 1, "Stopped at breakpoint in reexported function target.")
+        self.assertTrue(
+            curr_function == "call_through_indirect_hidden",
+            "Stepped into indirect symbols.")
+
+        # And the last bit is to set a breakpoint on the re-exported symbol and
+        # make sure we are again in out target function.
+        break_reexported = target.BreakpointCreateByName(
+            "reexport_to_indirect")
+        self.assertTrue(break_reexported, VALID_BREAKPOINT)
+
+        # Now continue should take us to the second call through the indirect
+        # symbol:
+
+        threads = lldbutil.continue_to_breakpoint(process, break_reexported)
+        self.assertTrue(
+            len(threads) == 1,
+            "Stopped at breakpoint in reexported function target.")
         curr_function = thread.GetFrameAtIndex(0).GetFunctionName()
-        self.assertTrue (curr_function == "call_through_indirect_hidden", "Stepped into indirect symbols.")
+        self.assertTrue(
+            curr_function == "call_through_indirect_hidden",
+            "Stepped into indirect symbols.")