Re: [Lldb-commits] [PATCH] D20567: Avoid using stdio in TestVirtual

2016-05-26 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270831: Avoid using stdio in TestVirtual (authored by labath). Changed prior to commit: http://reviews.llvm.org/D20567?vs=58418&id=58579#toc Repository: rL LLVM http://reviews.llvm.org/D20567 Files

Re: [Lldb-commits] [PATCH] D20567: Avoid using stdio in TestVirtual

2016-05-25 Thread Pavel Labath via lldb-commits
labath updated this revision to Diff 58418. labath added a comment. New version, which does not rely on pretty-printers. http://reviews.llvm.org/D20567 Files: packages/Python/lldbsuite/test/lang/cpp/virtual/TestVirtual.py packages/Python/lldbsuite/test/lang/cpp/virtual/main.cpp Index: pack

Re: [Lldb-commits] [PATCH] D20567: Avoid using stdio in TestVirtual

2016-05-24 Thread Greg Clayton via lldb-commits
clayborg added a comment. I would just make a large buffer and use snprintf: char buffer[4096]; char *p = buffer; char *end = p + sizeof(buffer); p += snprintf(p, end - p, "a_as_A->a() = '%s'\n", a_as_A->a()); p += snprintf(p, end - p, "a_as_A->b() = '%s'\n", a_as_A->b()); http://revi

Re: [Lldb-commits] [PATCH] D20567: Avoid using stdio in TestVirtual

2016-05-24 Thread Pavel Labath via lldb-commits
labath added inline comments. Comment at: packages/Python/lldbsuite/test/lang/cpp/virtual/TestVirtual.py:64 @@ +63,3 @@ +self.assertTrue(golden.IsValid(), "Encountered an error reading the process's golden variable") +golden_str = str(golden) +self.assertT

Re: [Lldb-commits] [PATCH] D20567: Avoid using stdio in TestVirtual

2016-05-24 Thread Greg Clayton via lldb-commits
clayborg requested changes to this revision. This revision now requires changes to proceed. Comment at: packages/Python/lldbsuite/test/lang/cpp/virtual/TestVirtual.py:64 @@ +63,3 @@ +self.assertTrue(golden.IsValid(), "Encountered an error reading the process's golden vari