fredagen den 23 maj 2008 22.34.28 skrev Florian Koeberle:
> ---
> .../src/org/spearce/jgit/lib/Repository.java | 57 +++++++++++++++++
> .../src/org/spearce/jgit/lib/WorkTree.java | 67 ++++++++++++++++++++
> 2 files changed, 124 insertions(+), 0 deletions(-)
> create mode 100644 org.spearce.jgit/src/org/spearce/jgit/lib/WorkTree.java
>
> diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java b/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java
> + public Commit setupHEADRef(final String remoteBranch, final String branch)
> + throws IOException {
> + Commit mapCommit = mapCommit(remoteBranch);
> + String refName = Constants.HEADS_PREFIX + "/" + branch;
> + LockFile masterRef = lockRef(refName);
> + try {
> + masterRef.write(mapCommit.getCommitId());
> + masterRef.commit();
> + } finally {
> + masterRef.unlock();
> + }
> + writeSymref(Constants.HEAD, refName);
> + return mapCommit;
> + }
Shouldn't someone be calling this?
> diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/WorkTree.java b/org.spearce.jgit/src/org/spearce/jgit/lib/WorkTree.java
> + /**
> + * Closes the git repository of this project.
> + */
> + public void closeRepository() {
> + repository.close();
> + }
This is redundant. Very few callers need it and those can just as well
call getRepsitory().close() anyway.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
