Mercurial > hg4j
comparison src/org/tmatesoft/hg/internal/DataAccess.java @ 418:528b6780a8bd
A bit of FIXME cleanup (mostly degraded to TODO post 1.0), comments and javadoc
author | Artem Tikhomirov <tikhomirov.artem@gmail.com> |
---|---|
date | Thu, 22 Mar 2012 21:02:20 +0100 |
parents | fdc1db8f7f61 |
children | 6c22bdc0bdfd |
comparison
equal
deleted
inserted
replaced
417:ccd7d25e5aea | 418:528b6780a8bd |
---|---|
85 return; | 85 return; |
86 } | 86 } |
87 throw new IOException(String.format("No data, can't read %d bytes", length)); | 87 throw new IOException(String.format("No data, can't read %d bytes", length)); |
88 } | 88 } |
89 // reads bytes into ByteBuffer, up to its limit or total data length, whichever smaller | 89 // reads bytes into ByteBuffer, up to its limit or total data length, whichever smaller |
90 // FIXME perhaps, in DataAccess paradigm (when we read known number of bytes, we shall pass specific byte count to read) | 90 // TODO post-1.0 perhaps, in DataAccess paradigm (when we read known number of bytes, we shall pass specific byte count to read) |
91 // for 1.0, it's ok as it's our internal class | |
91 public void readBytes(ByteBuffer buf) throws IOException { | 92 public void readBytes(ByteBuffer buf) throws IOException { |
92 // int toRead = Math.min(buf.remaining(), (int) length()); | 93 // int toRead = Math.min(buf.remaining(), (int) length()); |
93 // if (buf.hasArray()) { | 94 // if (buf.hasArray()) { |
94 // readBytes(buf.array(), buf.arrayOffset(), toRead); | 95 // readBytes(buf.array(), buf.arrayOffset(), toRead); |
95 // } else { | 96 // } else { |
96 // byte[] bb = new byte[toRead]; | 97 // byte[] bb = new byte[toRead]; |
97 // readBytes(bb, 0, bb.length); | 98 // readBytes(bb, 0, bb.length); |
98 // buf.put(bb); | 99 // buf.put(bb); |
99 // } | 100 // } |
100 // FIXME optimize to read as much as possible at once | 101 // TODO post-1.0 optimize to read as much as possible at once |
101 while (!isEmpty() && buf.hasRemaining()) { | 102 while (!isEmpty() && buf.hasRemaining()) { |
102 buf.put(readByte()); | 103 buf.put(readByte()); |
103 } | 104 } |
104 } | 105 } |
105 public byte readByte() throws IOException { | 106 public byte readByte() throws IOException { |