)]}'
{
  "commit": "7f92ed6b41c441ab2111c9f1212cbbc13222edcc",
  "tree": "a710067f83d4d11fd690a7cfc0d305e2689b2400",
  "parents": [
    "fde1a9a3ee8c95bb7bae6c764755bcbffdbb0308"
  ],
  "author": {
    "name": "Tom de Vries",
    "email": "tdevries@suse.de",
    "time": "Mon Sep 27 14:10:39 2021 +0200"
  },
  "committer": {
    "name": "Tom de Vries",
    "email": "tdevries@suse.de",
    "time": "Mon Sep 27 14:10:39 2021 +0200"
  },
  "message": "[gdb/build] Fix build with g++-4.8\n\nWhen building g++-4.8, we run into:\n...\nsrc/gdb/dwarf2/read.c:919:5: error: multiple fields in union \\\n  \u0027partial_die_info::\u003canonymous union\u003e\u0027 initialized\n...\n\nThis is due to:\n...\n    union\n    {\n      struct\n      {\n       CORE_ADDR lowpc \u003d 0;\n       CORE_ADDR highpc \u003d 0;\n      };\n      ULONGEST ranges_offset;\n    };\n...\n\nThe error looks incorrect, given that only one union member is initialized,\nand does not reproduce with newer g++.\n\nNevertheless, work around this by moving the initialization to a constructor.\n\n[ I considered just removing the initialization, with the idea that access\nshould be guarded by has_pc_info, but I ran into one failure in the testsuite,\nfor gdb.base/check-psymtab.exp due to add_partial_symbol using lowpc without\nchecking has_pc_info. ]\n\nTested on x86_64-linux.\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "d2501c9dd5624501ace63e75719cb3490f6b2189",
      "old_mode": 33188,
      "old_path": "gdb/dwarf2/read.c",
      "new_id": "00aa64dd0abc7b8ad27b09550195743d3e61ff60",
      "new_mode": 33188,
      "new_path": "gdb/dwarf2/read.c"
    }
  ]
}
