Mercurial > jhg
annotate src/org/tmatesoft/hg/repo/HgDataFile.java @ 361:8099939af5fa
Utilize status object to supply more information about manifest check for specific file
| author | Artem Tikhomirov <tikhomirov.artem@gmail.com> | 
|---|---|
| date | Fri, 09 Dec 2011 01:13:53 +0100 | 
| parents | f2c11fe7f3e9 | 
| children | 189dc6dc1c3e | 
| rev | line source | 
|---|---|
| 
17
 
571e1b2cc3f7
Query file for its parents. Demo of recently added ignore and digest support from within cat cmd
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
5 
diff
changeset
 | 
1 /* | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
2 * Copyright (c) 2010-2011 TMate Software Ltd | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
3 * | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
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: 
56 
diff
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: 
56 
diff
changeset
 | 
6 * the Free Software Foundation; version 2 of the License. | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
7 * | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
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: 
56 
diff
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: 
56 
diff
changeset
 | 
10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
11 * GNU General Public License for more details. | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
12 * | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
13 * For information on how to redistribute this software under | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
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: 
88 
diff
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: 
56 
diff
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 | 
| 
148
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
19 import static org.tmatesoft.hg.repo.HgInternals.wrongLocalRevision; | 
| 
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
20 import static org.tmatesoft.hg.repo.HgRepository.*; | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
21 | 
| 
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
 | 
22 import java.io.ByteArrayOutputStream; | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
23 import java.io.File; | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
24 import java.io.FileInputStream; | 
| 
148
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
25 import java.io.IOException; | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
26 import java.nio.ByteBuffer; | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
27 import java.nio.channels.FileChannel; | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
28 import java.util.ArrayList; | 
| 
240
 
29231022fec8
Do not expect file history to be ordered
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
237 
diff
changeset
 | 
29 import java.util.Arrays; | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
30 import java.util.Collection; | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
31 import java.util.Collections; | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
32 import java.util.List; | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
33 | 
| 
148
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
34 import org.tmatesoft.hg.core.HgDataStreamException; | 
| 
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
 | 
35 import org.tmatesoft.hg.core.HgException; | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
36 import org.tmatesoft.hg.core.HgInvalidControlFileException; | 
| 
347
 
8da7ade36c57
Add specific IAE subclass to handle wrong (e.g. outdated after rollback) revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
328 
diff
changeset
 | 
37 import org.tmatesoft.hg.core.HgInvalidRevisionException; | 
| 
328
 
a674b8590362
Move file tree history to upper API level
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
323 
diff
changeset
 | 
38 import org.tmatesoft.hg.core.HgLogCommand; | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
39 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
 | 
40 import org.tmatesoft.hg.internal.DataAccess; | 
| 
121
 
b1d6208fb517
Conditionally apply filters to file content
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
115 
diff
changeset
 | 
41 import org.tmatesoft.hg.internal.FilterByteChannel; | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
42 import org.tmatesoft.hg.internal.FilterDataAccess; | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
43 import org.tmatesoft.hg.internal.IntMap; | 
| 
77
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
44 import org.tmatesoft.hg.internal.RevlogStream; | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
45 import org.tmatesoft.hg.util.ByteChannel; | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
46 import org.tmatesoft.hg.util.CancelSupport; | 
| 
148
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
47 import org.tmatesoft.hg.util.CancelledException; | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
48 import org.tmatesoft.hg.util.Pair; | 
| 
133
 
4a948ec83980
core.Path to util.Path as it's not Hg repo dependant
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
129 
diff
changeset
 | 
49 import org.tmatesoft.hg.util.Path; | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
50 import org.tmatesoft.hg.util.ProgressSupport; | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
51 | 
| 
5
 
fc265ddeab26
File content and non-effective, although working, patch application
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
3 
diff
changeset
 | 
52 | 
| 
17
 
571e1b2cc3f7
Query file for its parents. Demo of recently added ignore and digest support from within cat cmd
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
5 
diff
changeset
 | 
53 | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
54 /** | 
| 
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
55 * ? name:HgFileNode? | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
56 * | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
57 * @author Artem Tikhomirov | 
| 
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
58 * @author TMate Software Ltd. | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
59 */ | 
| 
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
60 public class HgDataFile extends Revlog { | 
| 
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
61 | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
62 // absolute from repo root? | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
63 // slashes, unix-style? | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
64 // repo location agnostic, just to give info to user, not to access real storage | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
65 private final Path path; | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
66 private Metadata metadata; // get initialized on first access to file content. | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
67 | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
68 /*package-local*/HgDataFile(HgRepository hgRepo, Path filePath, RevlogStream content) { | 
| 
21
 
e929cecae4e1
Refactor to move revlog content to base class
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
17 
diff
changeset
 | 
69 super(hgRepo, content); | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
70 path = filePath; | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
71 } | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
72 | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
73 /*package-local*/HgDataFile(HgRepository hgRepo, Path filePath) { | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
74 super(hgRepo); | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
75 path = filePath; | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
76 } | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
77 | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
78 // exists is not the best name possible. now it means no file with such name was ever known to the repo. | 
| 
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
79 // it might be confused with files existed before but lately removed. | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
80 public boolean exists() { | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
81 return content != null; // XXX need better impl | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
82 } | 
| 
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
83 | 
| 
77
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
84 // human-readable (i.e. "COPYING", not "store/data/_c_o_p_y_i_n_g.i") | 
| 
74
 
6f1b88693d48
Complete refactoring to org.tmatesoft
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
56 
diff
changeset
 | 
85 public Path getPath() { | 
| 
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
 | 
86 return path; // hgRepo.backresolve(this) -> name? In this case, what about hashed long names? | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
87 } | 
| 
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
88 | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
89 /** | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
90 * Handy shorthand for {@link #length(int) length(getLocalRevision(nodeid))} | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
91 * | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
92 * @param nodeid revision of the file | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
93 * | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
94 * @return size of the file content at the given revision | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
95 * @throws HgInvalidRevisionException if supplied argument doesn't represent revision index in this revlog | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
96 * @throws HgDataStreamException if attempt to access file metadata failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
97 * @throws HgInvalidControlFileException if access to revlog index/data entry failed | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
98 */ | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
99 public int length(Nodeid nodeid) throws HgDataStreamException, HgInvalidControlFileException, HgInvalidRevisionException { | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
100 return length(getLocalRevision(nodeid)); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
101 } | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
102 | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
103 /** | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
104 * @return size of the file content at the revision identified by local revision number. | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
105 * @throws HgInvalidRevisionException if supplied argument doesn't represent revision index in this revlog | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
106 * @throws HgDataStreamException if attempt to access file metadata failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
107 * @throws HgInvalidControlFileException if access to revlog index/data entry failed | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
108 */ | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
109 public int length(int localRev) throws HgDataStreamException, HgInvalidControlFileException, HgInvalidRevisionException { | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
110 if (metadata == null || !metadata.checked(localRev)) { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
111 checkAndRecordMetadata(localRev); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
112 } | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
113 final int dataLen = content.dataLength(localRev); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
114 if (metadata.known(localRev)) { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
115 return dataLen - metadata.dataOffset(localRev); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
116 } | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
117 return dataLen; | 
| 
22
 
603806cd2dc6
Status of local working dir against non-tip base revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
21 
diff
changeset
 | 
118 } | 
| 
 
603806cd2dc6
Status of local working dir against non-tip base revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
21 
diff
changeset
 | 
119 | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
120 /** | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
121 * Reads content of the file from working directory. If file present in the working directory, its actual content without | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
122 * any filters is supplied through the sink. If file does not exist in the working dir, this method provides content of a file | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
123 * as if it would be refreshed in the working copy, i.e. its corresponding revision | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
124 * (XXX according to dirstate? file tip?) is read from the repository, and filters repo -> working copy get applied. | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
125 * | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
126 * @param sink where to pipe content to | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
127 * @throws HgDataStreamException to indicate troubles reading repository file | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
128 * @throws CancelledException if operation was cancelled | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
129 */ | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
130 public void workingCopy(ByteChannel sink) throws HgDataStreamException, CancelledException { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
131 File f = getRepo().getFile(this); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
132 if (f.exists()) { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
133 final CancelSupport cs = CancelSupport.Factory.get(sink); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
134 final ProgressSupport progress = ProgressSupport.Factory.get(sink); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
135 final long flength = f.length(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
136 final int bsize = (int) Math.min(flength, 32*1024); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
137 progress.start((int) (flength > Integer.MAX_VALUE ? flength >>> 15 /*32 kb buf size*/ : flength)); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
138 ByteBuffer buf = ByteBuffer.allocate(bsize); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
139 FileChannel fc = null; | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
140 try { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
141 fc = new FileInputStream(f).getChannel(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
142 while (fc.read(buf) != -1) { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
143 cs.checkCancelled(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
144 buf.flip(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
145 int consumed = sink.write(buf); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
146 progress.worked(flength > Integer.MAX_VALUE ? 1 : consumed); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
147 buf.compact(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
148 } | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
149 } catch (IOException ex) { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
150 throw new HgDataStreamException(getPath(), ex); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
151 } finally { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
152 progress.done(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
153 if (fc != null) { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
154 try { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
155 fc.close(); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
156 } catch (IOException ex) { | 
| 
295
 
981f9f50bb6c
Issue 11: Error log facility. SessionContext to share common facilities
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
277 
diff
changeset
 | 
157 getRepo().getContext().getLog().info(getClass(), ex, null); | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
158 } | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
159 } | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
160 } | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
161 } else { | 
| 
355
 
f2c11fe7f3e9
Newline filter shall respect whole stream when deciding whether to process line terminators, hence added stream preview functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
354 
diff
changeset
 | 
162 // FIXME not TIP, but revision according to dirstate!!! | 
| 
 
f2c11fe7f3e9
Newline filter shall respect whole stream when deciding whether to process line terminators, hence added stream preview functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
354 
diff
changeset
 | 
163 // add tests for this case | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
164 contentWithFilters(TIP, sink); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
165 } | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
166 } | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
167 | 
| 
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
 | 
168 // public void content(int revision, ByteChannel sink, boolean applyFilters) throws HgDataStreamException, 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
 | 
169 // byte[] content = content(revision); | 
| 
 
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
 | 
170 // final CancelSupport 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
 | 
171 // 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
 | 
172 // 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
 | 
173 // int left = content.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
 | 
174 // progressSupport.start(left); | 
| 
 
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
 | 
175 // int offset = 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
 | 
176 // 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
 | 
177 // ByteChannel _sink = applyFilters ? new FilterByteChannel(sink, getRepo().getFiltersFromRepoToWorkingDir(getPath())) : 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
 | 
178 // do { | 
| 
 
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
 | 
179 // buf.put(content, offset, Math.min(left, buf.remaining())); | 
| 
 
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
 | 
180 // 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
 | 
181 // 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
 | 
182 // // 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
 | 
183 // 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
 | 
184 // 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
 | 
185 // offset += 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
 | 
186 // left -= 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
 | 
187 // 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
 | 
188 // } while (left > 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
 | 
189 // 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
 | 
190 // } | 
| 
 
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
 | 
191 | 
| 
 
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
 | 
192 /*XXX not sure distinct method contentWithFilters() is the best way to do, perhaps, callers shall add filters themselves?*/ | 
| 
347
 
8da7ade36c57
Add specific IAE subclass to handle wrong (e.g. outdated after rollback) revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
328 
diff
changeset
 | 
193 public void contentWithFilters(int revision, ByteChannel sink) throws HgDataStreamException, CancelledException, HgInvalidRevisionException { | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
194 if (revision == WORKING_COPY) { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
195 workingCopy(sink); // pass un-mangled sink | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
196 } else { | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
197 content(revision, new FilterByteChannel(sink, getRepo().getFiltersFromRepoToWorkingDir(getPath()))); | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
198 } | 
| 
115
 
c0cc2535462c
Introduced channels to pipeline (and easily filter) data streams
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
102 
diff
changeset
 | 
199 } | 
| 
22
 
603806cd2dc6
Status of local working dir against non-tip base revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
21 
diff
changeset
 | 
200 | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
201 // for data files need to check heading of the file content for possible metadata | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
202 // @see http://mercurial.selenic.com/wiki/FileFormats#data.2BAC8- | 
| 
347
 
8da7ade36c57
Add specific IAE subclass to handle wrong (e.g. outdated after rollback) revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
328 
diff
changeset
 | 
203 public void content(int revision, ByteChannel sink) throws HgDataStreamException, CancelledException, HgInvalidRevisionException { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
204 if (revision == TIP) { | 
| 
135
 
3959bffb14e9
explicit op name instead math op to get last rev number
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
134 
diff
changeset
 | 
205 revision = getLastRevision(); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
206 } | 
| 
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
 | 
207 if (revision == WORKING_COPY) { | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
208 // sink is supposed to come into workingCopy without filters | 
| 
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
209 // thus we shall not get here (into #content) from #contentWithFilters(WC) | 
| 
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
 | 
210 workingCopy(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
 | 
211 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
 | 
212 } | 
| 
 
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
 | 
213 if (wrongLocalRevision(revision) || revision == BAD_REVISION) { | 
| 
347
 
8da7ade36c57
Add specific IAE subclass to handle wrong (e.g. outdated after rollback) revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
328 
diff
changeset
 | 
214 throw new HgInvalidRevisionException(revision); | 
| 
148
 
1a7a9a20e1f9
Exceptions, javadoc. Initial cancel and progress support
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
135 
diff
changeset
 | 
215 } | 
| 
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
 | 
216 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
 | 
217 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
 | 
218 } | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
219 if (metadata == null) { | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
220 metadata = new Metadata(); | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
221 } | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
222 ErrorHandlingInspector insp; | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
223 if (metadata.none(revision)) { | 
| 
355
 
f2c11fe7f3e9
Newline filter shall respect whole stream when deciding whether to process line terminators, hence added stream preview functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
354 
diff
changeset
 | 
224 insp = new ContentPipe(sink, 0, getRepo().getContext().getLog()); | 
| 
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
 | 
225 } else if (metadata.known(revision)) { | 
| 
355
 
f2c11fe7f3e9
Newline filter shall respect whole stream when deciding whether to process line terminators, hence added stream preview functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
354 
diff
changeset
 | 
226 insp = new ContentPipe(sink, metadata.dataOffset(revision), getRepo().getContext().getLog()); | 
| 
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
 | 
227 } else { | 
| 
 
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
 | 
228 // do not know if there's metadata | 
| 
355
 
f2c11fe7f3e9
Newline filter shall respect whole stream when deciding whether to process line terminators, hence added stream preview functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
354 
diff
changeset
 | 
229 insp = new MetadataInspector(metadata, getPath(), new ContentPipe(sink, 0, getRepo().getContext().getLog())); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
230 } | 
| 
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
 | 
231 insp.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
 | 
232 super.content.iterate(revision, revision, true, insp); | 
| 
 
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
 | 
233 try { | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
234 insp.checkFailed(); // XXX is there real need to throw IOException from ContentPipe? | 
| 
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
 | 
235 } catch (HgDataStreamException 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
 | 
236 throw ex; | 
| 
237
 
6e1373b54e9b
Allow access to working copy content through HgDataFile. Give access to repository's working dir
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
233 
diff
changeset
 | 
237 } catch (IOException ex) { | 
| 
323
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
238 throw new HgDataStreamException(getPath(), ex).setRevisionNumber(revision); | 
| 
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 } 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
 | 
240 // shall not happen, unless we changed ContentPipe or its subclass | 
| 
215
 
41a778e3fd31
Issue 5: Facilities for progress and cancellation. More specific exceptions. Exceptions from callbacks as RuntimeException
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
163 
diff
changeset
 | 
241 throw new HgDataStreamException(getPath(), ex.getClass().getName(), ex); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
242 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
243 } | 
| 
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
 | 
244 | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
245 private static class HistoryNode { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
246 int changeset; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
247 Nodeid cset; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
248 HistoryNode parent1, parent2; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
249 List<HistoryNode> children; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
250 | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
251 HistoryNode(int cs, HistoryNode p1, HistoryNode p2) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
252 changeset = cs; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
253 parent1 = p1; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
254 parent2 = p2; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
255 if (p1 != null) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
256 p1.addChild(this); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
257 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
258 if (p2 != null) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
259 p2.addChild(this); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
260 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
261 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
262 | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
263 Nodeid changesetRevision() { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
264 assert cset != null : "we initialize all csets prior to use"; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
265 return cset; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
266 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
267 | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
268 void addChild(HistoryNode child) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
269 if (children == null) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
270 children = new ArrayList<HistoryNode>(2); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
271 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
272 children.add(child); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
273 } | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
274 } | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
275 | 
| 
328
 
a674b8590362
Move file tree history to upper API level
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
323 
diff
changeset
 | 
276 /** | 
| 
 
a674b8590362
Move file tree history to upper API level
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
323 
diff
changeset
 | 
277 * @deprecated use {@link HgLogCommand#execute(org.tmatesoft.hg.core.HgChangesetTreeHandler)} instead | 
| 
 
a674b8590362
Move file tree history to upper API level
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
323 
diff
changeset
 | 
278 */ | 
| 
 
a674b8590362
Move file tree history to upper API level
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
323 
diff
changeset
 | 
279 @Deprecated | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
280 public void history(HgChangelog.TreeInspector inspector) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
281 final CancelSupport cancelSupport = CancelSupport.Factory.get(inspector); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
282 try { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
283 final boolean[] needsSorting = { false }; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
284 final HistoryNode[] completeHistory = new HistoryNode[getRevisionCount()]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
285 final int[] commitRevisions = new int[completeHistory.length]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
286 RevlogStream.Inspector insp = new RevlogStream.Inspector() { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
287 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess data) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
288 if (revisionNumber > 0) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
289 if (commitRevisions[revisionNumber-1] > linkRevision) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
290 needsSorting[0] = true; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
291 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
292 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
293 commitRevisions[revisionNumber] = linkRevision; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
294 HistoryNode p1 = null, p2 = null; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
295 if (parent1Revision != -1) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
296 p1 = completeHistory[parent1Revision]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
297 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
298 if (parent2Revision != -1) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
299 p2 = completeHistory[parent2Revision]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
300 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
301 completeHistory[revisionNumber] = new HistoryNode(linkRevision, p1, p2); | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
302 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
303 }; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
304 content.iterate(0, getLastRevision(), false, insp); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
305 cancelSupport.checkCancelled(); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
306 if (needsSorting[0]) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
307 Arrays.sort(commitRevisions); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
308 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
309 // read changeset revisions at once (to avoid numerous changelog.getRevision reads) | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
310 // but just nodeids, not RawChangeset (changelog.iterate(data=false) | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
311 ArrayList<Nodeid> changesetRevisions = new ArrayList<Nodeid>(commitRevisions.length); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
312 getRepo().getChangelog().getRevisionsInternal(changesetRevisions, commitRevisions); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
313 cancelSupport.checkCancelled(); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
314 // assign them to corresponding HistoryNodes | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
315 for (int i = 0; i < completeHistory.length; i++ ) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
316 final HistoryNode n = completeHistory[i]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
317 if (needsSorting[0]) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
318 int x = Arrays.binarySearch(commitRevisions, n.changeset); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
319 assert x >= 0; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
320 n.cset = changesetRevisions.get(x); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
321 } else { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
322 // commit revisions were not sorted, may use original index directly | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
323 n.cset = changesetRevisions.get(i); | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
324 } | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
325 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
326 cancelSupport.checkCancelled(); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
327 // XXX shall sort completeHistory according to changeset numbers? | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
328 for (int i = 0; i < completeHistory.length; i++ ) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
329 final HistoryNode n = completeHistory[i]; | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
330 HistoryNode p; | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
331 Nodeid p1, p2; | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
332 if ((p = n.parent1) != null) { | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
333 p1 = p.changesetRevision(); | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
334 } else { | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
335 p1 = Nodeid.NULL; | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
336 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
337 if ((p= n.parent2) != null) { | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
338 p2 = p.changesetRevision(); | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
339 } else { | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
340 p2 = Nodeid.NULL; | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
341 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
342 final Pair<Nodeid, Nodeid> parentChangesets = new Pair<Nodeid, Nodeid>(p1, p2); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
343 final List<Nodeid> childChangesets; | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
344 if (n.children == null) { | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
345 childChangesets = Collections.emptyList(); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
346 } else { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
347 Nodeid[] revisions = new Nodeid[n.children.size()]; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
348 int j = 0; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
349 for (HistoryNode hn : n.children) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
350 revisions[j++] = hn.changesetRevision(); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
351 } | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
352 childChangesets = Arrays.asList(revisions); | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
353 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
354 inspector.next(n.changesetRevision(), parentChangesets, childChangesets); | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
355 cancelSupport.checkCancelled(); | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
356 } | 
| 
317
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
357 } catch (CancelledException ex) { | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
358 return; | 
| 
 
09628675bcee
Rework file history build approach to match rest of the API
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
306 
diff
changeset
 | 
359 } | 
| 
305
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
360 } | 
| 
 
ae8d116f4ee2
Experimental code to build file history
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
300 
diff
changeset
 | 
361 | 
| 
129
 
645829962785
core.Cset renamed to HgChangeset; repo.Changeset moved into HgChangelog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
121 
diff
changeset
 | 
362 public void history(HgChangelog.Inspector inspector) { | 
| 
135
 
3959bffb14e9
explicit op name instead math op to get last rev number
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
134 
diff
changeset
 | 
363 history(0, getLastRevision(), inspector); | 
| 
48
 
e34f90b9ded1
Limit option for history/log
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
37 
diff
changeset
 | 
364 } | 
| 
 
e34f90b9ded1
Limit option for history/log
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
37 
diff
changeset
 | 
365 | 
| 
347
 
8da7ade36c57
Add specific IAE subclass to handle wrong (e.g. outdated after rollback) revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
328 
diff
changeset
 | 
366 public void history(int start, int end, HgChangelog.Inspector inspector) throws HgInvalidRevisionException { | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
367 if (!exists()) { | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
368 throw new IllegalStateException("Can't get history of invalid repository file node"); | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
369 } | 
| 
135
 
3959bffb14e9
explicit op name instead math op to get last rev number
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
134 
diff
changeset
 | 
370 final int last = getLastRevision(); | 
| 
77
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
371 if (end == TIP) { | 
| 
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
372 end = last; | 
| 
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
373 } | 
| 
300
 
650b45d290b1
Share range check code
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
295 
diff
changeset
 | 
374 if (start == TIP) { | 
| 
 
650b45d290b1
Share range check code
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
295 
diff
changeset
 | 
375 start = last; | 
| 
 
650b45d290b1
Share range check code
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
295 
diff
changeset
 | 
376 } | 
| 
 
650b45d290b1
Share range check code
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
295 
diff
changeset
 | 
377 HgInternals.checkRevlogRange(start, end, last); | 
| 
 
650b45d290b1
Share range check code
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
295 
diff
changeset
 | 
378 | 
| 
48
 
e34f90b9ded1
Limit option for history/log
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
37 
diff
changeset
 | 
379 final int[] commitRevisions = new int[end - start + 1]; | 
| 
242
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
380 final boolean[] needsSorting = { false }; | 
| 
77
 
c677e1593919
Moved RevlogStream implementation into .internal
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
74 
diff
changeset
 | 
381 RevlogStream.Inspector insp = new RevlogStream.Inspector() { | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
382 int count = 0; | 
| 
51
 
9429c7bd1920
Try DataAccess to reach revision data instead of plain byte arrays
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
49 
diff
changeset
 | 
383 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess data) { | 
| 
242
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
384 if (count > 0) { | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
385 if (commitRevisions[count -1] > linkRevision) { | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
386 needsSorting[0] = true; | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
387 } | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
388 } | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
389 commitRevisions[count++] = linkRevision; | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
390 } | 
| 
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
391 }; | 
| 
48
 
e34f90b9ded1
Limit option for history/log
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
37 
diff
changeset
 | 
392 content.iterate(start, end, false, insp); | 
| 
233
 
1d389c0cb0a5
Optimize file history walk not to iterat over whole changelog for sparse and distant revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
215 
diff
changeset
 | 
393 final HgChangelog changelog = getRepo().getChangelog(); | 
| 
242
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
394 if (needsSorting[0]) { | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
395 // automatic tools (svnmerge?) produce unnatural file history | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
396 // (e.g. cpython/Lib/doctest.py, revision 164 points to cset 63509, 165 - to 38453) | 
| 
 
ad6a046943be
Improved reading of sparse revisions from a revlog
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
240 
diff
changeset
 | 
397 Arrays.sort(commitRevisions); | 
| 
233
 
1d389c0cb0a5
Optimize file history walk not to iterat over whole changelog for sparse and distant revisions
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
215 
diff
changeset
 | 
398 } | 
| 
245
 
2fb439375ddc
Avoid sorting revision range twice
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
242 
diff
changeset
 | 
399 changelog.rangeInternal(inspector, commitRevisions); | 
| 
3
 
24bb4f365164
Rudimentary log functionality with basic infrastructure is in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
2 
diff
changeset
 | 
400 } | 
| 
88
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
401 | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
402 /** | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
403 * For a given local revision of the file, find out local revision in the changelog. | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
404 * FIXME rename to getChangesetRevisionIndex() | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
405 * | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
406 * @return changeset revision index | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
407 * @throws HgInvalidRevisionException if supplied argument doesn't represent revision index in this revlog | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
408 * @throws HgInvalidControlFileException if access to revlog index/data entry failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
409 */ | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
410 public int getChangesetLocalRevision(int revision) throws HgInvalidControlFileException, HgInvalidRevisionException { | 
| 
88
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
411 return content.linkRevision(revision); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
412 } | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
413 | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
414 /** | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
415 * Complements {@link #getChangesetLocalRevision(int)} to get changeset revision that corresponds to supplied file revision | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
416 * | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
417 * @param nid revision of the file | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
418 * @return changeset revision | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
419 * @throws HgInvalidRevisionException if supplied argument doesn't represent revision index in this revlog | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
420 * @throws HgInvalidControlFileException if access to revlog index/data entry failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
421 */ | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
422 public Nodeid getChangesetRevision(Nodeid nid) throws HgInvalidControlFileException, HgInvalidRevisionException { | 
| 
88
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
423 int changelogRevision = getChangesetLocalRevision(getLocalRevision(nid)); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
424 return getRepo().getChangelog().getRevision(changelogRevision); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
425 } | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
426 | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
427 /** | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
428 * | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
429 * @return | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
430 * @throws HgDataStreamException if attempt to access file metadata failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
431 */ | 
| 
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
 | 
432 public boolean isCopy() throws HgDataStreamException { | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
433 if (metadata == null || !metadata.checked(0)) { | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
434 checkAndRecordMetadata(0); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
435 } | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
436 if (!metadata.known(0)) { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
437 return false; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
438 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
439 return metadata.find(0, "copy") != null; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
440 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
441 | 
| 
354
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
442 /** | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
443 * Get name of the file this one was copied from. | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
444 * | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
445 * @return name of the file origin | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
446 * @throws HgDataStreamException if attempt to access file metadata failed | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
447 * @throws UnsupportedOperationException if this file doesn't represent a copy ({@link #isCopy()} was false) | 
| 
 
5f9073eabf06
Propagate errors with exceptions up to a end client
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
347 
diff
changeset
 | 
448 */ | 
| 
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
 | 
449 public Path getCopySourceName() throws HgDataStreamException { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
450 if (isCopy()) { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
451 return Path.create(metadata.find(0, "copy")); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
452 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
453 throw new UnsupportedOperationException(); // XXX REVISIT, think over if Exception is good (clients would check isCopy() anyway, perhaps null is sufficient?) | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
454 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
455 | 
| 
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
 | 
456 public Nodeid getCopySourceRevision() throws HgDataStreamException { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
457 if (isCopy()) { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
458 return Nodeid.fromAscii(metadata.find(0, "copyrev")); // XXX reuse/cache Nodeid | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
459 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
460 throw new UnsupportedOperationException(); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
461 } | 
| 
88
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
462 | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
463 @Override | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
464 public String toString() { | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
465 StringBuilder sb = new StringBuilder(getClass().getSimpleName()); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
466 sb.append('('); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
467 sb.append(getPath()); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
468 sb.append(')'); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
469 return sb.toString(); | 
| 
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
470 } | 
| 
275
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
471 | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
472 private void checkAndRecordMetadata(int localRev) throws HgDataStreamException { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
473 // content() always initializes metadata. | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
474 // FIXME this is expensive way to find out metadata, distinct RevlogStream.Iterator would be better. | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
475 // Alternatively, may parameterize MetadataContentPipe to do prepare only. | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
476 // For reference, when throwing CancelledException, hg status -A --rev 3:80 takes 70 ms | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
477 // however, if we just consume buffer instead (buffer.position(buffer.limit()), same command takes ~320ms | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
478 // (compared to command-line counterpart of 190ms) | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
479 try { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
480 content(localRev, new ByteChannel() { // No-op channel | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
481 public int write(ByteBuffer buffer) throws IOException, CancelledException { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
482 throw new CancelledException(); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
483 } | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
484 }); | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
485 } catch (CancelledException ex) { | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
486 // it's ok, we did that | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
487 } | 
| 
 
6d1804fe0ed7
Issue 10: Report file content length with respect of metadata. Respect dirstate parents for WC's status. Exceptions to keep useful attributes of the location
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
245 
diff
changeset
 | 
488 } | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
489 | 
| 
88
 
61eedab3eb3e
Status between two revisions to recognize copy/rename
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
79 
diff
changeset
 | 
490 private static final class MetadataEntry { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
491 private final String entry; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
492 private final int valueStart; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
493 /*package-local*/MetadataEntry(String key, String value) { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
494 entry = key + value; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
495 valueStart = key.length(); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
496 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
497 /*package-local*/boolean matchKey(String key) { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
498 return key.length() == valueStart && entry.startsWith(key); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
499 } | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
500 // uncomment once/if needed | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
501 // public String key() { | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
502 // return entry.substring(0, valueStart); | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
503 // } | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
504 public String value() { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
505 return entry.substring(valueStart); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
506 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
507 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
508 | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
509 private static class Metadata { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
510 private static class Record { | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
511 public final int offset; | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
512 public final MetadataEntry[] entries; | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
513 | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
514 public Record(int off, MetadataEntry[] entr) { | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
515 offset = off; | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
516 entries = entr; | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
517 } | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
518 } | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
519 // XXX sparse array needed | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
520 private final IntMap<Record> entries = new IntMap<Record>(5); | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
521 | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
522 private final Record NONE = new Record(-1, null); // don't want statics | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
523 | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
524 // true when there's metadata for given revision | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
525 boolean known(int revision) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
526 Record i = entries.get(revision); | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
527 return i != null && NONE != i; | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
528 } | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
529 | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
530 // true when revision has been checked for metadata presence. | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
531 public boolean checked(int revision) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
532 return entries.containsKey(revision); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
533 } | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
534 | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
535 // true when revision has been checked and found not having any metadata | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
536 boolean none(int revision) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
537 Record i = entries.get(revision); | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
538 return i == NONE; | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
539 } | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
540 | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
541 // mark revision as having no metadata. | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
542 void recordNone(int revision) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
543 Record i = entries.get(revision); | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
544 if (i == NONE) { | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
545 return; // already there | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
546 } | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
547 if (i != null) { | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
548 throw new IllegalStateException(String.format("Trying to override Metadata state for revision %d (known offset: %d)", revision, i)); | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
549 } | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
550 entries.put(revision, NONE); | 
| 
134
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
551 } | 
| 
 
afac8ddc5dd2
Keep record if we tried and found no metadata for a given revision
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
133 
diff
changeset
 | 
552 | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
553 // since this is internal class, callers are supposed to ensure arg correctness (i.e. ask known() before) | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
554 int dataOffset(int revision) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
555 return entries.get(revision).offset; | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
556 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
557 void add(int revision, int dataOffset, Collection<MetadataEntry> e) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
558 assert !entries.containsKey(revision); | 
| 
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
559 entries.put(revision, new Record(dataOffset, e.toArray(new MetadataEntry[e.size()]))); | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
560 } | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
561 | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
562 String find(int revision, String key) { | 
| 
276
 
6355ecda1f08
Tailored Map implementation with int keys
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
275 
diff
changeset
 | 
563 for (MetadataEntry me : entries.get(revision).entries) { | 
| 
78
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
564 if (me.matchKey(key)) { | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
565 return me.value(); | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
566 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
567 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
568 return null; | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
569 } | 
| 
 
c25c5c348d1b
Skip metadata in the beginning of a file content. Parse metadata, recognize copies/renames
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
77 
diff
changeset
 | 
570 } | 
| 
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
 | 
571 | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
572 private static class MetadataInspector extends ErrorHandlingInspector implements RevlogStream.Inspector { | 
| 
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
 | 
573 private final Metadata metadata; | 
| 
215
 
41a778e3fd31
Issue 5: Facilities for progress and cancellation. More specific exceptions. Exceptions from callbacks as RuntimeException
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
163 
diff
changeset
 | 
574 private final Path fname; // need this only for error reporting | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
575 private final RevlogStream.Inspector delegate; | 
| 
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
 | 
576 | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
577 public MetadataInspector(Metadata _metadata, Path file, RevlogStream.Inspector chain) { | 
| 
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
 | 
578 metadata = _metadata; | 
| 
215
 
41a778e3fd31
Issue 5: Facilities for progress and cancellation. More specific exceptions. Exceptions from callbacks as RuntimeException
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
163 
diff
changeset
 | 
579 fname = file; | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
580 delegate = chain; | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
581 setCancelSupport(CancelSupport.Factory.get(chain)); | 
| 
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
 | 
582 } | 
| 
 
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
 | 
583 | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
584 public void next(int revisionNumber, int actualLen, int baseRevision, int linkRevision, int parent1Revision, int parent2Revision, byte[] nodeid, DataAccess data) { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
585 try { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
586 final int daLength = data.length(); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
587 if (daLength < 4 || data.readByte() != 1 || data.readByte() != 10) { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
588 metadata.recordNone(revisionNumber); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
589 data.reset(); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
590 } else { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
591 ArrayList<MetadataEntry> _metadata = new ArrayList<MetadataEntry>(); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
592 int offset = parseMetadata(data, daLength, _metadata); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
593 metadata.add(revisionNumber, offset, _metadata); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
594 // da is in prepared state (i.e. we consumed all bytes up to metadata end). | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
595 // However, it's not safe to assume delegate won't call da.reset() for some reason, | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
596 // and we need to ensure predictable result. | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
597 data.reset(); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
598 data = new FilterDataAccess(data, offset, daLength - offset); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
599 } | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
600 if (delegate != null) { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
601 delegate.next(revisionNumber, actualLen, baseRevision, linkRevision, parent1Revision, parent2Revision, nodeid, data); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
602 } | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
603 } catch (IOException ex) { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
604 recordFailure(ex); | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
605 } catch (HgDataStreamException ex) { | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
606 recordFailure(ex.setRevisionNumber(revisionNumber)); | 
| 
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
 | 
607 } | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
608 } | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
609 | 
| 
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
610 private int parseMetadata(DataAccess data, final int daLength, ArrayList<MetadataEntry> _metadata) throws IOException, HgDataStreamException { | 
| 
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
 | 
611 int lastEntryStart = 2; | 
| 
 
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
 | 
612 int lastColon = -1; | 
| 
 
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
 | 
613 // XXX in fact, need smth like ByteArrayBuilder, similar to StringBuilder, | 
| 
 
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
 | 
614 // which can't be used here because we can't convert bytes to chars as we read them | 
| 
 
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
 | 
615 // (there might be multi-byte encoding), and we need to collect all bytes before converting to string | 
| 
 
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
 | 
616 ByteArrayOutputStream bos = new ByteArrayOutputStream(); | 
| 
 
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
 | 
617 String key = null, value = 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
 | 
618 boolean byteOne = false; | 
| 
323
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
619 boolean metadataIsComplete = false; | 
| 
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
 | 
620 for (int i = 2; i < daLength; i++) { | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
621 byte b = data.readByte(); | 
| 
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
 | 
622 if (b == '\n') { | 
| 
 
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
 | 
623 if (byteOne) { // i.e. \n follows 1 | 
| 
 
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
 | 
624 lastEntryStart = i+1; | 
| 
323
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
625 metadataIsComplete = true; | 
| 
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
 | 
626 // XXX is it possible to have here incomplete key/value (i.e. if last pair didn't end with \n) | 
| 
323
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
627 // if yes, need to set metadataIsComplete to true in that case as well | 
| 
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
 | 
628 break; | 
| 
 
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
 | 
629 } | 
| 
 
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
 | 
630 if (key == null || lastColon == -1 || i <= lastColon) { | 
| 
 
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
 | 
631 throw new IllegalStateException(); // FIXME log instead and record null key in the metadata. Ex just to fail fast during dev | 
| 
 
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
 | 
632 } | 
| 
 
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
 | 
633 value = new String(bos.toByteArray()).trim(); | 
| 
 
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
 | 
634 bos.reset(); | 
| 
 
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
 | 
635 _metadata.add(new MetadataEntry(key, value)); | 
| 
 
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
 | 
636 key = value = 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
 | 
637 lastColon = -1; | 
| 
 
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
 | 
638 lastEntryStart = i+1; | 
| 
 
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
 | 
639 continue; | 
| 
 
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
 | 
640 } | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
641 // byteOne has to be consumed up to this line, if not yet, consume it | 
| 
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
 | 
642 if (byteOne) { | 
| 
 
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
 | 
643 // insert 1 we've read on previous step into the byte builder | 
| 
 
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
 | 
644 bos.write(1); | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
645 byteOne = false; | 
| 
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
 | 
646 // fall-through to consume current byte | 
| 
 
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
 | 
647 } | 
| 
 
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
 | 
648 if (b == (int) ':') { | 
| 
 
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
 | 
649 assert value == 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
 | 
650 key = new String(bos.toByteArray()); | 
| 
 
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
 | 
651 bos.reset(); | 
| 
 
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
 | 
652 lastColon = i; | 
| 
 
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
 | 
653 } else if (b == 1) { | 
| 
 
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
 | 
654 byteOne = true; | 
| 
 
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
 | 
655 } else { | 
| 
 
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
 | 
656 bos.write(b); | 
| 
 
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
 | 
657 } | 
| 
 
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
 | 
658 } | 
| 
323
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
659 // data.isEmpty is not reliable, renamed files of size==0 keep only metadata | 
| 
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
660 if (!metadataIsComplete) { | 
| 
 
4c7e3ba67213
Exception when analyzing metadata of an empty renamed file
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
322 
diff
changeset
 | 
661 // XXX perhaps, worth a testcase (empty file, renamed, read or ask ifCopy | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
662 throw new HgDataStreamException(fname, "Metadata is not closed properly", null); | 
| 
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
 | 
663 } | 
| 
277
 
74e7493a042a
Favor delegation over generalization
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
276 
diff
changeset
 | 
664 return lastEntryStart; | 
| 
17
 
571e1b2cc3f7
Query file for its parents. Demo of recently added ignore and digest support from within cat cmd
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
5 
diff
changeset
 | 
665 } | 
| 
322
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
666 | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
667 @Override | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
668 public void checkFailed() throws HgException, IOException, CancelledException { | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
669 super.checkFailed(); | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
670 if (delegate instanceof ErrorHandlingInspector) { | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
671 // XXX need to add ErrorDestination and pass it around (much like CancelSupport get passed) | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
672 // so that delegate would be able report its failures directly to caller without this hack | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
673 ((ErrorHandlingInspector) delegate).checkFailed(); | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
674 } | 
| 
 
d68dcb3b5f49
Propagate command's CancelSupport to low-level API. CancelSupport from context got priority over one from command
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
317 
diff
changeset
 | 
675 } | 
| 
17
 
571e1b2cc3f7
Query file for its parents. Demo of recently added ignore and digest support from within cat cmd
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
5 
diff
changeset
 | 
676 } | 
| 
2
 
08db726a0fb7
Shaping out low-level Hg structures
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
677 } | 
