On Fri, Mar 22, 2013 at 4:35 PM, Mirco Dotta <notifications@github.com>wrote:

> In org.scala-ide.sdt.core/src/scala/tools/eclipse/Scaladoc.scala:
>
> > +    def htmlContents(header: String, comment: Comment): NodeSeq = {
> > +      val headerHtml =
> > +        if (header.isEmpty) NodeSeq.Empty
> > +        else
> > +          <h3>{ header }</h3>
> > +
> > +      val mainHtml = List(
> > +        bodyToHtml(comment.body),
> > +        bodiesToHtml("Example" + (if (comment.example.length > 1) "s" else ""), comment.example),
> > +        bodiesToHtml("Version", comment.version.toList),
> > +        bodiesToHtml("Since", comment.since.toList),
> > +        bodiesToHtml("Note", comment.note),
> > +        bodiesToHtml("See also", comment.see),
> > +        bodiesToHtml("To do", comment.todo),
> > +        bodiesToHtml("Deprecated", comment.deprecated.toList),
> > +        bodiesToHtml("Exceptions Thrown", comment.throws.toList.sortBy(_._1).map {
>
> +1 I think we should have params and return type as well.
>
Bot sure about @return, but params include implicit things that shouldn't
be mentioned I think. Look at all 'implicit cbf: CanBuildFrom' parameters
from stdlib, that are documented. So I vote against adding them...

> —
> Reply to this email directly or view it on GitHub<https://github.com/scala-ide/scala-ide/pull/336/files#r3488438>
> .
>