[dylan-lang/opendylan] 10a8aa: [c-ffi, dfmc-back-end] Fix nested structs by value.

GitHub <[email protected]>
Newsgroups gmane.comp.lang.dylan.gwydion.cvs
Message-ID <54af2b804bcb1_54b43fd911f852bc77178@hookshot-fe5-cp1-prd.iad.github.net.mail>
  Branch: refs/heads/master
  Home:   https://github.com/dylan-lang/opendylan
  Commit: 10a8aae6fcf255232da4e8064555ddf9f0094c49
      https://github.com/dylan-lang/opendylan/commit/10a8aae6fcf255232da4e8064555ddf9f0094c49
  Author: Bruce Mitchener <[email protected]>
  Date:   2015-01-09 (Fri, 09 Jan 2015)

  Changed paths:
    M sources/dfmc/back-end/heaps.dylan
    M sources/lib/c-ffi/test/main.c
    M sources/lib/c-ffi/test/tests.dylan

  Log Message:
  -----------
  [c-ffi,dfmc-back-end] Fix nested structs by value.

In maybe-claim-heap-element-references-internal, when processing
a raw aggregate type (used for values whose type is a C struct
or union and not a pointer to a struct/union), process the member
types first, if necessary, then the type itself.

This ensures that they're in the heap-referenced-objects list
in the correct order and then emitted in C in the correct order
in emit-typedefs in the C backend.  Without doing this, the C
output will be incorrect and compilation will fail.


  Commit: d7679c5a3fbf8e7ac7a7013cde153ff539774e26
      https://github.com/dylan-lang/opendylan/commit/d7679c5a3fbf8e7ac7a7013cde153ff539774e26
  Author: Peter S. Housel <[email protected]>
  Date:   2015-01-08 (Thu, 08 Jan 2015)

  Changed paths:
    M sources/dfmc/back-end/heaps.dylan
    M sources/lib/c-ffi/test/main.c
    M sources/lib/c-ffi/test/tests.dylan

  Log Message:
  -----------
  Merge pull request #846 from waywardmonkeys/nested-structs-by-value

[c-ffi,dfmc-back-end] Fix nested structs by value.


Compare: https://github.com/dylan-lang/opendylan/compare/cca42b9f2c4b...d7679c5a3fbf

_______________________________________________
chatter mailing list
[email protected]
https://lists.opendylan.org/mailman/listinfo/chatter
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.