[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Patch #642] simple nfs cleanups
From: |
nobody |
Subject: |
[Patch #642] simple nfs cleanups |
Date: |
Mon, 18 Nov 2002 08:17:24 -0500 |
Patch #642 has been updated.
Project:
Category: None
Status: Open
Summary: simple nfs cleanups
Follow-Ups:
Date: 2002-Nov-18 14:17
By: marcus
Comment:
It would be nice if you could get rid of the assert instead just moving it.
Also, I would like to see a rationale (if not as a comment, then along with the
patch) for each XXX you removed. Somebody put the comment there because he had
doubts. From looking at the patch, I can not decide if you didn't have those
doubts because you had a better or worse understanding of the code than the
person putting in those XXX :)
Thanks,
Marcus
-------------------------------------------------------
-------------------------------------------------------
For more info, visit:
http://savannah.gnu.org/patch/?func=detailpatch&patch_id=642&group_id=30