Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scaladoc doesn't distinguish case class argument lists #4324

Closed
scabug opened this issue Mar 8, 2011 · 4 comments
Closed

Scaladoc doesn't distinguish case class argument lists #4324

scabug opened this issue Mar 8, 2011 · 4 comments
Assignees
Labels
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Mar 8, 2011

E.g. the title of the MouseClicked doc shows

case class MouseClicked (source: Component, point: Point, modifiers: Modifiers, clicks: Int, triggersPopup: Boolean, peer: MouseEvent)

even though peer is actually in a separate argument list. Which is unhelpful since it looks like this title is supposed to show the form of unapply.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Mar 8, 2011

Imported From: https://issues.scala-lang.org/browse/SI-4324?orig=1
Reporter: Jamie Webb (jmawebb)

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Mar 20, 2012

@VladUreche said:
Test case:

case class M(i: Int)(s: String) {
  def f(i1: Int)(s1: String) = ()
}

The only broken signature is the one on the top of the page (case class M(i: Int, s: String) extends Product with Serializable).
Seems related to html generation.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 20, 2012

@scabug scabug closed this Jul 20, 2012
@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Jul 23, 2012

@VladUreche said:
Fixed in 8779ade6.

@scabug scabug added this to the 2.10.0-M5 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.