comparison src/com/tmate/hgkit/ll/Nodeid.java @ 28:b2251b7a9823

Explicit cons arg for clone is more error-prone
author Artem Tikhomirov <tikhomirov.artem@gmail.com>
date Tue, 11 Jan 2011 04:34:34 +0100
parents d4fdd1845b3f
children 346b66add79d
comparison
equal deleted inserted replaced
27:b0a15cefdfd6 28:b2251b7a9823
19 19
20 public static int NULLREV = -1; 20 public static int NULLREV = -1;
21 private final byte[] binaryData; 21 private final byte[] binaryData;
22 22
23 /** 23 /**
24 * @param binaryRepresentation - byte[20], kept by reference. Use {@link #clone()} if original array may get changed. 24 * @param binaryRepresentation - byte[20], kept by reference
25 * @param shallClone - true if array is subject to future modification and shall be copied, not referenced
25 */ 26 */
26 public Nodeid(byte[] binaryRepresentation) { 27 public Nodeid(byte[] binaryRepresentation, boolean shallClone) {
27 // 5 int fields => 32 bytes 28 // 5 int fields => 32 bytes
28 // byte[20] => 48 bytes 29 // byte[20] => 48 bytes
29 if (binaryRepresentation == null || binaryRepresentation.length != 20) { 30 if (binaryRepresentation == null || binaryRepresentation.length != 20) {
30 throw new IllegalArgumentException(); 31 throw new IllegalArgumentException();
31 } 32 }
32 this.binaryData = binaryRepresentation; 33 this.binaryData = shallClone ? binaryRepresentation.clone() : binaryRepresentation;
33 } 34 }
34 35
35 @Override 36 @Override
36 public int hashCode() { 37 public int hashCode() {
37 // TODO consider own impl, especially if byte[] get replaced with 5 ints 38 // TODO consider own impl, especially if byte[] get replaced with 5 ints
64 for (int i = 0, j = offset; i < data.length; i++) { 65 for (int i = 0, j = offset; i < data.length; i++) {
65 int hiNibble = Character.digit(asciiRepresentation[j++], 16); 66 int hiNibble = Character.digit(asciiRepresentation[j++], 16);
66 int lowNibble = Character.digit(asciiRepresentation[j++], 16); 67 int lowNibble = Character.digit(asciiRepresentation[j++], 16);
67 data[i] = (byte) (((hiNibble << 4) | lowNibble) & 0xFF); 68 data[i] = (byte) (((hiNibble << 4) | lowNibble) & 0xFF);
68 } 69 }
69 return new Nodeid(data); 70 return new Nodeid(data, false);
70 } 71 }
71 } 72 }