From 54c92070cebc236ad577272179c90f7c21880b1c Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Wed, 28 Nov 2001 22:47:30 +0000 Subject: [PATCH] clarify obvious fix. --- gdb/ChangeLog | 5 +++++ gdb/MAINTAINERS | 7 +++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9540a7faa2..884d17df0b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2001-11-27 Andrew Cagney + + * MAINTAINERS: Clarify obvious fix a little (as suggested by Eli + Zaretskii). + 2001-11-25 Jim Blandy * mn10300-tdep.c (mn10300_analyze_prologue): Doc fixes. diff --git a/gdb/MAINTAINERS b/gdb/MAINTAINERS index 03a54bd4cf..34cf20cf43 100644 --- a/gdb/MAINTAINERS +++ b/gdb/MAINTAINERS @@ -33,13 +33,16 @@ All maintainers listed in this file are allowed to check in obvious fixes. An "obvious fix" means that there is no possibility that anyone will -disagree with the change. Something like bypassing an interface will -result in instantaneous and loud complaints. +disagree with the change. A good mental test is "will the person who hates my work the most be able to find fault with the change" - if so, then it's not obvious and needs to be posted first. :-) +Something like changing or bypassing an interface is _not_ an obvious +fix, since such a change without discussion will result in +instantaneous and loud complaints. + Target/Architecture: