[Lldb-commits] [PATCH] D45554: Make sure deleting all breakpoints clears their sites first

Davide Italiano via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 12 10:23:23 PDT 2018


davide added inline comments.


================
Comment at: packages/Python/lldbsuite/test/functionalities/breakpoint/breakpoint_command/main.c:14-15
+    // This line adds a real body to the function, so we can set more than one
+    // breakpoint in it.
+    printf("Observable side effect\n");
     return 0; // Set break point at this line.
----------------
you don't really need printf.
You can just declare a set of few new fresh variables, one per line, I think.


https://reviews.llvm.org/D45554





More information about the lldb-commits mailing list