Mercurial > jhg
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 } |