Skip to content

"Concise and powerful. (...)" Java example in "See Scala in action" scroller has a bug. #2

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

Closed
sstucki opened this issue Mar 1, 2013 · 1 comment
Assignees
Labels

Comments

@sstucki
Copy link
Contributor

sstucki commented Mar 1, 2013

I think that the Java example in the "Concise and powerful. (...)" slide is not doing what the corresponding Scala code is doing. The last line might overwrite the contents of minors (which at this point is an alias to people) with that of adultsList.

Fix: replace the last two lines with

minors = minorsList.toArray(new Person[minorsList.size()])
adults = adultsList.toArray(new Person[adultsList.size()])
@heathermiller
Copy link
Member

This slider thing doesn't exist anymore yay :)

sjrd added a commit that referenced this issue Nov 29, 2013
Wording tweaks to the scala.js announcement
heathermiller pushed a commit that referenced this issue Jul 20, 2016
SIP meeting summary - updating filename, additions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants