<p dir="ltr">On 30 Aug 2014 12:35, "Jonathan M Davis via Digitalmars-d" <<a href="mailto:digitalmars-d@puremagic.com">digitalmars-d@puremagic.com</a>> wrote:<br>
><br>
> On Saturday, 30 August 2014 at 10:57:51 UTC, ketmar via Digitalmars-d wrote:<br>
>><br>
>> Hello.<br>
>><br>
>> there are some c-style array declarations both in druntime and in<br>
>> phobos. i made two patches that fixes 'em:<br>
>><br>
>><a href="https://issues.dlang.org/show_bug.cgi?id=13401">https://issues.dlang.org/show_bug.cgi?id=13401</a><br>
>><a href="https://issues.dlang.org/show_bug.cgi?id=13402">https://issues.dlang.org/show_bug.cgi?id=13402</a><br>
><br>
><br>
> If you want to contribute, please create pull requests on github:<br>
><br>
><a href="https://github.com/D-Programming-Language">https://github.com/D-Programming-Language</a><br>
><br>
> It's doubtful that anyone is going to take your patches and create PR's for you. So, creating patches and adding them to bugzilla isn't particularly useful, even if it's good code.<br>
><br>
> - Jonathan M Davis</p>
<p dir="ltr">Responses like this aren't particularly useful either.  We have a 'patch' keyword in bugzilla (from memory), and people are free to use that as a tool to manage bugs raised against D.</p>
<p dir="ltr">In fact, has anyone recently gone through bug tickets with attachments and checked for patches?  This could be one thing to do for EMSI's monthly bug squashing event (a similar thing is already done for squashing regressions or ICE's in the beta releases).</p>

<p dir="ltr">It doesn't take a lot of work to do this.</p>
<p dir="ltr">- Find and appropriately tag any bug reports with patches attached.<br>
- Review the tagged bug reports.<br>
- Mark as 'needs-work' if the patch doesn't apply.<br>
- Mark as 'pull-request' and raise a PR if it applies and fixes the bug/enhancement.  Use your best judgement when reviewing the code, but after a PR has been created, it will be properly code reviewed.<br>
- Close if the report is no longer reproducible, or patch has been applied or rejected (rejected enhancement patch).</p>
<p dir="ltr">And doing that for a couple hours a month is better than telling people not to send patches to bugzilla because no one will look at it.</p>
<p dir="ltr">If you put this doubt in people's minds, what does that say about the actual bug reports?  Should I even bother raising a bug report when no one will look at it?  Maybe I should just instead raise a PR with a test case and a comment next to the line that ICE's.  People read PRs, so my bug will be tended to answer fixed sooner.</p>

<p dir="ltr">(I don't actually plan in doing this).</p>
<p dir="ltr">Iain.</p>