Mercurial > jhg
comparison src/org/tmatesoft/hg/repo/HgRepository.java @ 418:528b6780a8bd
A bit of FIXME cleanup (mostly degraded to TODO post 1.0), comments and javadoc
| author | Artem Tikhomirov <tikhomirov.artem@gmail.com> |
|---|---|
| date | Thu, 22 Mar 2012 21:02:20 +0100 |
| parents | bb278ccf9866 |
| children | fdd7d756dea0 |
comparison
equal
deleted
inserted
replaced
| 417:ccd7d25e5aea | 418:528b6780a8bd |
|---|---|
| 178 public HgTags getTags() throws HgInvalidControlFileException { | 178 public HgTags getTags() throws HgInvalidControlFileException { |
| 179 if (tags == null) { | 179 if (tags == null) { |
| 180 tags = new HgTags(this); | 180 tags = new HgTags(this); |
| 181 HgDataFile hgTags = getFileNode(".hgtags"); | 181 HgDataFile hgTags = getFileNode(".hgtags"); |
| 182 if (hgTags.exists()) { | 182 if (hgTags.exists()) { |
| 183 for (int i = 0; i <= hgTags.getLastRevision(); i++) { // FIXME in fact, would be handy to have walk(start,end) | 183 for (int i = 0; i <= hgTags.getLastRevision(); i++) { // TODO post-1.0 in fact, would be handy to have walk(start,end) |
| 184 // method for data files as well, though it looks odd. | 184 // method for data files as well, though it looks odd. |
| 185 try { | 185 try { |
| 186 ByteArrayChannel sink = new ByteArrayChannel(); | 186 ByteArrayChannel sink = new ByteArrayChannel(); |
| 187 hgTags.content(i, sink); | 187 hgTags.content(i, sink); |
| 188 final String content = new String(sink.toArray(), "UTF8"); | 188 final String content = new String(sink.toArray(), "UTF8"); |
| 313 // shall be of use only for internal classes | 313 // shall be of use only for internal classes |
| 314 /*package-local*/ File getRepositoryRoot() { | 314 /*package-local*/ File getRepositoryRoot() { |
| 315 return repoDir; | 315 return repoDir; |
| 316 } | 316 } |
| 317 | 317 |
| 318 // FIXME remove once NPE in HgWorkingCopyStatusCollector.areTheSame is solved | |
| 319 /*package-local, debug*/String getStoragePath(HgDataFile df) { | 318 /*package-local, debug*/String getStoragePath(HgDataFile df) { |
| 319 // may come handy for debug | |
| 320 return dataPathHelper.rewrite(df.getPath().toString()).toString(); | 320 return dataPathHelper.rewrite(df.getPath().toString()).toString(); |
| 321 } | 321 } |
| 322 | 322 |
| 323 // XXX package-local, unless there are cases when required from outside (guess, working dir/revision walkers may hide dirstate access and no public visibility needed) | 323 // XXX package-local, unless there are cases when required from outside (guess, working dir/revision walkers may hide dirstate access and no public visibility needed) |
| 324 // XXX consider passing Path pool or factory to produce (shared) Path instead of Strings | 324 // XXX consider passing Path pool or factory to produce (shared) Path instead of Strings |
