Re: [equinox-dev] StateHelperImpl.getExportedPackageMap: map/set initial

[Thomas Watson <tjwatson@xxxxxxxxxx>]

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

Tom

-----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

Hello

 

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);

or

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
equinox-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/equinox-dev