Return-Path: Delivered-To: apmail-apr-cvs-archive@apr.apache.org Received: (qmail 23528 invoked by uid 500); 14 Jul 2003 00:38:19 -0000 Mailing-List: contact cvs-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Reply-To: dev@apr.apache.org Delivered-To: mailing list cvs@apr.apache.org Received: (qmail 23517 invoked from network); 14 Jul 2003 00:38:18 -0000 Date: 14 Jul 2003 00:38:18 -0000 Message-ID: <20030714003818.52899.qmail@icarus.apache.org> From: jwoolley@apache.org To: apr-cvs@apache.org Subject: cvs commit: apr/mmap/unix mmap.c X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N jwoolley 2003/07/13 17:38:17 Modified: mmap/unix mmap.c Log: it's confusing as hell to get an allocated object back that's all zeros. yeah, yeah, we returned something other than APR_SUCCESS, but still. anyway, the win32 implementation already does this, so now we're consistent. Submitted by: Sander, Branko, Cliff Revision Changes Path 1.50 +2 -0 apr/mmap/unix/mmap.c Index: mmap.c =================================================================== RCS file: /home/cvs/apr/mmap/unix/mmap.c,v retrieving revision 1.49 retrieving revision 1.50 diff -u -d -u -r1.49 -r1.50 --- mmap.c 10 Mar 2003 19:02:38 -0000 1.49 +++ mmap.c 14 Jul 2003 00:38:17 -0000 1.50 @@ -143,6 +143,7 @@ if (aid < B_NO_ERROR) { /* we failed to get an area we can use... */ + *new = NULL; return APR_ENOMEM; } @@ -163,6 +164,7 @@ if (mm == (void *)-1) { /* we failed to get an mmap'd file... */ + *new = NULL; return errno; } #endif