search

material-motion / material-motion-android

Reactive motion for Android. Deprecated; please use the Motion library in Material Components for Android instead: https://material.io/develop/android/theming/motion/.
Apache License 2.0
444 stars 26 forks source link

Use WeakHashMap for MotionRuntime's target-based caches #78

Closed randcode-generator closed 7 years ago

randcode-generator commented 7 years ago

Summary: Use WeakHashMap instead of SimpleArrayMap

Fixes https://github.com/material-motion/material-motion-android/issues/76

Reviewers: O2 Material Motion, O6 Material Android platform reviewers, #material_motion, markwei

Reviewed By: O2 Material Motion, O6 Material Android platform reviewers, #material_motion, markwei

Tags: #material_motion

Differential Revision: http://codereview.cc/D3102

codecov[bot] commented 7 years ago

Codecov Report

Merging #78 into develop will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #78   +/-   ##
========================================
  Coverage    25.46%   25.46%           
========================================
  Files           70       70           
  Lines          962      962           
  Branches        65       65           
========================================
  Hits           245      245           
  Misses         714      714           
  Partials         3        3
Impacted Files Coverage Δ
.../google/android/material/motion/MotionRuntime.java 48.38% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0ff7ba5...0c1c4d4. Read the comment docs.