[144] in Information Retrieval

home help back first fref pref prev next nref lref last post

Reviewspeak [rec.humor.funny #4154]

daemon@ATHENA.MIT.EDU (rcharbon@Athena.MIT.EDU)
Mon Mar 8 08:19:23 1993

From: rcharbon@Athena.MIT.EDU
Date: Mon, 8 Mar 93 08:17:44 -0500
To: elibdev@Athena.MIT.EDU


------ Forwarded Article <S512.6163@looking.on.ca>
------ From sybase!tim@sun.com (Tim Wood)


In the shower this morning, it occurred to me it would be useful 
to write a guide listing certain words and phrases that arise
during code reviews and their actual, candid meanings.  Hopefully
this will help the communication process at technical reviews.


Review term or phrase                   Real Meaning
---------------------                   ------------
I'm not sure this fits in               If I have anything to do
with our architecture.                  with it, this code will never
                                        see the light of day!

It's not clear that ...                 There's no way that ...

It occurred to me in the                It occurred to me while reading
shower this morning ...                 netnews today ...

I think that's against the              That's not the way I arrange my
coding standards.                       {whitespace, declarations, ifs, etc.}

This logic doesn't belong in            If you do it this way, I'll have to
this module.                            rework a whole bunch of stuff of mine.

Let's take this off line.		If you carry on this pedantic
                                        discussion another minute, I'll scream.

Let's make that an action item.         There's another thing you forgot.

Do we really want the user to see       Wouldn't it be funny if the user
this?                                   saw this?

Doing it this way will be hard to       I might have to maintain this 
maintain.                               someday.

This really needs to be documented.     We could just leave it undocumented.

Let's set up a time to continue the     If you all didn't talk so much, we'd
review.                                 be done by now.




--
Selected by Maddi Hausmann.  MAIL your joke (jokes ONLY) to funny@clarinet.com.

This newsgroup is sponsored by ClariNet Communications Corp.  The "executive
moderator" is Brad Templeton.

------ End of Forwarded Article

-- 
Ray Charbonneau              | The IBM compatible sector has not yet recognized
rcharbon@athena.mit.edu      | that 95% of computer usage is devoted to 
-----------------------------| experimenting with different fonts and character
Disclaimer?  Why?            | styles in documents. - Ron Reiner


home help back first fref pref prev next nref lref last post