Release D 2.078.1

Radu void at null.pt
Wed Jan 24 20:32:55 UTC 2018


On Wednesday, 24 January 2018 at 20:05:42 UTC, Martin Nowak wrote:
> On Wednesday, 24 January 2018 at 13:30:42 UTC, Radu wrote:
>> Just want to bring to your attention a major regression 
>> introduced in 2.078, and still present on current master, re. 
>> the new __equals template.
>>
>> https://issues.dlang.org/show_bug.cgi?id=18252
>>
>> Comparing arrays of associative arrays is broken!
>
> Not to be insulting, but hoping for better understanding.
>
> Why, even when the compiler almost directly points to the 
> problem at hand (a missing check for associative arrays :o, 
> https://github.com/dlang/druntime/blob/a66f9fa78f2e295146fb2b3fc8e567983c9e9aed/src/object.d#L3486), doesn't this simply convert to a small pull request to fix the problem (https://github.com/dlang/druntime/pull/2058)?

First of all, thanks for fixing it!

No injuries suffered :D, but here is a list:
- not so confident I understood the whole issue form that 
implementation, count it as lack of experience.
- if wanting to get hands dirty - setting up the whole D dev env 
(compiler, runtime, tests) on a new machine it takes a while, at 
least the way I count it.
- even if I get that running and make a change that runs on my 
machine, the outlook of time spent in ping-pong with the 
reviewers/auto-testers on a PR is daunting.

To wrap it up - it is not `easy as pie` for random contributor to 
make even a simple PR because of the perceived overhead both in 
engineering and social aspects, that or I'm to lazy :O


More information about the Digitalmars-d-announce mailing list