Mercurial > jhg
comparison src/org/tmatesoft/hg/internal/RepositoryComparator.java @ 274:9fb50c04f03c
Use Nodeid.isNull check instead of NULL.equals
author | Artem Tikhomirov <tikhomirov.artem@gmail.com> |
---|---|
date | Thu, 25 Aug 2011 03:57:39 +0200 |
parents | 41a778e3fd31 |
children | 981f9f50bb6c |
comparison
equal
deleted
inserted
replaced
273:8f872bd7ddd2 | 274:9fb50c04f03c |
---|---|
153 if (localRepo.knownNode(rb.root)) { | 153 if (localRepo.knownNode(rb.root)) { |
154 // we known branch start, common head is somewhere in its descendants line | 154 // we known branch start, common head is somewhere in its descendants line |
155 checkUp2Head.add(rb); | 155 checkUp2Head.add(rb); |
156 } else { | 156 } else { |
157 // dig deeper in the history, if necessary | 157 // dig deeper in the history, if necessary |
158 if (!NULL.equals(rb.p1) && !localRepo.knownNode(rb.p1)) { | 158 if (!rb.p1.isNull() && !localRepo.knownNode(rb.p1)) { |
159 toQuery.add(rb.p1); | 159 toQuery.add(rb.p1); |
160 } | 160 } |
161 if (!NULL.equals(rb.p2) && !localRepo.knownNode(rb.p2)) { | 161 if (!rb.p2.isNull() && !localRepo.knownNode(rb.p2)) { |
162 toQuery.add(rb.p2); | 162 toQuery.add(rb.p2); |
163 } | 163 } |
164 } | 164 } |
165 } | 165 } |
166 } | 166 } |
243 // we known branch start, common head is somewhere in its descendants line | 243 // we known branch start, common head is somewhere in its descendants line |
244 checkUp2Head.add(rb); | 244 checkUp2Head.add(rb); |
245 } else { | 245 } else { |
246 chainElement.branchRoot = rb.root; | 246 chainElement.branchRoot = rb.root; |
247 // dig deeper in the history, if necessary | 247 // dig deeper in the history, if necessary |
248 boolean hasP1 = !NULL.equals(rb.p1), hasP2 = !NULL.equals(rb.p2); | 248 boolean hasP1 = !rb.p1.isNull(), hasP2 = !rb.p2.isNull(); |
249 if (hasP1 && !localRepo.knownNode(rb.p1)) { | 249 if (hasP1 && !localRepo.knownNode(rb.p1)) { |
250 toQuery.add(rb.p1); | 250 toQuery.add(rb.p1); |
251 // we might have seen parent node already, and recorded it as a branch chain | 251 // we might have seen parent node already, and recorded it as a branch chain |
252 // we shall reuse existing BC to get it completely initializer (head2chain map | 252 // we shall reuse existing BC to get it completely initializer (head2chain map |
253 // on second put with the same key would leave first BC uninitialized. | 253 // on second put with the same key would leave first BC uninitialized. |
354 } | 354 } |
355 | 355 |
356 // true when this BranchChain is a branch that spans up to very start of the repository | 356 // true when this BranchChain is a branch that spans up to very start of the repository |
357 // Thus, the only common revision is NULL, recorded in a fake BranchChain object shared between p1 and p2 | 357 // Thus, the only common revision is NULL, recorded in a fake BranchChain object shared between p1 and p2 |
358 /*package-local*/ boolean isRepoStart() { | 358 /*package-local*/ boolean isRepoStart() { |
359 return p1 == p2 && p1 != null && p1.branchHead == p1.branchRoot && NULL.equals(p1.branchHead); | 359 return p1 == p2 && p1 != null && p1.branchHead == p1.branchRoot && p1.branchHead.isNull(); |
360 } | 360 } |
361 | 361 |
362 @Override | 362 @Override |
363 public String toString() { | 363 public String toString() { |
364 return String.format("BranchChain [%s, %s]", branchRoot, branchHead); | 364 return String.format("BranchChain [%s, %s]", branchRoot, branchHead); |