diff options
author | Raphael Moll <ralf@android.com> | 2011-08-05 15:33:31 -0700 |
---|---|---|
committer | Raphael Moll <ralf@android.com> | 2011-08-05 16:31:47 -0700 |
commit | 1b8141546746d907e4d310152456f8068d559a9c (patch) | |
tree | 37a7fa9e6dff022d9afa40189eb736d53f5c070a /eclipse/buildConfig | |
parent | 258788eea05d6a5ebeb4f9947ae4d3060744985b (diff) | |
download | sdk-1b8141546746d907e4d310152456f8068d559a9c.zip sdk-1b8141546746d907e4d310152456f8068d559a9c.tar.gz sdk-1b8141546746d907e4d310152456f8068d559a9c.tar.bz2 |
SdkMan2: Fix edge case when install/delete packages.
One issue when installing or deleting packages is that
sometimes after an install the display would show both
the uninstalled and installed package. And when deleting
it would also show both states.
The core issue is that it's a bad idea to rely on
Set<>.contains() when your objects equality isn't quite
what the hash code says.
Or said otherwise, there's more than one definition
of "equals()" for Package objects. That is when a
package is installed or deleted, its internal state
changes and its archive changes (it looses or gain an
OS path), which changes the overall hash code but
at that point it's the same object from the user point
of view.
Anyway, long story made short: this fixes that.
Change-Id: I6ec705cc807d96d097cf9ea7faf8234f7ed34416
Diffstat (limited to 'eclipse/buildConfig')
0 files changed, 0 insertions, 0 deletions