DMD backend quality (Was: Re: DIP 1031--Deprecate Brace-Style Struct Initializers--Community Review Round 1 Discussion)

Walter Bright newshound2 at digitalmars.com
Thu Feb 20 10:58:36 UTC 2020


On 2/19/2020 10:34 PM, Seb wrote:
> And I haven't even mentioned the bottleneck of any major DMD PR needing to be 
> approved by Walter.

Given the current regression list:

https://issues.dlang.org/buglist.cgi?bug_severity=regression&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&list_id=230215&query_format=advanced

I'm probably being too permissive. I'm usually the one who gets to fix them. I 
asked in this forum for some help isolating which PRs caused the regressions, to 
no avail.

Major PRs *should* be regarded with skepticism. For example, I recently fixed a 
regression caused by a 2 or 3 hundred line PR, where the substance of the PR was 
one line and the rest of it was refactoring.

Folks, with PRs, smaller, tightly focused PRs are better. Do not:

1. lump multiple issues into one PR

2. include your favorite refactorings in with it

3. refactors must not change observable behavior

Do:

1. minimize the PR even if the result of the PR suggests a refactoring

2. do that refactor SEPARATELY AFTER THE PR WAS MERGED


More information about the Digitalmars-d mailing list