---

jean-philippe-martin · mziccard · commit 66d76e26a7e3 · 2016-06-01T08:12:24.000+02:00
yaml --- r: 4317 b: refs/heads/gcs-nio c: ed440e9 h: refs/heads/master i: 4315: 8aeb551
diff --git a/[refs] b/[refs]
@@ -11,7 +11,7 @@ refs/tags/v0.0.12: 2fd8066e891fb3dfea69b65f6bf6461db79342b9
 refs/heads/compute-alpha: 969cba2627f1d53d352cc4a5ffe0879dacf65e6c
 refs/heads/dns-alpha: 2f90e7e338349287ace33375896907af0f032ca1
 refs/heads/dns-alpha-batch: 17442b07867021b85d0452f5f3eda29a3413288f
-refs/heads/gcs-nio: 441918b95daf8b04554f99e54753e13802b184d7
+refs/heads/gcs-nio: ed440e95d223c6d50621a06c329a57f80028b8a5
 refs/heads/logging-alpha: db5312bffa7fccac194f6a7feb8cc3066de16aff
 refs/tags/v0.1.0: a615317f7424ed58621b1f65d5c4d8cbbe8a6ed8
 refs/tags/v0.1.1: 7a7f6985fe465e9dd6a075af55493f42b4933be0
diff --git a/branches/gcs-nio/gcloud-java-examples/src/main/java/com/google/cloud/examples/nio/CountBytes.java b/branches/gcs-nio/gcloud-java-examples/src/main/java/com/google/cloud/examples/nio/CountBytes.java
@@ -71,25 +71,26 @@ private static void countFile(String fname) {
       ByteBuffer buf = ByteBuffer.allocate(bufSize);
       System.out.println("Reading the whole file...");
       Stopwatch sw = Stopwatch.createStarted();
-      SeekableByteChannel chan = Files.newByteChannel(path);
-      long total = 0;
-      int readCalls = 0;
-      MessageDigest md = MessageDigest.getInstance("MD5");
-      while (chan.read(buf) > 0) {
-        readCalls++;
-        md.update(buf.array(), 0, buf.position());
-        total += buf.position();
-        buf.flip();
-      }
-      readCalls++; // We must count the last call
-      long elapsed = sw.elapsed(TimeUnit.SECONDS);
-      System.out.println("Read all " + total + " bytes in " + elapsed + "s. " +
-          "(" + readCalls +" calls to chan.read)");
-      String hex = String.valueOf(BaseEncoding.base16().encode(md.digest()));
-      System.out.println("The MD5 is: 0x" + hex);
-      if (total != size) {
-        System.out.println("Wait, this doesn't match! We saw " + total + " bytes, " +
-            "yet the file size is listed at " + size + " bytes.");
+      try (SeekableByteChannel chan = Files.newByteChannel(path)) {
+        long total = 0;
+        int readCalls = 0;
+        MessageDigest md = MessageDigest.getInstance("MD5");
+        while (chan.read(buf) > 0) {
+          readCalls++;
+          md.update(buf.array(), 0, buf.position());
+          total += buf.position();
+          buf.flip();
+        }
+        readCalls++; // We must count the last call
+        long elapsed = sw.elapsed(TimeUnit.SECONDS);
+        System.out.println("Read all " + total + " bytes in " + elapsed + "s. " +
+            "(" + readCalls +" calls to chan.read)");
+        String hex = String.valueOf(BaseEncoding.base16().encode(md.digest()));
+        System.out.println("The MD5 is: 0x" + hex);
+        if (total != size) {
+          System.out.println("Wait, this doesn't match! We saw " + total + " bytes, " +
+              "yet the file size is listed at " + size + " bytes.");
+        }
       }
     } catch (Exception ex) {
       System.out.println(fname + ": " + ex.toString());
diff --git a/branches/gcs-nio/gcloud-java-examples/src/main/java/com/google/cloud/examples/nio/ParallelCountBytes.java b/branches/gcs-nio/gcloud-java-examples/src/main/java/com/google/cloud/examples/nio/ParallelCountBytes.java
@@ -79,7 +79,7 @@ public WorkUnit call() throws IOException {
         return this;
       }
       chan.position(pos);
-      // read until buffer it is full, or EOF
+      // read until buffer is full, or EOF
       while (chan.read(buf) > 0) {};
       return this;
     }

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public WorkUnit call() throws IOException {`
`79`	`79`	`return this;`
`80`	`80`	`}`
`81`	`81`	`chan.position(pos);`
`82`		`- // read until buffer it is full, or EOF`
	`82`	`+ // read until buffer is full, or EOF`
`83`	`83`	`while (chan.read(buf) > 0) {};`
`84`	`84`	`return this;`
`85`	`85`	`}`