Should we remove int[$] before 2.067?

H. S. Teoh via Digitalmars-d digitalmars-d at puremagic.com
Fri Jan 30 09:57:45 PST 2015


On Fri, Jan 30, 2015 at 06:47:22AM -0800, Andrei Alexandrescu via Digitalmars-d wrote:
> As discussed in this forum, Kenji has authored
> https://github.com/D-Programming-Language/dmd/pull/3615 which has been
> recently merged.
> 
> By this I am proposing we revert that decision, and quickly - before 2.067
> is released lest we'll need to support it forever. Here's why.

If it wasn't a good idea, I don't have a problem with reverting it, but
what I'm wondering is, why raise the objection *now* rather than *months*
ago when the PR was sitting in the queue idle? From the discussion on
github, it appeared that the only objection against it was that Walter
didn't like the syntax. Where were the arguments about it being a
superfluous syntax change? Why raise the objections now rather than back
then?

I think we need to improve the process here. If a PR is not up to par or
is a bad idea, or approval from Walter/Andrei is required, can we pretty
please mark it as such beforehand? Rather than, as it would appear, let
it sit there until someone merges it, only to parachute in after the
fact to blast it to bits? (I know that's not the intention, but that's
what it looks like, since I've lost count of how many months this
particular PR was sitting in the queue with only minor nitpicks raised
against it and no sign of imminent doom like it's made out to be now.)


T

-- 
This is not a sentence.


More information about the Digitalmars-d mailing list