This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Reduce Dwarf Debug Size
On Thu, 1 Mar 2007, Andrew Pinski wrote:
> On 3/1/07, Mike Stump <mrs@apple.com> wrote:
> > people with CVS write access can approve it.
>
> I would not like someone with only libobjc knowledge approving patches
> that touch fold-const.c, do you?
> -- Pinski
It's possible for developers to "know what they don't know". Not every
developer reaches that level of self-awareness. But it's often clear when
they do.
If we had a more liberal checkin policy, in your above example I would
expect someone who is only expert in libobjc to only review libobjc
patches and if they touch other areas they are not familiar with, they
should request reviews same as other people. Once they gain sufficient
expertise, then they could begin to checkin patches to new areas without
review. We might create a mini-GWP once someone has shown sufficient
judgement that they respect their own limitations.
I also like to idea of N patches before becoming a maintainer of some
area. If person X touches a file or area 100 times (or some cut-off),
they should be considered it's de-facto maintainer. :-)
--Kaveh
--
Kaveh R. Ghazi ghazi@caip.rutgers.edu