From owner-cvs-gnu Mon Apr 3 08:04:44 1995 Return-Path: cvs-gnu-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id IAA10650 for cvs-gnu-outgoing; Mon, 3 Apr 1995 08:04:44 -0700 Received: (from nate@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id IAA10639; Mon, 3 Apr 1995 08:04:31 -0700 Date: Mon, 3 Apr 1995 08:04:31 -0700 From: Nate Williams Message-Id: <199504031504.IAA10639@freefall.cdrom.com> To: CVS-commiters, cvs-gnu Subject: cvs commit: src/gnu/usr.bin/cvs/cvs lock.c Sender: cvs-gnu-owner@freebsd.org Precedence: bulk nate 95/04/03 08:04:30 Modified: gnu/usr.bin/cvs/cvs lock.c Log: The conditional in unlock() stats the master lock file (expensive) before it has determined whether or not removing it is even appropriate. i.e. if cleanup_lckdir is set. The enclosed patch shuffles around the conditional, and it clears cleanup_lckdir in clear_lock(), so the sprintf()/stat() is only performed when it is really needed. This patch is relative to my last one that introduced clear_lock(). Submitted by: "J.T. Conklin"