Thanks for merging in the vcbuild patch.

I guess this commit was made to fix building dmc, but it broke the MS build at 2 places. I have prepared a pull request, but wanted to ask what the right way to fix them is:

On 25.02.2012 03:47, GitHub wrote:
  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/dmd
  Commit: eaac7ba53fc1f913120693b2c8bca4d22574fa7e
      https://github.com/D-Programming-Language/dmd/commit/eaac7ba53fc1f913120693b2c8bca4d22574fa7e
  Author: Walter Bright <walter@walterbright.com>
  Date:   2012-02-24 (Fri, 24 Feb 2012)

  Changed paths:
    M src/backend/bcomplex.h
  M src/backend/cc.h
  M src/backend/cdef.h

this line was reverted:
+inline char *prettyident(Symbol *s) { return CPP ? cpp_prettyident(s) : s->Sident; }

but it causes a link failure due to missing symbol cpp_prettyident. Where should I put its implementation? dsymbol.c? Maybe it should be added to the D source unconditionally, not only for _MSC_VER.

  M src/backend/cg87.c

reverted the use of longdouble. If I read the includes correctly, it should be defined in cdef.h for a C compiler aswell. Why should it not be used?

  M src/backend/debug.c
  M src/backend/el.c
  M src/posix.mak
  M src/root/longdouble.c
  M src/root/longdouble.h
  M src/win32.mak

  Log Message:
  -----------
  fix build breakage





_______________________________________________
dmd-internals mailing list
dmd-internals@puremagic.com
http://lists.puremagic.com/mailman/listinfo/dmd-internals