Mercurial > jhg
annotate src/org/tmatesoft/hg/repo/Revlog.java @ 260:61cb6724ff36
Experimental alternative to HashMap in Pool to reduce memory footprint
| author | Artem Tikhomirov <tikhomirov.artem@gmail.com> | 
|---|---|
| date | Wed, 17 Aug 2011 02:35:15 +0200 | 
| parents | 0e01f9182e16 | 
| children | 9fb50c04f03c | 
| rev | line source | 
|---|---|
| 22 
603806cd2dc6
Status of local working dir against non-tip base revision
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
21diff
changeset | 1 /* | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 2 * Copyright (c) 2010-2011 TMate Software Ltd | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 3 * | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 4 * This program is free software; you can redistribute it and/or modify | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 5 * it under the terms of the GNU General Public License as published by | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 6 * the Free Software Foundation; version 2 of the License. | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 7 * | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 8 * This program is distributed in the hope that it will be useful, | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 9 * but WITHOUT ANY WARRANTY; without even the implied warranty of | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 11 * GNU General Public License for more details. | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 12 * | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 13 * For information on how to redistribute this software under | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 14 * the terms of a license other than GNU General Public License | 
| 102 
a3a2e5deb320
Updated contact address to support@hg4j.com
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
88diff
changeset | 15 * contact TMate Software at support@hg4j.com | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 16 */ | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 17 package org.tmatesoft.hg.repo; | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 18 | 
| 243 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 19 import static org.tmatesoft.hg.core.Nodeid.NULL; | 
| 80 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 20 import static org.tmatesoft.hg.repo.HgRepository.BAD_REVISION; | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 21 import static org.tmatesoft.hg.repo.HgRepository.TIP; | 
| 56 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 22 | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 23 import java.io.IOException; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 24 import java.nio.ByteBuffer; | 
| 56 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 25 import java.util.Arrays; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 26 import java.util.Collection; | 
| 171 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 27 import java.util.HashSet; | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 28 import java.util.LinkedList; | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 29 import java.util.List; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 30 | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 31 import org.tmatesoft.hg.core.HgBadStateException; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 32 import org.tmatesoft.hg.core.HgException; | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 33 import org.tmatesoft.hg.core.Nodeid; | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 34 import org.tmatesoft.hg.internal.DataAccess; | 
| 77 
c677e1593919
Moved RevlogStream implementation into .internal
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
74diff
changeset | 35 import org.tmatesoft.hg.internal.RevlogStream; | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 36 import org.tmatesoft.hg.util.ByteChannel; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 37 import org.tmatesoft.hg.util.CancelSupport; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 38 import org.tmatesoft.hg.util.CancelledException; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 39 import org.tmatesoft.hg.util.ProgressSupport; | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 40 | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 41 | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 42 /** | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 43 * Base class for all Mercurial entities that are serialized in a so called revlog format (changelog, manifest, data files). | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 44 * | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 45 * Implementation note: | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 46 * Hides actual actual revlog stream implementation and its access methods (i.e. RevlogStream.Inspector), iow shall not expose anything internal | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 47 * in public methods. | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 48 * | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 49 * @author Artem Tikhomirov | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 50 * @author TMate Software Ltd. | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 51 */ | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 52 abstract class Revlog { | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 53 | 
| 115 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 54 private final HgRepository repo; | 
| 21 
e929cecae4e1
Refactor to move revlog content to base class
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
3diff
changeset | 55 protected final RevlogStream content; | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 56 | 
| 115 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 57 protected Revlog(HgRepository hgRepo, RevlogStream contentStream) { | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 58 if (hgRepo == null) { | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 59 throw new IllegalArgumentException(); | 
| 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 60 } | 
| 115 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 61 if (contentStream == null) { | 
| 74 
6f1b88693d48
Complete refactoring to org.tmatesoft
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
56diff
changeset | 62 throw new IllegalArgumentException(); | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 63 } | 
| 115 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 64 repo = hgRepo; | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 65 content = contentStream; | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 66 } | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 67 | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 68 // invalid Revlog | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 69 protected Revlog(HgRepository hgRepo) { | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 70 repo = hgRepo; | 
| 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 71 content = null; | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 72 } | 
| 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 73 | 
| 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 74 public final HgRepository getRepo() { | 
| 115 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
102diff
changeset | 75 return repo; | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 76 } | 
| 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 77 | 
| 135 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 78 public final int getRevisionCount() { | 
| 21 
e929cecae4e1
Refactor to move revlog content to base class
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
3diff
changeset | 79 return content.revisionCount(); | 
| 
e929cecae4e1
Refactor to move revlog content to base class
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
3diff
changeset | 80 } | 
| 80 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 81 | 
| 135 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 82 public final int getLastRevision() { | 
| 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 83 return content.revisionCount() - 1; | 
| 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 84 } | 
| 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 85 | 
| 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 86 public final Nodeid getRevision(int revision) { | 
| 80 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 87 // XXX cache nodeids? | 
| 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 88 return Nodeid.fromBinary(content.nodeid(revision), 0); | 
| 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 89 } | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 90 | 
| 243 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 91 /** | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 92 * Get local revision number (index) of the specified revision. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 93 * | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 94 * For occasional queries, this method works with decent performance, despite its O(n/2) approach. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 95 * Alternatively, if you need to perform multiple queries (e.g. at least 15-20), {@link RevisionMap} may come handy. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 96 * | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 97 * @param nid revision to look up | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 98 * @return revision index, or {@link HgRepository#BAD_REVISION} if specified revision not found. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 99 */ | 
| 135 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 100 public final int getLocalRevision(Nodeid nid) { | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 101 int revision = content.findLocalRevisionNumber(nid); | 
| 80 
4222b04f34ee
Follow history of a file
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
77diff
changeset | 102 if (revision == BAD_REVISION) { | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 103 throw new IllegalArgumentException(String.format("%s doesn't represent a revision of %s", nid.toString(), this /*XXX HgDataFile.getPath might be more suitable here*/)); | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 104 } | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 105 return revision; | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 106 } | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 107 | 
| 39 
4e9b66b07a28
Check changelog group starts with proper (known) base
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
37diff
changeset | 108 // Till now, i follow approach that NULL nodeid is never part of revlog | 
| 135 
3959bffb14e9
explicit op name instead math op to get last rev number
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
115diff
changeset | 109 public final boolean isKnown(Nodeid nodeid) { | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 110 final int rn = content.findLocalRevisionNumber(nodeid); | 
| 218 
047b1dec7a04
Issue 7: Correctly handle manifest and changelog with different number of (or non-matching) revisions
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
200diff
changeset | 111 if (BAD_REVISION == rn) { | 
| 39 
4e9b66b07a28
Check changelog group starts with proper (known) base
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
37diff
changeset | 112 return false; | 
| 
4e9b66b07a28
Check changelog group starts with proper (known) base
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
37diff
changeset | 113 } | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 114 if (rn < 0 || rn >= content.revisionCount()) { | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 115 // Sanity check | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 116 throw new IllegalStateException(); | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 117 } | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 118 return true; | 
| 39 
4e9b66b07a28
Check changelog group starts with proper (known) base
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
37diff
changeset | 119 } | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 120 | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 121 /** | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 122 * Access to revision data as is (decompressed, but otherwise unprocessed, i.e. not parsed for e.g. changeset or manifest entries) | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 123 * @param nodeid | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 124 */ | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 125 protected void rawContent(Nodeid nodeid, ByteChannel sink) throws HgException, IOException, CancelledException { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 126 rawContent(getLocalRevision(nodeid), sink); | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 127 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 128 | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 129 /** | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 130 * @param revision - repo-local index of this file change (not a changelog revision number!) | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 131 */ | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 132 protected void rawContent(int revision, ByteChannel sink) throws HgException, IOException, CancelledException { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 133 if (sink == null) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 134 throw new IllegalArgumentException(); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 135 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 136 ContentPipe insp = new ContentPipe(sink, 0); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 137 insp.checkCancelled(); | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 138 content.iterate(revision, revision, true, insp); | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 139 insp.checkFailed(); | 
| 37 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 140 } | 
| 
e45e75e22523
Parse changesets from bundle's changelog group. Refactor Revlog to provide access to revision's raw content
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
31diff
changeset | 141 | 
| 56 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 142 /** | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 143 * XXX perhaps, return value Nodeid[2] and boolean needNodeids is better (and higher level) API for this query? | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 144 * | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 145 * @param revision - revision to query parents, or {@link HgRepository#TIP} | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 146 * @param parentRevisions - int[2] to get local revision numbers of parents (e.g. {6, -1}) | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 147 * @param parent1 - byte[20] or null, if parent's nodeid is not needed | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 148 * @param parent2 - byte[20] or null, if second parent's nodeid is not needed | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 149 * @return | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 150 */ | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 151 public void parents(int revision, int[] parentRevisions, byte[] parent1, byte[] parent2) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 152 if (revision != TIP && !(revision >= 0 && revision < content.revisionCount())) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 153 throw new IllegalArgumentException(String.valueOf(revision)); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 154 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 155 if (parentRevisions == null || parentRevisions.length < 2) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 156 throw new IllegalArgumentException(String.valueOf(parentRevisions)); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 157 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 158 if (parent1 != null && parent1.length < 20) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 159 throw new IllegalArgumentException(parent1.toString()); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 160 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 161 if (parent2 != null && parent2.length < 20) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 162 throw new IllegalArgumentException(parent2.toString()); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 163 } | 
| 77 
c677e1593919
Moved RevlogStream implementation into .internal
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
74diff
changeset | 164 class ParentCollector implements RevlogStream.Inspector { | 
| 56 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 165 public int p1 = -1; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 166 public int p2 = -1; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 167 public byte[] nodeid; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 168 | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 169 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess da) { | 
| 56 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 170 p1 = parent1Revision; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 171 p2 = parent2Revision; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 172 this.nodeid = new byte[20]; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 173 // nodeid arg now comes in 32 byte from (as in file format description), however upper 12 bytes are zeros. | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 174 System.arraycopy(nodeid, nodeid.length > 20 ? nodeid.length - 20 : 0, this.nodeid, 0, 20); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 175 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 176 }; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 177 ParentCollector pc = new ParentCollector(); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 178 content.iterate(revision, revision, false, pc); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 179 parentRevisions[0] = pc.p1; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 180 parentRevisions[1] = pc.p2; | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 181 if (parent1 != null) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 182 if (parentRevisions[0] == -1) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 183 Arrays.fill(parent1, 0, 20, (byte) 0); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 184 } else { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 185 content.iterate(parentRevisions[0], parentRevisions[0], false, pc); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 186 System.arraycopy(pc.nodeid, 0, parent1, 0, 20); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 187 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 188 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 189 if (parent2 != null) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 190 if (parentRevisions[1] == -1) { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 191 Arrays.fill(parent2, 0, 20, (byte) 0); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 192 } else { | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 193 content.iterate(parentRevisions[1], parentRevisions[1], false, pc); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 194 System.arraycopy(pc.nodeid, 0, parent2, 0, 20); | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 195 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 196 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 197 } | 
| 
576d6e8a09f6
Analog of 'hg status --change' command
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
49diff
changeset | 198 | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 199 /* | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 200 * XXX think over if it's better to do either: | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 201 * pw = getChangelog().new ParentWalker(); pw.init() and pass pw instance around as needed | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 202 * or | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 203 * add Revlog#getParentWalker(), static class, make cons() and #init package-local, and keep SoftReference to allow walker reuse. | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 204 * | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 205 * and yes, walker is not a proper name | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 206 */ | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 207 public final class ParentWalker { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 208 | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 209 | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 210 private Nodeid[] sequential; // natural repository order, childrenOf rely on ordering | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 211 private Nodeid[] sorted; // for binary search | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 212 private int[] sorted2natural; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 213 private Nodeid[] firstParent; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 214 private Nodeid[] secondParent; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 215 | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 216 // Nodeid instances shall be shared between all arrays | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 217 | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 218 public ParentWalker() { | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 219 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 220 | 
| 192 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 221 public HgRepository getRepo() { | 
| 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 222 return Revlog.this.getRepo(); | 
| 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 223 } | 
| 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 224 | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 225 public void init() { | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 226 final RevlogStream stream = Revlog.this.content; | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 227 final int revisionCount = stream.revisionCount(); | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 228 firstParent = new Nodeid[revisionCount]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 229 // although branches/merges are less frequent, and most of secondParent would be -1/null, some sort of | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 230 // SparseOrderedList might be handy, provided its inner structures do not overweight simplicity of an array | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 231 secondParent = new Nodeid[revisionCount]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 232 // | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 233 sequential = new Nodeid[revisionCount]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 234 sorted = new Nodeid[revisionCount]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 235 | 
| 77 
c677e1593919
Moved RevlogStream implementation into .internal
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
74diff
changeset | 236 RevlogStream.Inspector insp = new RevlogStream.Inspector() { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 237 | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 238 int ix = 0; | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 239 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess da) { | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 240 if (ix != revisionNumber) { | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 241 // XXX temp code, just to make sure I understand what's going on here | 
| 243 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 242 // FIXME check against cpython or another tool-mangled repository | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 243 throw new IllegalStateException(); | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 244 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 245 if (parent1Revision >= revisionNumber || parent2Revision >= revisionNumber) { | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 246 throw new IllegalStateException(); // sanity, revisions are sequential | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 247 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 248 final Nodeid nid = new Nodeid(nodeid, true); | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 249 sequential[ix] = sorted[ix] = nid; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 250 if (parent1Revision != -1) { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 251 assert parent1Revision < ix; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 252 firstParent[ix] = sequential[parent1Revision]; | 
| 183 
9807bf8f3a9c
Do not assume second parent can be set only when first one is set
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
171diff
changeset | 253 } | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 254 if (parent2Revision != -1) { // revlog of DataAccess.java has p2 set when p1 is -1 | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 255 assert parent2Revision < ix; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 256 secondParent[ix] = sequential[parent2Revision]; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 257 } | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 258 ix++; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 259 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 260 }; | 
| 197 
3a7696fb457c
Investigate optimization options to allow fast processing of huge repositories. Fix defect in StatusCollector that lead to wrong result comparing first revision to empty repo (-1 to 0), due to same TIP constant value
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
192diff
changeset | 261 stream.iterate(0, TIP, false, insp); | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 262 Arrays.sort(sorted); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 263 sorted2natural = new int[revisionCount]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 264 for (int i = 0; i < revisionCount; i++) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 265 Nodeid n = sequential[i]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 266 int x = Arrays.binarySearch(sorted, n); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 267 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 268 sorted2natural[x] = i; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 269 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 270 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 271 | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 272 private void assertSortedIndex(int x) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 273 if (x < 0) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 274 throw new HgBadStateException(); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 275 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 276 } | 
| 31 
346b66add79d
Basic lookup for incoming changes
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
29diff
changeset | 277 | 
| 
346b66add79d
Basic lookup for incoming changes
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
29diff
changeset | 278 // FIXME need to decide whether Nodeid(00 * 20) is always known or not | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 279 // right now Nodeid.NULL is not recognized as known if passed to this method, | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 280 // caller is supposed to make explicit check | 
| 31 
346b66add79d
Basic lookup for incoming changes
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
29diff
changeset | 281 public boolean knownNode(Nodeid nid) { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 282 return Arrays.binarySearch(sorted, nid) >= 0; | 
| 31 
346b66add79d
Basic lookup for incoming changes
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
29diff
changeset | 283 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 284 | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 285 /** | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 286 * null if none. only known nodes (as per #knownNode) are accepted as arguments | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 287 */ | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 288 public Nodeid firstParent(Nodeid nid) { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 289 int x = Arrays.binarySearch(sorted, nid); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 290 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 291 int i = sorted2natural[x]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 292 return firstParent[i]; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 293 } | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 294 | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 295 // never null, Nodeid.NULL if none known | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 296 public Nodeid safeFirstParent(Nodeid nid) { | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 297 Nodeid rv = firstParent(nid); | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 298 return rv == null ? Nodeid.NULL : rv; | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 299 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 300 | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 301 public Nodeid secondParent(Nodeid nid) { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 302 int x = Arrays.binarySearch(sorted, nid); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 303 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 304 int i = sorted2natural[x]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 305 return secondParent[i]; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 306 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 307 | 
| 49 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 308 public Nodeid safeSecondParent(Nodeid nid) { | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 309 Nodeid rv = secondParent(nid); | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 310 return rv == null ? Nodeid.NULL : rv; | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 311 } | 
| 
26e3eeaa3962
branch and user filtering for log operation
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
39diff
changeset | 312 | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 313 public boolean appendParentsOf(Nodeid nid, Collection<Nodeid> c) { | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 314 int x = Arrays.binarySearch(sorted, nid); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 315 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 316 int i = sorted2natural[x]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 317 Nodeid p1 = firstParent[i]; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 318 boolean modified = false; | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 319 if (p1 != null) { | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 320 modified = c.add(p1); | 
| 191 
b777502a06f5
Do not assume second parent can be set only when first one is set, again
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
183diff
changeset | 321 } | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 322 Nodeid p2 = secondParent[i]; | 
| 191 
b777502a06f5
Do not assume second parent can be set only when first one is set, again
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
183diff
changeset | 323 if (p2 != null) { | 
| 
b777502a06f5
Do not assume second parent can be set only when first one is set, again
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
183diff
changeset | 324 modified = c.add(p2) || modified; | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 325 } | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 326 return modified; | 
| 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 327 } | 
| 171 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 328 | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 329 // XXX alternative (and perhaps more reliable) approach would be to make a copy of allNodes and remove | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 330 // nodes, their parents and so on. | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 331 | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 332 // @return ordered collection of all children rooted at supplied nodes. Nodes shall not be descendants of each other! | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 333 // Nodeids shall belong to this revlog | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 334 public List<Nodeid> childrenOf(List<Nodeid> roots) { | 
| 171 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 335 HashSet<Nodeid> parents = new HashSet<Nodeid>(); | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 336 LinkedList<Nodeid> result = new LinkedList<Nodeid>(); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 337 int earliestRevision = Integer.MAX_VALUE; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 338 assert sequential.length == firstParent.length && firstParent.length == secondParent.length; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 339 // first, find earliest index of roots in question, as there's no sense | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 340 // to check children among nodes prior to branch's root node | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 341 for (Nodeid r : roots) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 342 int x = Arrays.binarySearch(sorted, r); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 343 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 344 int i = sorted2natural[x]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 345 if (i < earliestRevision) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 346 earliestRevision = i; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 347 } | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 348 parents.add(sequential[i]); // add canonical instance in hope equals() is bit faster when can do a == | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 349 } | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 350 for (int i = earliestRevision + 1; i < sequential.length; i++) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 351 if (parents.contains(firstParent[i]) || parents.contains(secondParent[i])) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 352 parents.add(sequential[i]); // to find next child | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 353 result.add(sequential[i]); | 
| 171 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 354 } | 
| 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 355 } | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 356 return result; | 
| 171 
2c3e96674e2a
Towards outgoing changes - initial detection logic, get connected with remote repo stub
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
157diff
changeset | 357 } | 
| 192 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 358 | 
| 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 359 /** | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 360 * @param nid possibly parent node, shall be {@link #knownNode(Nodeid) known} in this revlog. | 
| 192 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 361 * @return <code>true</code> if there's any node in this revlog that has specified node as one of its parents. | 
| 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 362 */ | 
| 200 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 363 public boolean hasChildren(Nodeid nid) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 364 int x = Arrays.binarySearch(sorted, nid); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 365 assertSortedIndex(x); | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 366 int i = sorted2natural[x]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 367 assert firstParent.length == secondParent.length; // just in case later I implement sparse array for secondParent | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 368 assert firstParent.length == sequential.length; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 369 // to use == instead of equals, take the same Nodeid instance we used to fill all the arrays. | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 370 final Nodeid canonicalNode = sequential[i]; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 371 i++; // no need to check node itself. child nodes may appear in sequential only after revision in question | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 372 for (; i < sequential.length; i++) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 373 // FIXME likely, not very effective. | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 374 // May want to optimize it with another (Tree|Hash)Set, created on demand on first use, | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 375 // however, need to be careful with memory usage | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 376 if (firstParent[i] == canonicalNode || secondParent[i] == canonicalNode) { | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 377 return true; | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 378 } | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 379 } | 
| 
114c9fe7b643
Performance optimization: reduce memory ParentWalker hogs
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
197diff
changeset | 380 return false; | 
| 192 
e5407b5a586a
Incoming and Outgoing commands are alive
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
191diff
changeset | 381 } | 
| 29 
6cce719bbb62
Collector for nodes and their parents
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
24diff
changeset | 382 } | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 383 | 
| 243 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 384 /** | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 385 * Effective int to Nodeid and vice versa translation. It's advised to use this class instead of | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 386 * multiple {@link Revlog#getLocalRevision(Nodeid)} calls. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 387 * | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 388 * getLocalRevision(Nodeid) with straightforward lookup approach performs O(n/2) | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 389 * #localRevision() is log(n), plus initialization is O(n) | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 390 */ | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 391 public final class RevisionMap { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 392 /* | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 393 * in fact, initialization is much slower as it instantiates Nodeids, while #getLocalRevision | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 394 * compares directly against byte buffer. Measuring cpython with 70k+ gives 3 times difference (47 vs 171) | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 395 * for complete changelog iteration. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 396 */ | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 397 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 398 /* | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 399 * XXX 3 * (x * 4) bytes. Can I do better? | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 400 */ | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 401 private Nodeid[] sequential; // natural repository order, childrenOf rely on ordering | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 402 private Nodeid[] sorted; // for binary search | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 403 private int[] sorted2natural; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 404 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 405 public RevisionMap() { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 406 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 407 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 408 public HgRepository getRepo() { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 409 return Revlog.this.getRepo(); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 410 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 411 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 412 /** | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 413 * @return <code>this</code> for convenience. | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 414 */ | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 415 public RevisionMap init(/*XXX Pool<Nodeid> to reuse nodeids, if possible. */) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 416 // XXX HgRepository.register((RepoChangeListener) this); // listen to changes in repo, re-init if needed? | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 417 final int revisionCount = Revlog.this.content.revisionCount(); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 418 sequential = new Nodeid[revisionCount]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 419 sorted = new Nodeid[revisionCount]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 420 sorted2natural = new int[revisionCount]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 421 RevlogStream.Inspector insp = new RevlogStream.Inspector() { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 422 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 423 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess data) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 424 final Nodeid nid = new Nodeid(nodeid, true); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 425 sequential[revisionNumber] = sorted[revisionNumber] = nid; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 426 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 427 }; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 428 Revlog.this.content.iterate(0, TIP, false, insp); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 429 Arrays.sort(sorted); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 430 for (int i = 0; i < revisionCount; i++) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 431 Nodeid n = sequential[i]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 432 int x = Arrays.binarySearch(sorted, n); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 433 sorted2natural[x] = i; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 434 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 435 return this; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 436 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 437 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 438 public Nodeid revision(int localRevision) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 439 return sequential[localRevision]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 440 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 441 public int localRevision(Nodeid revision) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 442 if (revision == null || NULL.equals(revision)) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 443 return BAD_REVISION; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 444 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 445 int x = Arrays.binarySearch(sorted, revision); | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 446 if (x < 0) { | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 447 return BAD_REVISION; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 448 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 449 return sorted2natural[x]; | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 450 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 451 } | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 452 | 
| 
0e01f9182e16
External cache Nodeid<->int added, Revlog.RevisionMap
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: 
218diff
changeset | 453 | 
| 157 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 454 protected static class ContentPipe implements RevlogStream.Inspector, CancelSupport { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 455 private final ByteChannel sink; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 456 private final CancelSupport cancelSupport; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 457 private Exception failure; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 458 private final int offset; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 459 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 460 /** | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 461 * @param _sink - cannot be <code>null</code> | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 462 * @param seekOffset - when positive, orders to pipe bytes to the sink starting from specified offset, not from the first byte available in DataAccess | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 463 */ | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 464 public ContentPipe(ByteChannel _sink, int seekOffset) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 465 assert _sink != null; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 466 sink = _sink; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 467 cancelSupport = CancelSupport.Factory.get(_sink); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 468 offset = seekOffset; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 469 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 470 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 471 protected void prepare(int revisionNumber, DataAccess da) throws HgException, IOException { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 472 if (offset > 0) { // save few useless reset/rewind operations | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 473 da.seek(offset); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 474 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 475 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 476 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 477 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess da) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 478 try { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 479 prepare(revisionNumber, da); // XXX perhaps, prepare shall return DA (sliced, if needed) | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 480 final ProgressSupport progressSupport = ProgressSupport.Factory.get(sink); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 481 ByteBuffer buf = ByteBuffer.allocate(512); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 482 progressSupport.start(da.length()); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 483 while (!da.isEmpty()) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 484 cancelSupport.checkCancelled(); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 485 da.readBytes(buf); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 486 buf.flip(); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 487 // XXX I may not rely on returned number of bytes but track change in buf position instead. | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 488 int consumed = sink.write(buf); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 489 // FIXME in fact, bad sink implementation (that consumes no bytes) would result in endless loop. Need to account for this | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 490 buf.compact(); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 491 progressSupport.worked(consumed); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 492 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 493 progressSupport.done(); // XXX shall specify whether #done() is invoked always or only if completed successfully. | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 494 } catch (IOException ex) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 495 recordFailure(ex); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 496 } catch (CancelledException ex) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 497 recordFailure(ex); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 498 } catch (HgException ex) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 499 recordFailure(ex); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 500 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 501 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 502 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 503 public void checkCancelled() throws CancelledException { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 504 cancelSupport.checkCancelled(); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 505 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 506 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 507 protected void recordFailure(Exception ex) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 508 assert failure == null; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 509 failure = ex; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 510 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 511 | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 512 public void checkFailed() throws HgException, IOException, CancelledException { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 513 if (failure == null) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 514 return; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 515 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 516 if (failure instanceof IOException) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 517 throw (IOException) failure; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 518 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 519 if (failure instanceof CancelledException) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 520 throw (CancelledException) failure; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 521 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 522 if (failure instanceof HgException) { | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 523 throw (HgException) failure; | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 524 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 525 throw new HgBadStateException(failure); | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 526 } | 
| 
d5268ca7715b
Merged branch wrap-data-access into default for resource-friendly data access. Updated API to promote that friendliness to clients (channels, not byte[]). More exceptions
 Artem Tikhomirov <tikhomirov.artem@gmail.com>diff
changeset | 527 } | 
| 2 
08db726a0fb7
Shaping out low-level Hg structures
 Artem Tikhomirov <tikhomirov.artem@gmail.com> parents: diff
changeset | 528 } | 
