Clean up code when copying personal tool, explain the raii comment better.

Author: Jlalond

lldb/test/API/tools/lldb-dap/progress/Progress_emitter.py

@@ -29,7 +29,7 @@ def register_lldb_command(cls, debugger, module_name):
     @classmethod
     def create_options(cls):
         usage = "usage: %prog [options]"
-        description = "Jacob Lalonde's sbprogress testing tool"
+        description = "SBProgress testing tool"
         # Opt parse is deprecated, but leaving this the way it is because it allows help formating
         # Additionally all our commands use optparse right now, ideally we migrate them all in one go.
         parser = optparse.OptionParser(
@@ -70,7 +70,6 @@ def __call__(self, debugger, command, exe_ctx, result):
         total = cmd_options.total
         progress = lldb.SBProgress("Progress tester", "Detail", total, debugger)
 
-        # This actually should start at 1 but it's 6:30 on a Friday...
         for i in range(1, total):
             progress.Increment(1, f"Step {i}")
             time.sleep(cmd_options.seconds)

lldb/test/API/tools/lldb-dap/progress/TestDAP_Progress.py

@@ -32,7 +32,7 @@ def test_output(self):
 
         self.dap_server.wait_for_event("progressEnd", 15)
         # Expect at least a start, an update, and end event
-        # However because the progress is an RAII object and we can't guaruntee
+        # However because the underlying Progress instance is an RAII object and we can't guaruntee
         # it's deterministic destruction in the python API, we verify just start and update
         # otherwise this test could be flakey.
         self.assertTrue(len(self.dap_server.progress_events) > 0)