)]}'
{
  "commit": "26f3f7d57bd7e5bab18af5b239c0a93d2fc61922",
  "tree": "fd2e8e6845ab3c7987ac55ea77b8e116836c55a0",
  "parents": [
    "9903a538f6bad88aaeead9d9a04bd9dff961312f"
  ],
  "author": {
    "name": "Pierre-Charles David",
    "email": "pierre-charles.david@obeo.fr",
    "time": "Wed Feb 14 08:38:53 2018 +0100"
  },
  "committer": {
    "name": "Pierre-Charles David",
    "email": "pierre-charles.david@obeo.fr",
    "time": "Fri Apr 13 12:10:45 2018 +0200"
  },
  "message": "[501508] Remove assumption about model implementation class name\n\nGraphicalEditPart.getAccessibleEditPart() assumes all the\nimplementation classes of the semantic model elements end with the\n\"Impl\" suffix. This is not necessarily true if the EMF generators have\nbeen customized, or if sub-classes with different names are used to\nisolate non-generated code (for example Sirius uses XXXSpec classes\nwhich override the generated XXXImpl with custom code).\n\nWhen called on a class which does not match this assumption,\ngetAccessibleEditPart() crashes with a StringIndexOutOfBoundsException\nbecause it fails to check the result of indexOf().\n\nThis code branch is only used as a fallback when a proper label is not\navailable, and the EClass name is certainly not worse than a raw Java\nclass name.\n\nBug: 501508\nChange-Id: Iea77f77a64561a1c9dcf7f76a2e4da2c3c58968a\nSigned-off-by: Pierre-Charles David \u003cpierre-charles.david@obeo.fr\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "5cbc29de0f6405ec2f803a392e01fd60eecce737",
      "old_mode": 33188,
      "old_path": "org.eclipse.gmf.runtime.diagram.ui/src/org/eclipse/gmf/runtime/diagram/ui/editparts/GraphicalEditPart.java",
      "new_id": "cdfa463b16710142b86b5b76ced0454c4c0376be",
      "new_mode": 33188,
      "new_path": "org.eclipse.gmf.runtime.diagram.ui/src/org/eclipse/gmf/runtime/diagram/ui/editparts/GraphicalEditPart.java"
    }
  ]
}
