[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [equinox-dev] StateHelperImpl.getExportedPackageMap: map/set initial capacity

Please open a bug and supply a patch with a suggested improvement.


-----equinox-dev-bounces@xxxxxxxxxxx wrote: -----

To: "equinox-dev@xxxxxxxxxxx" <equinox-dev@xxxxxxxxxxx>
From: "Ivanov, Radoslav" <radoslav.ivanov@xxxxxxx>
Sent by: equinox-dev-bounces@xxxxxxxxxxx
Date: 01/24/2011 07:14AM
Subject: [equinox-dev] StateHelperImpl.getExportedPackageMap: map/set initial capacity

I am writing with regards to method “org.eclipse.osgi.internal.resolver.StateHelperImpl.getExportedPackageMap”, which constructs Map and Set with initial capacity. Is there a particular reason for constructing them in that way?
For example,
Map<String, Set<ExportPackageDescription>> result = new HashMap<String, Set<ExportPackageDescription>>(11);
exports = new HashSet<ExportPackageDescription>(1);
IMHO, in most cases, it causes array resizing, which, I saw, reflects to performance analysis.
Best regards
Radoslav Ivanov
equinox-dev mailing list