[Issue 14564] [REG2.067] dmd -property -unittest combination causes compiler error

via Digitalmars-d-bugs digitalmars-d-bugs at puremagic.com
Wed Jul 22 13:15:58 PDT 2015


https://issues.dlang.org/show_bug.cgi?id=14564

--- Comment #12 from Jonathan M Davis <issues.dlang at jmdavisProg.com> ---
(In reply to Илья Ярошенко from comment #11)
> We need make the choice:
> 1. Remove -property option from compiler
> OR
> 2. Add the -property option to the GitHub testing engine and fix ALL Phobos.

Exactly. And given that even if we want some type of property enforcement, the
-property flag is enforcing something that we definitely _don't_ want at this
point (that non- at property functions be called with parens), I think that we'd
be better off to just remove -property. And presumably, that would mean making
it do nothing to begin with rather than simply removing it, but we clearly
don't want it, and it seems rather silly to be making a bunch of changes to
Phobos at this point to support it (though arguably, whatever PR that removed
the -property flag from the Phobos build shouldn't have been pulled as long as
-property was still in the compiler, but it's a bit late for that now).

So, personally, I think that the clear choice is to make -property do nothing
(except maybe print out a warning about how it now does nothing), but I don't
know how much work that would be. Regardless, we have to get rid of it at some
point, and if we don't do it now, we need to start building Phobos with
-property, and make all of the changes associated with that.

--


More information about the Digitalmars-d-bugs mailing list