This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]