måndagen den 2 juni 2008 23.24.40 skrev Marek Zawirski: > --- a/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectIdSubclassMap.java > +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectIdSubclassMap.java > @@ -107,6 +107,15 @@ public class ObjectIdSubclassMap<V extends ObjectId> { > size++; > } > > + /** > + * Returns number of objects in map. > + * > + * @return number of objects in map > + */ > + public int size() { > + return size; > + } Just the @return is usually enough for methods that simply return a value. If you have both the untagged part of the comment should explain much more. Patch 10 is a good example. -- robin -- 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