From dev-return-19351-apmail-apr-dev-archive=apr.apache.org@apr.apache.org Tue Nov 06 02:55:57 2007 Return-Path: Delivered-To: apmail-apr-dev-archive@www.apache.org Received: (qmail 8168 invoked from network); 6 Nov 2007 02:55:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 6 Nov 2007 02:55:56 -0000 Received: (qmail 18629 invoked by uid 500); 6 Nov 2007 02:55:44 -0000 Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 18564 invoked by uid 500); 6 Nov 2007 02:55:43 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Id: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 18553 invoked by uid 99); 6 Nov 2007 02:55:43 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 05 Nov 2007 18:55:43 -0800 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [64.202.165.39] (HELO smtpauth14.prod.mesa1.secureserver.net) (64.202.165.39) by apache.org (qpsmtpd/0.29) with SMTP; Tue, 06 Nov 2007 02:55:45 +0000 Received: (qmail 29928 invoked from network); 6 Nov 2007 02:55:22 -0000 Received: from unknown (24.15.193.17) by smtpauth14.prod.mesa1.secureserver.net (64.202.165.39) with ESMTP; 06 Nov 2007 02:55:21 -0000 Message-ID: <472FD799.8000002@rowe-clan.net> Date: Mon, 05 Nov 2007 20:55:21 -0600 From: "William A. Rowe, Jr." User-Agent: Thunderbird 2.0.0.5 (X11/20070727) MIME-Version: 1.0 To: Lucian Adrian Grijincu CC: dev@apr.apache.org Subject: Re: svn commit: r592216 - in /apr/apr/branches/1.2.x: include/apr_file_io.h test/testdir.c References: <20071106005123.9C9EE1A9832@eris.apache.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Lucian Adrian Grijincu wrote: > On Nov 6, 2007 2:51 AM, wrote: >> + APR_ASSERT_SUCCESS(tc, "remove cwd", rv = apr_dir_remove(path, p)); > > What purpose does "rv = " have in this code? > the apr_status_t isn't used anywhere in there None - good catch, thanks. Patching now. Bill