<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    On 01/13/2011 12:03 PM, Alan Bateman wrote:
    <blockquote cite="mid:4D2EDC0B.7030207@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      <title></title>
      R&eacute;mi Forax wrote:
      <blockquote cite="mid:4D2E3C26.40901@univ-mlv.fr" type="cite">
        <meta content="text/html; charset=ISO-8859-1"
          http-equiv="Content-Type">
        :<br>
        First, I think this version of the API easier to use.<br>
      </blockquote>
      Thanks for spending time on it.<br>
    </blockquote>
    <br>
    [...]<br>
    <br>
    <blockquote cite="mid:4D2EDC0B.7030207@oracle.com" type="cite">
      <br>
      <blockquote cite="mid:4D2E3C26.40901@univ-mlv.fr" type="cite"> <br>
        readLines: why not returning a lazy Collection instead of a
        List.<br>
        if a user want a List, this code <br>
        List&lt;String&gt; lines = new
        ArrayList&lt;&gt;(Files.readLines(...))<br>
      </blockquote>
      Like readBytes (or readAllBytes as you've suggested), this method
      reads
      the entire file so I didn't make it lazy. An
      Iterable&lt;String&gt;
      might be better but I initially proposed it as List so that it
      could be
      used as such if needed.<br>
      <br>
      <br>
      <blockquote cite="mid:4D2E3C26.40901@univ-mlv.fr" type="cite"> Or
        perhaps better, wait jdk 8 and the stream API.<br>
        <br>
        readBytes() can be renamed to readAllBytes().<br>
      </blockquote>
      Maybe, it was originally readAllBytes way back.<br>
    </blockquote>
    <br>
    I think I prefer readAllBytes and readAllLines. And future will tell
    if lazy versions will be added later.<br>
    <br>
    <blockquote cite="mid:4D2EDC0B.7030207@oracle.com" type="cite">
      <br>
      <blockquote cite="mid:4D2E3C26.40901@univ-mlv.fr" type="cite"> <br>
        write(...): javadoc should also contains an example showing how
        to
        create a new File:<br>
        <pre>Files.write(path, bytes, StandardOpenOption.CREATE, StandardOpenOption.TRUNCATE_EXISTING, StandardOpenOption.WRITE);
  </pre>
        because I'm sure that users will forget TRUNCATE_EXISTING.<br>
      </blockquote>
      This is the default if you don't specify any options ("That is, if
      no
      options are present then file is created if it doesn't exist or is
      first truncated to a size of 0 if it exists"). The idea is that
      you
      shouldn't need to specify any ideas for the common case, ie:
      Files.write(path, bytes). Maybe I need to make that clearer in the
      docs.<br>
    </blockquote>
    <br>
    Perhaps just something saying that this is equivalent to
    write(WRITE|CREATE|TRUNCATE_EXISTING)<br>
    <br>
    <blockquote cite="mid:4D2EDC0B.7030207@oracle.com" type="cite">
      <br>
      Thanks again (as always).<br>
      <br>
      -Alan<br>
      <br>
      <br>
    </blockquote>
    <br>
    cheers,<br>
    R&eacute;mi<br>
  </body>
</html>