[dmd-internals] [D-Programming-Language/dmd] 98467f: fix Issue 8437 - [2.060 beta] static struct no siz...

GitHub noreply at github.com
Sat Jul 28 13:12:31 PDT 2012


  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/dmd
  Commit: 98467f4202d7bca6abd18cc28a51e95a0bc57100
      https://github.com/D-Programming-Language/dmd/commit/98467f4202d7bca6abd18cc28a51e95a0bc57100
  Author: k-hara <k.hara.pg at gmail.com>
  Date:   2012-07-28 (Sat, 28 Jul 2012)

  Changed paths:
    M src/class.c
    M test/runnable/xtest46.d

  Log Message:
  -----------
  fix Issue 8437 - [2.060 beta] static struct no size yet for forward reference

This is a regression introduced fixing aggregate alignment.
Introduced Commit: 154e44a006270d53745f99ec3e538a0ce526ae76

In VarDeclaration::semantic, if scope alignment is default, then its type's alignment is used, and it needs to resolve forward reference.
But, in ClassDeclaration::semantic, the calling setScope for AttribDeclaration members was lacked.


  Commit: c59699ae29dde0f32239bcadb853c7793ee4bada
      https://github.com/D-Programming-Language/dmd/commit/c59699ae29dde0f32239bcadb853c7793ee4bada
  Author: Walter Bright <walter at walterbright.com>
  Date:   2012-07-28 (Sat, 28 Jul 2012)

  Changed paths:
    M src/class.c
    M test/runnable/xtest46.d

  Log Message:
  -----------
  Merge pull request #1067 from 9rnsr/fix8437

Issue 8437 - [2.060 beta] static struct no size yet for forward reference


Compare: https://github.com/D-Programming-Language/dmd/compare/988414c6564f...c59699ae29dd


More information about the dmd-internals mailing list