Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.2 9/18/84; site ritcv.UUCP
Path: utzoo!linus!philabs!cmcl2!seismo!rochester!ritcv!mjl
From: mjl@ritcv.UUCP (Mike Lutz)
Newsgroups: net.unix-wizards
Subject: Re:  code quality
Message-ID: <8952@ritcv.UUCP>
Date: Sat, 12-Oct-85 10:24:01 EDT
Article-I.D.: ritcv.8952
Posted: Sat Oct 12 10:24:01 1985
Date-Received: Tue, 15-Oct-85 05:44:13 EDT
References: <2012@brl-tgr.ARPA>
Reply-To: mjl@ritcv.UUCP (Michael Lutz)
Organization: Rochester Institute of Technology, Rochester, NY
Lines: 25
Keywords: style subjectivism

In article <2012@brl-tgr.ARPA> glenn@LL-XN.ARPA (Glenn Adams) writes:
>... It is on this aesthetic level that code is often judged fish
>or fowl.  But, one may argue that code really doesn't "work" when it "looks"
>bad.  This often comes into play when someone, usually not the original
>author, must "look" at such code, and "fix" it.  Usually, the "fix" involves
>serious mastication resulting in a different "look" found to be more pleasing
>to the person performing the "fix".

Of course anyone who doesn't follow *my* style is a troglodyte (and probably
a tool of the military-industrial complex to boot) ;-)

My rule-of-thumb in fixing others' code is to adopt whatever style they
used so that the result is at least consistent.  If there is no
identifiable style (unfortunately, true of all too many student
programs), or the style makes me ill, I rearrange *everything* to
follow *my* style.

This isn't a perfect solution, especially if I rearrange code that
comes from the net, and later see "diff listing" bug fixes.  So, I
rearrange as infrequently as my highly discriminatory aesthetics
permit.  The result: tempering of my professional hubris.
-- 
Mike Lutz	Rochester Institute of Technology, Rochester NY
UUCP:		{allegra,seismo}!rochester!ritcv!mjl
CSNET:		mjl%rit@csnet-relay.ARPA