[Lldb-commits] [PATCH] D26233: Fix Clang-tidy readability-redundant-string-cstr warnings

2016-11-02 Thread Malcolm Parsons via lldb-commits
malcolm.parsons created this revision. malcolm.parsons added a reviewer: zturner. malcolm.parsons added a subscriber: lldb-commits. Herald added subscribers: danalbert, tberghammer. https://reviews.llvm.org/D26233 Files: source/Breakpoint/BreakpointResolverAddress.cpp source/Breakpoint/Breakp

[Lldb-commits] [PATCH] D26233: Fix Clang-tidy readability-redundant-string-cstr warnings

2016-11-02 Thread Malcolm Parsons via lldb-commits
malcolm.parsons added a comment. In https://reviews.llvm.org/D26233#586005, @zturner wrote: > Nice improvement. I assume you ran the test suite and everything passed? I think a change like this would fail to compile if it was wrong. check-lldb passed: == Issue Details ===

[Lldb-commits] [lldb] r285855 - Fix Clang-tidy readability-redundant-string-cstr warnings

2016-11-02 Thread Malcolm Parsons via lldb-commits
Author: malcolm.parsons Date: Wed Nov 2 15:34:10 2016 New Revision: 285855 URL: http://llvm.org/viewvc/llvm-project?rev=285855&view=rev Log: Fix Clang-tidy readability-redundant-string-cstr warnings Reviewers: zturner, labath Subscribers: tberghammer, danalbert, lldb-commits Differential R

[Lldb-commits] [PATCH] D26233: Fix Clang-tidy readability-redundant-string-cstr warnings

2016-11-02 Thread Malcolm Parsons via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL285855: Fix Clang-tidy readability-redundant-string-cstr warnings (authored by malcolm.parsons). Changed prior to commit: https://reviews.llvm.org/D26233?vs=76679&id=76776#toc Repository: rL LLVM ht