diff src/org/tmatesoft/hg/repo/HgDataFile.java @ 426:063b0663495a

HgManifest#getFileRevisions refactored into #walkFileRevisions to match pattern throught rest of the library
author Artem Tikhomirov <tikhomirov.artem@gmail.com>
date Wed, 28 Mar 2012 19:34:37 +0200
parents 48f993aa2f41
children 32184ddcf46d
line wrap: on
line diff
--- a/src/org/tmatesoft/hg/repo/HgDataFile.java	Wed Mar 28 18:39:29 2012 +0200
+++ b/src/org/tmatesoft/hg/repo/HgDataFile.java	Wed Mar 28 19:34:37 2012 +0200
@@ -58,7 +58,7 @@
  * @author Artem Tikhomirov
  * @author TMate Software Ltd.
  */
-public class HgDataFile extends Revlog {
+public final class HgDataFile extends Revlog {
 
 	// absolute from repo root?
 	// slashes, unix-style?
@@ -77,12 +77,15 @@
 	}
 
 	// exists is not the best name possible. now it means no file with such name was ever known to the repo.
-	// it might be confused with files existed before but lately removed. 
+	// it might be confused with files existed before but lately removed. TODO HgFileNode.exists makes more sense.
+	// or HgDataFile.known()
 	public boolean exists() {
 		return content != null; // XXX need better impl
 	}
 
-	// human-readable (i.e. "COPYING", not "store/data/_c_o_p_y_i_n_g.i")
+	/**
+	 * Human-readable file name, i.e. "COPYING", not "store/data/_c_o_p_y_i_n_g.i"
+	 */
 	public Path getPath() {
 		return path; // hgRepo.backresolve(this) -> name? In this case, what about hashed long names?
 	}