I think code reviews are very useful, especially for not yet experts. It is good education for all levels. We had reviews, and comments like - Do you need a latch across these instructions for when there is concurrent execution? - If it abends here, how will the frr code ... - Would it help capturing foot prints to show how we got here, before the following code which may abend. - Some customers will not send us a dump, so we need to capture as much as we can in registers. Colin
On Sun, 17 Mar 2024 at 13:40, Bob Bridges < [email protected]> wrote: > I suppose code reviews are like post-battle debriefs, in which every > choice of the commander is questioned, and when challenged with a better > option he can only say "I didn't think of that at the time". It must be > extremely painful for the guy in the glare of the spotlight, but if it's > done right -- too late to blame anyone now, we're just trying to make sure > everyone has a chance to learn how to do it better next time -- it's also > extremely valuable. I truly hated having the code reviewers poke at my > baby, and at the start of this thread I pointed out ways in which they > didn't do it right. But I agree with Shmuel, when done right you might > even say they're necessary. > > (And post-surgical reviews, too. The patient lived, or the patient died, > fine, but now we want to know what to do differently next time.) > > --- > Bob Bridges, [email protected], cell 336 382-7313 > > /* The kingdom of Heaven is not for the well-meaning; it is for the > desperate. -James Denney (1856-1917) */ > > -----Original Message----- > From: IBM Mainframe Discussion List <[email protected]> On Behalf > Of Seymour J Metz > Sent: Sunday, March 17, 2024 00:41 > > However, disagree vehemently about "Code reviews are dumb and not needed > by good programmers." The first two things that a good programmer learns > are that nobody has a monopoly on good ideas and that "Even Jove nods". > Code reviews are like auditors and tech writers: when they are good they > are very very good, and when they are bad they are horrid. > > ---------------------------------------------------------------------- > For IBM-MAIN subscribe / signoff / archive access instructions, > send email to [email protected] with the message: INFO IBM-MAIN > ---------------------------------------------------------------------- For IBM-MAIN subscribe / signoff / archive access instructions, send email to [email protected] with the message: INFO IBM-MAIN
