Mercurial > jhg
comparison src/org/tmatesoft/hg/repo/HgStatusCollector.java @ 431:12f668401613
FIXMEs: awkward API refactored, what need to be internal got hidden; public aspects got captured in slim interfaces
| author | Artem Tikhomirov <tikhomirov.artem@gmail.com> |
|---|---|
| date | Thu, 29 Mar 2012 20:54:04 +0200 |
| parents | 48f993aa2f41 |
| children | 7bcfbc255f48 |
comparison
equal
deleted
inserted
replaced
| 430:d280759c2a3f | 431:12f668401613 |
|---|---|
| 30 import org.tmatesoft.hg.internal.IntMap; | 30 import org.tmatesoft.hg.internal.IntMap; |
| 31 import org.tmatesoft.hg.internal.ManifestRevision; | 31 import org.tmatesoft.hg.internal.ManifestRevision; |
| 32 import org.tmatesoft.hg.internal.Pool; | 32 import org.tmatesoft.hg.internal.Pool; |
| 33 import org.tmatesoft.hg.util.CancelSupport; | 33 import org.tmatesoft.hg.util.CancelSupport; |
| 34 import org.tmatesoft.hg.util.CancelledException; | 34 import org.tmatesoft.hg.util.CancelledException; |
| 35 import org.tmatesoft.hg.util.Convertor; | |
| 35 import org.tmatesoft.hg.util.Path; | 36 import org.tmatesoft.hg.util.Path; |
| 36 import org.tmatesoft.hg.util.PathPool; | |
| 37 import org.tmatesoft.hg.util.PathRewrite; | |
| 38 | 37 |
| 39 | 38 |
| 40 /** | 39 /** |
| 41 * Collect status information for changes between two repository revisions. | 40 * Collect status information for changes between two repository revisions. |
| 42 * | 41 * |
| 50 // with cpython repository, ~70 000 changes, complete Log (direct out, no reverse) output | 49 // with cpython repository, ~70 000 changes, complete Log (direct out, no reverse) output |
| 51 // no cache limit, no nodeids and fname caching - OOME on changeset 1035 | 50 // no cache limit, no nodeids and fname caching - OOME on changeset 1035 |
| 52 // no cache limit, but with cached nodeids and filenames - 1730+ | 51 // no cache limit, but with cached nodeids and filenames - 1730+ |
| 53 // cache limit 100 - 19+ minutes to process 10000, and still working (too long, stopped) | 52 // cache limit 100 - 19+ minutes to process 10000, and still working (too long, stopped) |
| 54 private final int cacheMaxSize = 50; // do not keep too much manifest revisions | 53 private final int cacheMaxSize = 50; // do not keep too much manifest revisions |
| 55 private PathPool pathPool; | 54 private Convertor<Path> pathPool; |
| 56 private final Pool<Nodeid> cacheNodes; | 55 private final Pool<Nodeid> cacheNodes; |
| 57 private final Pool<Path> cacheFilenames; | 56 private final Pool<Path> cacheFilenames; |
| 58 private final ManifestRevision emptyFakeState; | 57 private final ManifestRevision emptyFakeState; |
| 59 private Path.Matcher scope = new Path.Matcher.Any(); | 58 private Path.Matcher scope = new Path.Matcher.Any(); |
| 60 | 59 |
| 159 * @throws HgInvalidControlFileException | 158 * @throws HgInvalidControlFileException |
| 160 */ | 159 */ |
| 161 /*package-local*/ ManifestRevision raw(int rev) throws HgInvalidControlFileException { | 160 /*package-local*/ ManifestRevision raw(int rev) throws HgInvalidControlFileException { |
| 162 return get(rev); | 161 return get(rev); |
| 163 } | 162 } |
| 164 /*package-local*/ PathPool getPathPool() { | 163 /*package-local*/ Convertor<Path> getPathPool() { |
| 165 if (pathPool == null) { | 164 if (pathPool == null) { |
| 166 pathPool = new PathPool(new PathRewrite.Empty()); | 165 pathPool = cacheFilenames; |
| 167 } | 166 } |
| 168 return pathPool; | 167 return pathPool; |
| 169 } | 168 } |
| 170 | 169 |
| 171 /** | 170 /** |
| 172 * Allows sharing of a common path cache | 171 * Allows sharing of a common path cache |
| 173 */ | 172 */ |
| 174 public void setPathPool(PathPool pathPool) { | 173 public void setPathPool(Convertor<Path> pathConvertor) { |
| 175 this.pathPool = pathPool; | 174 pathPool = pathConvertor; |
| 176 } | 175 } |
| 177 | 176 |
| 178 /** | 177 /** |
| 179 * Limit activity of the collector to certain sub-tree of the repository. | 178 * Limit activity of the collector to certain sub-tree of the repository. |
| 180 * @param scopeMatcher tells whether collector shall report specific path, can be <code>null</code> | 179 * @param scopeMatcher tells whether collector shall report specific path, can be <code>null</code> |
| 291 } else { | 290 } else { |
| 292 try { | 291 try { |
| 293 Path copyTarget = r2fname; | 292 Path copyTarget = r2fname; |
| 294 Path copyOrigin = getOriginIfCopy(repo, copyTarget, r1Files, rev1); | 293 Path copyOrigin = getOriginIfCopy(repo, copyTarget, r1Files, rev1); |
| 295 if (copyOrigin != null) { | 294 if (copyOrigin != null) { |
| 296 inspector.copied(getPathPool().path(copyOrigin) /*pipe through pool, just in case*/, copyTarget); | 295 inspector.copied(getPathPool().mangle(copyOrigin) /*pipe through pool, just in case*/, copyTarget); |
| 297 } else { | 296 } else { |
| 298 inspector.added(copyTarget); | 297 inspector.added(copyTarget); |
| 299 } | 298 } |
| 300 } catch (HgInvalidFileException ex) { | 299 } catch (HgInvalidFileException ex) { |
| 301 // record exception to a mediator and continue, | 300 // record exception to a mediator and continue, |
