Robin Rosenberg <robin.rosenberg.lists@xxxxxxxxxx> wrote:
> s?ndag 10 maj 2009 23:30:46 skrev "Shawn O. Pearce" <spearce@xxxxxxxxxxx>:
> > diff --git a/org.spearce.jgit/src/org/spearce/jgit/transport/IndexPack.java b/org.spearce.jgit/src/org/spearce/jgit/transport/IndexPack.java
> > index b2bcbb7..1eb40d4 100644
> > --- a/org.spearce.jgit/src/org/spearce/jgit/transport/IndexPack.java
> > +++ b/org.spearce.jgit/src/org/spearce/jgit/transport/IndexPack.java
> > @@ -1037,6 +1037,14 @@ public PackLock renameAndOpenPack(final String lockMessage)
> > final File finalIdx = new File(packDir, "pack-" + name + ".idx");
> > final PackLock keep = new PackLock(finalPack);
> >
> > + if (!packDir.exists() && !packDir.mkdir() && !packDir.exists()) {
>
> Is there a lot to gain by this instead of just if (!packDir.mkdir() && !packDir.exists()) ?
No, not really. But usually the directory exists, its rare when
it doesn't.
We probably just wasted more CPU time in our MUAs debating this than
it really costs in the long run. I'm too #@!** lazy to go back
and amend this patch. But you can amend it if it really bothers
you to do the "!exists && !mkdir && !exists".
--
Shawn.
--
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