X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7D481.26EFC1D2@onstor-exch02.onstor.net>; Wed, 1 Aug 2007 13:15:57 -0800
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 24795 for review
Date: Wed, 1 Aug 2007 13:15:57 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E04D2065F@onstor-exch02.onstor.net>
In-Reply-To: <20070801101729.6c23cc12@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 24795 for review
Thread-Index: AcfUX9hU+T3CQjaQSU2FpbLo94kJhgAITxPg
References: <WEBMAIL9aWYqhwFKp4W00000f88@mail.onstor.com> <20070801101729.6c23cc12@ripper.onstor.net>
From: "Amit Bothra" <amit.bothra@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Hi Andy,

Thanks for pointing it out. I'll keep this in mind for my future
checkins.

Thanks,
Amit

-----Original Message-----
From: Andy Sharp=20
Sent: Wednesday, August 01, 2007 10:17 AM
To: Amit Bothra
Subject: Re: PERFORCE change 24795 for review

Hi Amit,

I know you just started so no one has probably told you, but our
convention for code checkin comments is to have a lot more
information.  For example, 3 months from now if you went and looked at
the history of the file fs-log-replay.c and saw this comment, would you
know enough to determine if you needed to examine the change(s) more
closely?  Probably not, since "Coverity Fixes" doesn't really tell you
anything.  In general try to include more information, but don't spend
too much time on it.  Think of the checkin comments as a quick note you
are sending to yourself in the future to remind you what this change
was about.

Thanks,

a


On 31 Jul 2007 22:14:22 -0700 amitb <amit.bothra@onstor.com> wrote:

> Change 24795 by amitb@amitb-zonda on 2007/07/31 22:06:30
>=20
> 	Coverity Fixes=20
> 	Reviewed by jobia
>=20
> Affected files ...
>=20
> ... //depot/dev/nfx-tree/code/sm-fs/btree/emaproot.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-admin.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-am.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-api.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-buf.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-context.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-debug.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-dir.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-err.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-global.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-inode.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-log-replay.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-log.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-ref.c#2 edit
> ... //depot/dev/nfx-tree/code/sm-fs/fs-trashcan.c#2 edit
>=20
>=20
> http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/24795?ac=3D10
