It sounds reasonable only if everyone is ready to "sign" certificates for each of non-trivial patches so that he or she has assured that each does not break anything.
I don't think such "review" work well. It should be fine if someone who broke tests fix the problem, or just revert in case it was impossible. That's what has happened to such patches that are accompanied by "please review the patch" posts. The same amount of "massive" patches happen to mcs/gmcs land without any approvals from the maintainers, kinda everyday. Atsushi Eno Boris Kirzner wrote:
Hello Senga, One of your latest updates introduces a regression (see attached test case, passes on r57148 but fails on r57149), due to the change in Key default row state filter. Please, next time you want to introduce some changes that has such a wide effect, especially in the code you do not wrote by your own, maybe it worth to give this fixes an opportunity to be reviewed by the community before applying them. Thanks, Boris
_______________________________________________ Mono-devel-list mailing list Mono-devel-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-devel-list