tree 655781978c99a80c5ae2818644b46f965d23cce6
parent bcac1f07d503acbaff72cae696d1bc6d871bbbfc
author Lars Vogel <Lars.Vogel@vogella.com> 1567412670 +0200
committer Lars Vogel <Lars.Vogel@vogella.com> 1567510280 -0400

Bug 550644 - Remove redundant contains check in
InjectorImpl#processMethods

InjectorImpl#processMethods performs a contains check followed by a get.
As isOverriddenCache does not contain null elements, this check is
redundant.

Some logic applied for isOverriddenCache and methodMap

Manual measurement (see Bug 550208 for missing automatic way of
measuring that) with 5 times started (after two warmups) shows approx.
3.1 % performance improvement.

Old Code:

Application started in : 8919ms
Application started in : 8793ms
Application started in : 8799ms
Application started in : 8581ms
Application started in : 8764ms

Average: 8771,2


With change:
Application started in : 8801ms
Application started in : 8363ms
Application started in : 8246ms
Application started in : 8770ms
Application started in : 8335ms

Average: 8503

Also yourkit shows an improvement of approx. 635ms.


Change-Id: Id5a535899f0b1a7299332c2b5a3c82bd9c3087a5
Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>