https://issues.dlang.org/show_bug.cgi?id=22043
Nick Treleaven <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #2 from Nick Treleaven <[email protected]> --- > writeln(pretty); // ugly lineSplitter is even worse: https://dlang.org/phobos/std_string.html#lineSplitter > writeln(lineSplitter(s).array); // splitLines(s) What? Does it mean the two expressions are equivalent??! Then: foreach (line; lines) { writeln(line); // witness[i++] } writeln(i); // witness.length Transforming those assert statements makes them unintelligible. It should only do it (if at all) when the right hand side has no side effects and is not a function call. It's OK when the RHS is a literal. --
