Return-Path: Delivered-To: apmail-apache-cvs-archive@apache.org Received: (qmail 98613 invoked by uid 500); 11 Jun 2000 14:53:53 -0000 Mailing-List: contact apache-cvs-help@apache.org; run by ezmlm Precedence: bulk X-No-Archive: yes Reply-To: new-httpd@apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list apache-cvs@apache.org Received: (qmail 98590 invoked by uid 500); 11 Jun 2000 14:53:52 -0000 Delivered-To: apmail-apache-2.0-cvs@apache.org Date: Sun, 11 Jun 2000 10:50:58 -0400 Message-Id: <200006111450.KAA17953@k5.localdomain> X-Authentication-Warning: k5.localdomain: trawick set sender to trawickj@bellsouth.net using -f From: Jeff Trawick To: new-httpd@apache.org CC: apache-2.0-cvs@apache.org In-reply-to: <20000611140453.88092.qmail@locus.apache.org> (ben@locus.apache.org) Subject: Re: cvs commit: apache-2.0/src/modules/standard mod_include.c Reply-to: trawickj@bellsouth.net References: <20000611140453.88092.qmail@locus.apache.org> X-Spam-Rating: locus.apache.org 1.6.2 0/1000/N > From: ben@locus.apache.org > Date: 11 Jun 2000 14:04:53 -0000 > > ben 00/06/11 07:04:53 > > Modified: src/modules/standard mod_include.c > Log: > Do printf formatting properly. > > int l, x; > -#if defined(AP_OFF_T_IS_QUAD) > - ap_snprintf(tag, sizeof(tag), "%qd", finfo.size); > -#else > - ap_snprintf(tag, sizeof(tag), "%ld", (long)finfo.size); > -#endif > + ap_snprintf(tag, sizeof(tag), "%" APR_OFF_T_FMT, finfo.size); > l = strlen(tag); /* grrr */ As of this commit we now have warnings on Linux. Did you see the discussion (or rather, me explaining why I couldn't proceed with Ryan's sugession) yesterday? What's the plan on getting APR_OFF_T_FMT defined properly? I don't know how to work around the problem I described to Ryan yesterday. How can we safely check during autoconfiguration whether off_t is long int or just int when the size is the same? Unless this is resolved, I don't see the value in AP_OFF_T_FMT unless there is also an AP_OFF_T_CAST which is used to avoid warnings. That combination isn't pretty, but at least it avoids the old need to check AP_OFF_T_IS_QUAD. -- Jeff Trawick | trawick@ibm.net | PGP public key at web site: http://www.geocities.com/SiliconValley/Park/9289/ Born in Roswell... married an alien...