[asterisk-dev] [Code Review] Fix multiple parking issues.

Shaun Ruffell sruffell at digium.com
Thu Aug 11 10:25:28 CDT 2011


On Thu, Aug 11, 2011 at 02:28:40PM -0000, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1358/#review4032
> -----------------------------------------------------------
> 
> 
> Honestly, patches this huge are very difficult to track with
> reviewboard, and in fact I don't really believe there is a diff viewer
> on Earth that could adequately portray something with this much
> reordering...

Hopefully in the not-too-distant future we will have a good way to
handle patch series instead of individual patches. A side effect of
taking large groups of related, but independent, commits as patch series
is ease of bisecting for regressions after the fact.

Looking at the description in https://reviewboard.asterisk.org/r/1358
leads me to believe this change is a prime candidate.

-- 
Shaun Ruffell
Digium, Inc. | Linux Kernel Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at: www.digium.com & www.asterisk.org



More information about the asterisk-dev mailing list