)]}'
{
  "commit": "0b85841cbaaacb2baa54aab25114eb285e2dc70e",
  "tree": "655781978c99a80c5ae2818644b46f965d23cce6",
  "parents": [
    "bcac1f07d503acbaff72cae696d1bc6d871bbbfc"
  ],
  "author": {
    "name": "Lars Vogel",
    "email": "Lars.Vogel@vogella.com",
    "time": "Mon Sep 02 10:24:30 2019 +0200"
  },
  "committer": {
    "name": "Lars Vogel",
    "email": "Lars.Vogel@vogella.com",
    "time": "Tue Sep 03 07:31:20 2019 -0400"
  },
  "message": "Bug 550644 - Remove redundant contains check in\nInjectorImpl#processMethods\n\nInjectorImpl#processMethods performs a contains check followed by a get.\nAs isOverriddenCache does not contain null elements, this check is\nredundant.\n\nSome logic applied for isOverriddenCache and methodMap\n\nManual measurement (see Bug 550208 for missing automatic way of\nmeasuring that) with 5 times started (after two warmups) shows approx.\n3.1 % performance improvement.\n\nOld Code:\n\nApplication started in : 8919ms\nApplication started in : 8793ms\nApplication started in : 8799ms\nApplication started in : 8581ms\nApplication started in : 8764ms\n\nAverage: 8771,2\n\n\nWith change:\nApplication started in : 8801ms\nApplication started in : 8363ms\nApplication started in : 8246ms\nApplication started in : 8770ms\nApplication started in : 8335ms\n\nAverage: 8503\n\nAlso yourkit shows an improvement of approx. 635ms.\n\n\nChange-Id: Id5a535899f0b1a7299332c2b5a3c82bd9c3087a5\nSigned-off-by: Lars Vogel \u003cLars.Vogel@vogella.com\u003e",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "44e133bdc8fd785252b2dad1899be3ddac081081",
      "old_mode": 33188,
      "old_path": "bundles/org.eclipse.e4.core.di/src/org/eclipse/e4/core/internal/di/InjectorImpl.java",
      "new_id": "3d7d239acfb6e5cb206c8ecde6bf14115186246a",
      "new_mode": 33188,
      "new_path": "bundles/org.eclipse.e4.core.di/src/org/eclipse/e4/core/internal/di/InjectorImpl.java"
    }
  ]
}
