This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH] Enforce GMP/MPFR version requires and fix fortran/28276,27021
On Sep 25, 2006, at 2:12 PM, Ian Lance Taylor wrote:
I believe that it would help to create a new class of maintainers:
people who are permitted to approve patches in certain areas, but are
not permitted to commit patches in those areas without approval from
some other maintainer.
Internally we do exactly this at Apple. The general rule is, everyone
needs patches to be reviewed, and everyone can so review patches.
People review what they can, as they can.
I'd rather have this sort of class to help ensure adequate reviewer
bandwidth than to simply go without if the SC is reluctant to appoint
any additional reviewers.
My feeling is that the project would be benefitted by additional
reviewer bandwidth. More bandwidth means the work load is lessoned
for existing reviewers and leaves each reviewer more time to spend on
reviews (if they so choose) and/or allows them the latitude to focus
reviews into smaller areas of code. We can use the blanket people
more and more as architects and master planners to help the day-to-day
reviewers with direction and guidance.
Also, I we can squeeze more work out of people by making the process
more efficient. :-) More reviewers I think would help make the
process more efficient.