Re: [JGIT PATCH v3 09/23] Added a Rules interface implementation and a factory for it.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



fredagen den 23 maj 2008 22.34.18 skrev Florian Koeberle:
> +	/**
> +	 * @see Rules#toIgnore(java.lang.String, boolean)
> +	 */
> +	public boolean toIgnore(String fileName, boolean fileIsDirectory) {

(Applies to many places)
Don't make javadoc style comments for overrides. Rather just omit it
or use a plain comment. Also use the @Override annotation if you
override an implementation. We use Java 5, so we may as well use
all bells and whistles that make sense.

I'm not sure about other IDE's, but Eclipse will automaticall picks up then
javadoc for the superclass or interface that declare the methos being 
overridden if the overriding method does not hava a javadoc comment.
The javadoc tool also does this so I assume any descent IDE
does too (if not it's a bug).

-- 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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux