Mercurial > jhg
annotate src/org/tmatesoft/hg/internal/AnnotateFacility.java @ 551:4ea0351ca878
Better (precise) name for diff facility, tests
| author | Artem Tikhomirov <tikhomirov.artem@gmail.com> | 
|---|---|
| date | Wed, 20 Feb 2013 18:19:52 +0100 | 
| parents | 83afa680555d | 
| children | 45751456b471 | 
| rev | line source | 
|---|---|
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
1 /* | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
2 * Copyright (c) 2013 TMate Software Ltd | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
3 * | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
4 * This program is free software; you can redistribute it and/or modify | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
5 * it under the terms of the GNU General Public License as published by | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
6 * the Free Software Foundation; version 2 of the License. | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
7 * | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
8 * This program is distributed in the hope that it will be useful, | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
9 * but WITHOUT ANY WARRANTY; without even the implied warranty of | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
11 * GNU General Public License for more details. | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
12 * | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
13 * For information on how to redistribute this software under | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
14 * the terms of a license other than GNU General Public License | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
15 * contact TMate Software at support@hg4j.com | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
16 */ | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
17 package org.tmatesoft.hg.internal; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
18 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
19 import static org.tmatesoft.hg.repo.HgRepository.NO_REVISION; | 
| 
548
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
20 import static org.tmatesoft.hg.repo.HgRepository.TIP; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
21 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
22 import org.tmatesoft.hg.core.Nodeid; | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
23 import org.tmatesoft.hg.internal.DiffHelper.LineSequence; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
24 import org.tmatesoft.hg.repo.HgDataFile; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
25 import org.tmatesoft.hg.repo.HgInvalidStateException; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
26 import org.tmatesoft.hg.util.CancelledException; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
27 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
28 /** | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
29 * | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
30 * @author Artem Tikhomirov | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
31 * @author TMate Software Ltd. | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
32 */ | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
33 @Experimental(reason="work in progress") | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
34 public class AnnotateFacility { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
35 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
36 /** | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
37 * mimic 'hg diff -r csetRevIndex1 -r csetRevIndex2' | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
38 */ | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
39 public void diff(HgDataFile df, int csetRevIndex1, int csetRevIndex2, BlockInspector insp) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
40 int fileRevIndex1 = fileRevIndex(df, csetRevIndex1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
41 int fileRevIndex2 = fileRevIndex(df, csetRevIndex2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
42 LineSequence c1 = lines(df, fileRevIndex1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
43 LineSequence c2 = lines(df, fileRevIndex2); | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
44 DiffHelper<LineSequence> pg = new DiffHelper<LineSequence>(); | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
45 pg.init(c1, c2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
46 pg.findMatchingBlocks(new BlameBlockInspector(insp, csetRevIndex1, csetRevIndex2)); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
47 } | 
| 
548
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
48 | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
49 /** | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
50 * Annotate file revision, line by line. | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
51 */ | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
52 public void annotate(HgDataFile df, int changesetRevisionIndex, LineInspector insp) { | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
53 if (!df.exists()) { | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
54 return; | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
55 } | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
56 int fileRevIndex = fileRevIndex(df, changesetRevisionIndex); | 
| 
548
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
57 int[] fileRevParents = new int[2]; | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
58 FileAnnotation fa = new FileAnnotation(insp); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
59 do { | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
60 // also covers changesetRevisionIndex == TIP, #implAnnotateChange doesn't tolerate constants | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
61 changesetRevisionIndex = df.getChangesetRevisionIndex(fileRevIndex); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
62 df.parents(fileRevIndex, fileRevParents, null, null); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
63 implAnnotateChange(df, changesetRevisionIndex, fileRevIndex, fileRevParents, fa); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
64 fileRevIndex = fileRevParents[0]; | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
65 } while (fileRevIndex != NO_REVISION); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
66 } | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
67 | 
| 
544
 
7f5998a9619d
Refactor PatchGenerator to be generic and welcome sequence of any nature
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
543 
diff
changeset
 | 
68 /** | 
| 
 
7f5998a9619d
Refactor PatchGenerator to be generic and welcome sequence of any nature
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
543 
diff
changeset
 | 
69 * Annotates changes of the file against its parent(s) | 
| 
 
7f5998a9619d
Refactor PatchGenerator to be generic and welcome sequence of any nature
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
543 
diff
changeset
 | 
70 */ | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
71 public void annotateChange(HgDataFile df, int changesetRevisionIndex, BlockInspector insp) { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
72 // TODO detect if file is text/binary (e.g. looking for chars < ' ' and not \t\r\n\f | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
73 int fileRevIndex = fileRevIndex(df, changesetRevisionIndex); | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
74 int[] fileRevParents = new int[2]; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
75 df.parents(fileRevIndex, fileRevParents, null, null); | 
| 
548
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
76 if (changesetRevisionIndex == TIP) { | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
77 changesetRevisionIndex = df.getChangesetRevisionIndex(fileRevIndex); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
78 } | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
79 implAnnotateChange(df, changesetRevisionIndex, fileRevIndex, fileRevParents, insp); | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
80 } | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
81 | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
82 private void implAnnotateChange(HgDataFile df, int csetRevIndex, int fileRevIndex, int[] fileParentRevs, BlockInspector insp) { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
83 final LineSequence fileRevLines = lines(df, fileRevIndex); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
84 if (fileParentRevs[0] != NO_REVISION && fileParentRevs[1] != NO_REVISION) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
85 LineSequence p1Lines = lines(df, fileParentRevs[0]); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
86 LineSequence p2Lines = lines(df, fileParentRevs[1]); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
87 int p1ClogIndex = df.getChangesetRevisionIndex(fileParentRevs[0]); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
88 int p2ClogIndex = df.getChangesetRevisionIndex(fileParentRevs[1]); | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
89 DiffHelper<LineSequence> pg = new DiffHelper<LineSequence>(); | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
90 pg.init(p2Lines, fileRevLines); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
91 EqualBlocksCollector p2MergeCommon = new EqualBlocksCollector(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
92 pg.findMatchingBlocks(p2MergeCommon); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
93 // | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
94 pg.init(p1Lines); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
95 BlameBlockInspector bbi = new BlameBlockInspector(insp, p1ClogIndex, csetRevIndex); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
96 bbi.setMergeParent2(p2MergeCommon, p2ClogIndex); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
97 pg.findMatchingBlocks(bbi); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
98 } else if (fileParentRevs[0] == fileParentRevs[1]) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
99 // may be equal iff both are unset | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
100 assert fileParentRevs[0] == NO_REVISION; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
101 // everything added | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
102 BlameBlockInspector bbi = new BlameBlockInspector(insp, NO_REVISION, csetRevIndex); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
103 bbi.begin(LineSequence.newlines(new byte[0]), fileRevLines); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
104 bbi.match(0, fileRevLines.chunkCount()-1, 0); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
105 bbi.end(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
106 } else { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
107 int soleParent = fileParentRevs[0] == NO_REVISION ? fileParentRevs[1] : fileParentRevs[0]; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
108 assert soleParent != NO_REVISION; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
109 LineSequence parentLines = lines(df, soleParent); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
110 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
111 int parentChangesetRevIndex = df.getChangesetRevisionIndex(soleParent); | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
112 DiffHelper<LineSequence> pg = new DiffHelper<LineSequence>(); | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
113 pg.init(parentLines, fileRevLines); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
114 pg.findMatchingBlocks(new BlameBlockInspector(insp, parentChangesetRevIndex, csetRevIndex)); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
115 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
116 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
117 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
118 private static int fileRevIndex(HgDataFile df, int csetRevIndex) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
119 Nodeid fileRev = df.getRepo().getManifest().getFileRevision(csetRevIndex, df.getPath()); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
120 return df.getRevisionIndex(fileRev); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
121 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
122 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
123 private static LineSequence lines(HgDataFile df, int fileRevIndex) { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
124 try { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
125 ByteArrayChannel c; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
126 df.content(fileRevIndex, c = new ByteArrayChannel()); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
127 return LineSequence.newlines(c.toArray()); | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
128 } catch (CancelledException ex) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
129 // TODO likely it was bad idea to throw cancelled exception from content() | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
130 // deprecate and provide alternative? | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
131 HgInvalidStateException ise = new HgInvalidStateException("ByteArrayChannel never throws CancelledException"); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
132 ise.initCause(ex); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
133 throw ise; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
134 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
135 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
136 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
137 @Callback | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
138 public interface BlockInspector { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
139 void same(EqualBlock block); | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
140 void added(AddBlock block); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
141 void changed(ChangeBlock block); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
142 void deleted(DeleteBlock block); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
143 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
144 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
145 @Callback | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
146 public interface BlockInspectorEx extends BlockInspector { // XXX better name | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
147 // XXX perhaps, shall pass object instead of separate values for future extension? | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
148 void start(int originLineCount, int targetLineCount); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
149 void done(); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
150 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
151 | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
152 public interface Block { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
153 int originChangesetIndex(); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
154 int targetChangesetIndex(); | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
155 // boolean isMergeRevision(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
156 // int fileRevisionIndex(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
157 // int originFileRevisionIndex(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
158 // String[] lines(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
159 // byte[] data(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
160 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
161 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
162 public interface EqualBlock extends Block { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
163 int originStart(); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
164 int targetStart(); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
165 int length(); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
166 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
167 | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
168 public interface AddBlock extends Block { | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
169 int insertedAt(); // line index in the old file | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
170 int firstAddedLine(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
171 int totalAddedLines(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
172 String[] addedLines(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
173 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
174 public interface DeleteBlock extends Block { | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
175 int removedAt(); // line index in the new file | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
176 int firstRemovedLine(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
177 int totalRemovedLines(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
178 String[] removedLines(); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
179 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
180 public interface ChangeBlock extends AddBlock, DeleteBlock { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
181 } | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
182 | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
183 @Callback | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
184 public interface LineInspector { | 
| 
548
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
185 /** | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
186 * Not necessarily invoked sequentially by line numbers | 
| 
 
ab21ac7dd833
Line-by-line annotation API and support code in place
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
546 
diff
changeset
 | 
187 */ | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
188 void line(int lineNumber, int changesetRevIndex, LineDescriptor ld); | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
189 } | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
190 | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
191 public interface LineDescriptor { | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
192 int totalLines(); | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
193 } | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
194 | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
195 | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
196 | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
197 static class BlameBlockInspector extends DiffHelper.DeltaInspector<LineSequence> { | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
198 private final BlockInspector insp; | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
199 private final int csetOrigin; | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
200 private final int csetTarget; | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
201 private EqualBlocksCollector p2MergeCommon; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
202 private int csetMergeParent; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
203 private IntVector mergeRanges; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
204 | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
205 public BlameBlockInspector(BlockInspector inspector, int originCset, int targetCset) { | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
206 assert inspector != null; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
207 insp = inspector; | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
208 csetOrigin = originCset; | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
209 csetTarget = targetCset; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
210 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
211 | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
212 public void setMergeParent2(EqualBlocksCollector p2Merge, int parentCset2) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
213 p2MergeCommon = p2Merge; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
214 csetMergeParent = parentCset2; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
215 mergeRanges = new IntVector(3*10, 3*10); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
216 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
217 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
218 @Override | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
219 public void begin(LineSequence s1, LineSequence s2) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
220 super.begin(s1, s2); | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
221 if (insp instanceof BlockInspectorEx) { | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
222 ((BlockInspectorEx) insp).start(s1.chunkCount() - 1, s2.chunkCount() - 1); | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
223 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
224 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
225 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
226 @Override | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
227 public void end() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
228 super.end(); | 
| 
546
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
229 if(insp instanceof BlockInspectorEx) { | 
| 
 
cd78e8b9d7bc
File annotate test. Refactored FileAnnotation as standalone class, introduced LineInspector to make line offset calc code shared
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
545 
diff
changeset
 | 
230 ((BlockInspectorEx) insp).done(); | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
231 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
232 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
233 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
234 @Override | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
235 protected void changed(int s1From, int s1To, int s2From, int s2To) { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
236 if (p2MergeCommon != null) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
237 mergeRanges.clear(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
238 p2MergeCommon.combineAndMarkRangesWithTarget(s2From, s2To - s2From, csetOrigin, csetMergeParent, mergeRanges); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
239 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
240 /* | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
241 * Usecases: | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
242 * 3 lines changed to 10 lines. range of 10 lines breaks down to 2 from p2, 3 from p1, and 5 from p2. | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
243 * We report: 2 lines changed to 2(p2), then 1 line changed with 3(p1) and 5 lines added from p2. | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
244 * | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
245 * 10 lines changed to 3 lines, range of 3 lines breaks down to 2 line from p1 and 1 line from p2. | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
246 * We report: 2 lines changed to 2(p1) and 8 lines changed to 1(p2) | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
247 */ | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
248 int s1TotalLines = s1To - s1From, s1ConsumedLines = 0, s1Start = s1From; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
249 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
250 for (int i = 0; i < mergeRanges.size(); i += 3) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
251 final int rangeOrigin = mergeRanges.get(i); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
252 final int rangeStart = mergeRanges.get(i+1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
253 final int rangeLen = mergeRanges.get(i+2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
254 final boolean lastRange = i+3 >= mergeRanges.size(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
255 final int s1LinesLeft = s1TotalLines - s1ConsumedLines; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
256 // how many lines we may reported as changed (don't use more than in range unless it's the very last range) | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
257 final int s1LinesToBorrow = lastRange ? s1LinesLeft : Math.min(s1LinesLeft, rangeLen); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
258 if (s1LinesToBorrow > 0) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
259 BlockImpl2 block = new BlockImpl2(seq1, seq2, s1Start, s1LinesToBorrow, rangeStart, rangeLen, s1Start, rangeStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
260 block.setOriginAndTarget(rangeOrigin, csetTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
261 insp.changed(block); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
262 s1ConsumedLines += s1LinesToBorrow; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
263 s1Start += s1LinesToBorrow; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
264 } else { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
265 BlockImpl2 block = getAddBlock(rangeStart, rangeLen, s1Start); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
266 block.setOriginAndTarget(rangeOrigin, csetTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
267 insp.added(block); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
268 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
269 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
270 if (s1ConsumedLines != s1TotalLines) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
271 throw new HgInvalidStateException(String.format("Expected to process %d lines, but actually was %d", s1TotalLines, s1ConsumedLines)); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
272 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
273 } else { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
274 BlockImpl2 block = new BlockImpl2(seq1, seq2, s1From, s1To-s1From, s2From, s2To - s2From, s1From, s2From); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
275 block.setOriginAndTarget(csetOrigin, csetTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
276 insp.changed(block); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
277 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
278 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
279 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
280 @Override | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
281 protected void added(int s1InsertPoint, int s2From, int s2To) { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
282 if (p2MergeCommon != null) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
283 mergeRanges.clear(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
284 p2MergeCommon.combineAndMarkRangesWithTarget(s2From, s2To - s2From, csetOrigin, csetMergeParent, mergeRanges); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
285 int insPoint = s1InsertPoint; // track changes to insertion point | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
286 for (int i = 0; i < mergeRanges.size(); i += 3) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
287 int rangeOrigin = mergeRanges.get(i); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
288 int rangeStart = mergeRanges.get(i+1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
289 int rangeLen = mergeRanges.get(i+2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
290 BlockImpl2 block = getAddBlock(rangeStart, rangeLen, insPoint); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
291 block.setOriginAndTarget(rangeOrigin, csetTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
292 insp.added(block); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
293 // indicate insPoint moved down number of lines we just reported | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
294 insPoint += rangeLen; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
295 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
296 } else { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
297 BlockImpl2 block = getAddBlock(s2From, s2To - s2From, s1InsertPoint); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
298 block.setOriginAndTarget(csetOrigin, csetTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
299 insp.added(block); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
300 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
301 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
302 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
303 @Override | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
304 protected void deleted(int s2DeletePoint, int s1From, int s1To) { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
305 BlockImpl2 block = new BlockImpl2(seq1, null, s1From, s1To - s1From, -1, -1, -1, s2DeletePoint); | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
306 block.setOriginAndTarget(csetOrigin, csetTarget); | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
307 insp.deleted(block); | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
308 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
309 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
310 @Override | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
311 protected void unchanged(int s1From, int s2From, int length) { | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
312 BlockImpl1 block = new BlockImpl1(s1From, s2From, length); | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
313 block.setOriginAndTarget(csetOrigin, csetTarget); | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
314 insp.same(block); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
315 } | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
316 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
317 private BlockImpl2 getAddBlock(int start, int len, int insPoint) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
318 return new BlockImpl2(null, seq2, -1, -1, start, len, insPoint, -1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
319 } | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
320 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
321 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
322 static class BlockImpl implements Block { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
323 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
324 private int originCset; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
325 private int targetCset; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
326 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
327 void setOriginAndTarget(int originChangesetIndex, int targetChangesetIndex) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
328 // XXX perhaps, shall be part of Inspector API, rather than Block's | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
329 // as they don't change between blocks (although the moment about merged revisions) | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
330 // is not yet clear to me | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
331 originCset = originChangesetIndex; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
332 targetCset = targetChangesetIndex; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
333 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
334 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
335 public int originChangesetIndex() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
336 return originCset; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
337 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
338 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
339 public int targetChangesetIndex() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
340 return targetCset; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
341 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
342 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
343 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
344 static class BlockImpl1 extends BlockImpl implements EqualBlock { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
345 private final int start1, start2; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
346 private final int length; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
347 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
348 BlockImpl1(int blockStartSeq1, int blockStartSeq2, int blockLength) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
349 start1 = blockStartSeq1; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
350 start2 = blockStartSeq2; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
351 length = blockLength; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
352 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
353 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
354 public int originStart() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
355 return start1; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
356 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
357 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
358 public int targetStart() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
359 return start2; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
360 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
361 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
362 public int length() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
363 return length; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
364 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
365 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
366 @Override | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
367 public String toString() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
368 return String.format("@@ [%d..%d) == [%d..%d) @@", start1, start1+length, start2, start2+length); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
369 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
370 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
371 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
372 static class BlockImpl2 extends BlockImpl implements ChangeBlock { | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
373 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
374 private final LineSequence oldSeq; | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
375 private final LineSequence newSeq; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
376 private final int s1Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
377 private final int s1Len; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
378 private final int s2Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
379 private final int s2Len; | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
380 private final int s1InsertPoint; | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
381 private final int s2DeletePoint; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
382 | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
383 public BlockImpl2(LineSequence s1, LineSequence s2, int s1Start, int s1Len, int s2Start, int s2Len, int s1InsertPoint, int s2DeletePoint) { | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
384 oldSeq = s1; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
385 newSeq = s2; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
386 this.s1Start = s1Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
387 this.s1Len = s1Len; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
388 this.s2Start = s2Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
389 this.s2Len = s2Len; | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
390 this.s1InsertPoint = s1InsertPoint; | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
391 this.s2DeletePoint = s2DeletePoint; | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
392 } | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
393 | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
394 public int insertedAt() { | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
395 return s1InsertPoint; | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
396 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
397 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
398 public int firstAddedLine() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
399 return s2Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
400 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
401 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
402 public int totalAddedLines() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
403 return s2Len; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
404 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
405 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
406 public String[] addedLines() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
407 return generateLines(totalAddedLines(), firstAddedLine()); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
408 } | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
409 | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
410 public int removedAt() { | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
411 return s2DeletePoint; | 
| 
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
412 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
413 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
414 public int firstRemovedLine() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
415 return s1Start; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
416 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
417 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
418 public int totalRemovedLines() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
419 return s1Len; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
420 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
421 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
422 public String[] removedLines() { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
423 return generateLines(totalRemovedLines(), firstRemovedLine()); | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
424 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
425 | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
426 private String[] generateLines(int count, int startFrom) { | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
427 String[] rv = new String[count]; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
428 for (int i = 0; i < count; i++) { | 
| 
543
 
1e95f48d9886
Report line index for insertion and deletion, test against 'hg diff' output
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
542 
diff
changeset
 | 
429 rv[i] = String.format("LINE %d", startFrom + i+1); | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
430 } | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
431 return rv; | 
| 
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
432 } | 
| 
545
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
433 | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
434 @Override | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
435 public String toString() { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
436 if (s2DeletePoint == -1) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
437 return String.format("@@ -%d,0 +%d,%d @@", insertedAt(), firstAddedLine(), totalAddedLines()); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
438 } else if (s1InsertPoint == -1) { | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
439 // delete only | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
440 return String.format("@@ -%d,%d +%d,0 @@", firstRemovedLine(), totalRemovedLines(), removedAt()); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
441 } | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
442 return String.format("@@ -%d,%d +%d,%d @@", firstRemovedLine(), totalRemovedLines(), firstAddedLine(), totalAddedLines()); | 
| 
 
15b406c7cd9d
First round of annotate file is functional
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
544 
diff
changeset
 | 
443 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
444 } | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
445 | 
| 
551
 
4ea0351ca878
Better (precise) name for diff facility, tests
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
549 
diff
changeset
 | 
446 static class EqualBlocksCollector implements DiffHelper.MatchInspector<LineSequence> { | 
| 
549
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
447 private final IntVector matches = new IntVector(10*3, 2*3); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
448 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
449 public void begin(LineSequence s1, LineSequence s2) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
450 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
451 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
452 public void match(int startSeq1, int startSeq2, int matchLength) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
453 matches.add(startSeq1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
454 matches.add(startSeq2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
455 matches.add(matchLength); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
456 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
457 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
458 public void end() { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
459 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
460 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
461 // true when specified line in origin is equal to a line in target | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
462 public boolean includesOriginLine(int ln) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
463 return includes(ln, 0); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
464 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
465 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
466 // true when specified line in target is equal to a line in origin | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
467 public boolean includesTargetLine(int ln) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
468 return includes(ln, 1); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
469 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
470 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
471 public void intersectWithTarget(int start, int length, IntVector result) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
472 int s = start; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
473 for (int l = start, x = start + length; l < x; l++) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
474 if (!includesTargetLine(l)) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
475 if (l - s > 0) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
476 result.add(s); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
477 result.add(l - s); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
478 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
479 s = l+1; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
480 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
481 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
482 if (s < start+length) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
483 result.add(s); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
484 result.add((start + length) - s); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
485 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
486 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
487 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
488 /* | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
489 * intersects [start..start+length) with ranges of target lines, and based on the intersection | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
490 * breaks initial range into smaller ranges and records them into result, with marker to indicate | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
491 * whether the range is from initial range (markerSource) or is a result of the intersection with target | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
492 * (markerTarget) | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
493 */ | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
494 public void combineAndMarkRangesWithTarget(int start, int length, int markerSource, int markerTarget, IntVector result) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
495 int sourceStart = start, targetStart = start, sourceEnd = start + length; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
496 for (int l = sourceStart; l < sourceEnd; l++) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
497 if (includesTargetLine(l)) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
498 // l is from target | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
499 if (sourceStart < l) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
500 // few lines from source range were not in the target, report them | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
501 result.add(markerSource); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
502 result.add(sourceStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
503 result.add(l - sourceStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
504 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
505 // indicate the earliest line from source range to use | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
506 sourceStart = l + 1; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
507 } else { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
508 // l is not in target | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
509 if (targetStart < l) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
510 // report lines from target range | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
511 result.add(markerTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
512 result.add(targetStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
513 result.add(l - targetStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
514 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
515 // next line *may* be from target | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
516 targetStart = l + 1; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
517 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
518 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
519 // if source range end with line from target, sourceStart would be == sourceEnd, and we need to add range with markerTarget | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
520 // if source range doesn't end with target line, targetStart == sourceEnd, while sourceStart < sourceEnd | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
521 if (sourceStart < sourceEnd) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
522 assert targetStart == sourceEnd; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
523 // something left from the source range | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
524 result.add(markerSource); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
525 result.add(sourceStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
526 result.add(sourceEnd - sourceStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
527 } else if (targetStart < sourceEnd) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
528 assert sourceStart == sourceEnd; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
529 result.add(markerTarget); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
530 result.add(targetStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
531 result.add(sourceEnd - targetStart); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
532 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
533 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
534 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
535 private boolean includes(int ln, int o) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
536 for (int i = 2; i < matches.size(); o += 3, i+=3) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
537 int rangeStart = matches.get(o); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
538 if (rangeStart > ln) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
539 return false; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
540 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
541 int rangeLen = matches.get(i); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
542 if (rangeStart + rangeLen > ln) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
543 return true; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
544 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
545 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
546 return false; | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
547 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
548 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
549 | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
550 public static void main(String[] args) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
551 EqualBlocksCollector bc = new EqualBlocksCollector(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
552 bc.match(-1, 5, 3); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
553 bc.match(-1, 10, 2); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
554 bc.match(-1, 15, 3); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
555 bc.match(-1, 20, 3); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
556 assert !bc.includesTargetLine(4); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
557 assert bc.includesTargetLine(7); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
558 assert !bc.includesTargetLine(8); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
559 assert bc.includesTargetLine(10); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
560 assert !bc.includesTargetLine(12); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
561 IntVector r = new IntVector(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
562 bc.intersectWithTarget(7, 10, r); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
563 for (int i = 0; i < r.size(); i+=2) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
564 System.out.printf("[%d..%d) ", r.get(i), r.get(i) + r.get(i+1)); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
565 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
566 System.out.println(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
567 r.clear(); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
568 bc.combineAndMarkRangesWithTarget(0, 16, 508, 514, r); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
569 for (int i = 0; i < r.size(); i+=3) { | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
570 System.out.printf("%d:[%d..%d) ", r.get(i), r.get(i+1), r.get(i+1) + r.get(i+2)); | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
571 } | 
| 
 
83afa680555d
Annotate merge revision (combined diff against two parents without looking further)
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents: 
548 
diff
changeset
 | 
572 } | 
| 
542
 
a71a05ec11bc
Towards annotate/blame support: general outline of the functionality
 
Artem Tikhomirov <tikhomirov.artem@gmail.com> 
parents:  
diff
changeset
 | 
573 } | 
