Skip to content

Enable sc:compile for stdlib package scala.collection.mutable#25860

Open
cheeseng wants to merge 1 commit intoscala:mainfrom
artimahub:feature-sc-compile-for-scala-collection-mutable
Open

Enable sc:compile for stdlib package scala.collection.mutable#25860
cheeseng wants to merge 1 commit intoscala:mainfrom
artimahub:feature-sc-compile-for-scala-collection-mutable

Conversation

@cheeseng
Copy link
Copy Markdown
Contributor

Added sc:compile for classes under scala.collection.mutable .

@cheeseng cheeseng requested a review from a team as a code owner April 19, 2026 02:34
@cheeseng cheeseng force-pushed the feature-sc-compile-for-scala-collection-mutable branch from 78aa928 to 7c6005c Compare April 19, 2026 02:35
@@ -55,7 +55,12 @@ trait Builder[-A, +To] extends Growable[A] { self: Builder[A, To]^ =>
* This method provides a hint only if the collection has a known size,
* as specified by the following pseudocode:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick: it's actual code not pseudocode now?

@@ -411,7 +411,10 @@ private[collection] object HashTable {
* The goal is to distribute across bins as well as possible even if a hash code has low entropy at some bits.
* <p/>
* OLD VERSION - quick, but bad for sequence 0-10000 - little entropy in higher bits - since 2003
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this still belong here?

…n scaladoc of improve method mutable.HashTable which seems to be outdated.
@cheeseng cheeseng force-pushed the feature-sc-compile-for-scala-collection-mutable branch from 7c6005c to 5f5554a Compare April 23, 2026 13:29
@cheeseng
Copy link
Copy Markdown
Contributor Author

@SethTisue I adjusted and squashed the commit into one, can you see if this is fit enough now?

Thanks.

@SethTisue SethTisue self-assigned this Apr 23, 2026
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.

2 participants