<div class="gmail_quote">On Fri, Jun 8, 2012 at 1:41 PM, Ojan Vafai <span dir="ltr"><<a href="mailto:ojan@chromium.org" target="_blank">ojan@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_quote"><div><div class="h5">On Fri, Jun 8, 2012 at 1:37 PM, Ryosuke Niwa <span dir="ltr"><<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div class="gmail_quote"><div>On Fri, Jun 8, 2012 at 1:27 PM, Dirk Pranke <span dir="ltr"><<a href="mailto:dpranke@chromium.org" target="_blank">dpranke@chromium.org</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





Most of these abstractions were either added to make testing easier<br>
(and faster since we didn't have to write to a real filesystem)<br></blockquote><div><br></div></div><div>That sounds like a bad idea.</div></div></blockquote><div><br></div></div></div><div>FWIW, I thought so at first as well, but as I've worked with the FileSystem abstraction, I've found it to be very helpful and provide almost no extra abstraction burden since it tries to closely match the Python APIs.</div>

</div></blockquote><div><br></div><div>It's extra code we have to read. For testing, we can just directly mock the native filesystem classes & methods Python provides.</div><div><br></div><div>In general, I agree with Filip's sentiment that we have way too much code.</div>

<div><br></div><div>- Ryosuke</div><div><br></div></div>