<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Feb 28, 2017 at 4:09 PM Jason Molenda <<a href="mailto:jmolenda@apple.com">jmolenda@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br class="gmail_msg">
You mentioned that this kind of mechanical churn of the code is something everyone else in the LLVM/LLDB community supports.  I don't know about the LLVM community, but I strongly argue that this is the wrong way to develop a stable, reliable product.  I know I'm on the conservative side here, but these kinds of changes add no value to the product and add plenty of risk as they're being done while everyone is half paying attention to them going in.  Maybe if there was a vastly more extensive testsuite we could make large scale changes without fear of breaking the debugger, but we don't have that, no one has had the time to add one yet, and it's definitely something that we need to be putting energy into.  We don't need to be putting energy into switching to a different string representation across the board simply because it shows benefits in a handful of methods.<br class="gmail_msg"></blockquote><div><br></div><div>The way to develop a stable, reliable product is with an extensive test suite.  And the way to get an extensive test suite is to make a test infrastructure that encourages / enables low-footprint, low-overhead, fast, portable, and most importantly easy to write tests.</div><div><br></div><div>I wish working on LLDB were still my primary responsibility, but sadly it's not.  But rejecting obvious improvements to the codebase because there is insufficient test coverage for reasons outside the control of the person making the improvement is a quick way to send LLDB to the graveyard, not a way to make things better.</div><div><br></div><div>I mean, Chris was 100% in agreement before he left Apple, as was the rest of the LLVM community, and I haven't heard anything about priorities changing amongst leadership (if anything I've heard the opposite), so it's a bit strange to see this disconnect here</div></div></div>