Skip to content

Refactor: replace HashMap with ContainerUtil.newHashMap in src and testSrc#8704

Closed
jwren wants to merge 2 commits intoflutter:mainfrom
jwren:api-02
Closed

Refactor: replace HashMap with ContainerUtil.newHashMap in src and testSrc#8704
jwren wants to merge 2 commits intoflutter:mainfrom
jwren:api-02

Conversation

@jwren
Copy link
Copy Markdown
Member

@jwren jwren commented Jan 9, 2026

Optimizes memory usage and ensures consistency with IntelliJ Platform APIs by replacing standard java.util.HashMap instantiations with com.intellij.util.containers.ContainerUtil.newHashMap(). This migration covers:

  • src files (ElementIO, FlutterSdk, FlutterApp, VmServiceWrapper, etc.)
  • testSrc files (FlutterPositionMapperTest)
  • Added necessary imports for ContainerUtil. Verified with ./gradlew testClasses and ./gradlew verifyPlugin.

…stSrc

Optimizes memory usage and ensures consistency with IntelliJ Platform APIs
by replacing standard java.util.HashMap instantiations with
com.intellij.util.containers.ContainerUtil.newHashMap().
This migration covers:
- src files (ElementIO, FlutterSdk, FlutterApp, VmServiceWrapper, etc.)
- testSrc files (FlutterPositionMapperTest)
- Added necessary imports for ContainerUtil.
Verified with ./gradlew testClasses and ./gradlew verifyPlugin.
@jwren jwren closed this Jan 28, 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.

1 participant