Skip to content

Do not use ListBuffer#readOnly#2095

Merged
jsuereth merged 1 commit intosbt:0.13from
adriaanm:ListBuffer
Jul 10, 2015
Merged

Do not use ListBuffer#readOnly#2095
jsuereth merged 1 commit intosbt:0.13from
adriaanm:ListBuffer

Conversation

@adriaanm
Copy link
Contributor

@adriaanm adriaanm commented Jul 9, 2015

It's dangerous, deprecated, and was removed in 2.12.0-M1.
See scala/scala#4140.

ListBuffer#toList has equivalent performance,
except it actually returns an immutable copy(-on-write).

This turned up while trying to get the community build to pass for 2.12.0-M2

It's dangerous, deprecated, and was removed in 2.12.0-M1.
See scala/scala#4140.

`ListBuffer#toList` has equivalent performance,
except it actually returns an immutable copy(-on-write).
@eed3si9n eed3si9n added the ready label Jul 9, 2015
@typesafe-tools
Copy link

Can one of the admins verify this patch?

@eed3si9n
Copy link
Member

eed3si9n commented Jul 9, 2015

LGTM pending Travis

jsuereth added a commit that referenced this pull request Jul 10, 2015
Do not use `ListBuffer#readOnly`
@jsuereth jsuereth merged commit b52f115 into sbt:0.13 Jul 10, 2015
@jsuereth jsuereth removed the ready label Jul 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants