[dmd-internals] DVCS and patches feedback [was: dmd commit, revision 515]

Walter Bright walter at digitalmars.com
Thu Jun 3 14:16:36 PDT 2010



Don Clugston wrote:
> This shows a significant misunderstanding of where the time goes.
> There are three things that take a lot of time: (1) understanding the
> bug, and how the patch is trying to fix it;  (2) running the test
> suite (takes about half an hour); (3) adding the bug to the test
> suite.
> Making the changes to the existing code is usually (though not always)
> pretty trivial and quick. The majority of the patches which I have
> made are just one-line or two-line changes.
>
> You've also missed the steps of reproducing the existing bug, and
> checking that the bug now passes. This always involves copying code
> from bugzilla into a text editor.
>
> It should be pretty clear from this that (2) and (3) have some major
> opportunities for efficiency improvement. Far more significant IMHO
> than any changes to the compiler version control system.
>
>   

A couple more things need to happen:

4. have to fold it into D1, or if it's a D1 patch, fold it into D2
5. update change logs, bugzilla resolution messages


More information about the dmd-internals mailing list