Mailing-List: contact bugs-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache HTTPD Bugs Notification List" <bugs@httpd.apache.org>
From: bugzilla@apache.org
To: bugs@httpd.apache.org
Subject: DO NOT REPLY [Bug 46578] New: Flawed header
Message-ID: <bug-46578-7868@https.issues.apache.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Date: Wed, 21 Jan 2009 12:26:01 -0800 (PST)

https://issues.apache.org/bugzilla/show_bug.cgi?id=46578

           Summary: Flawed header
           Product: Apache httpd-2
           Version: 2.3-HEAD
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core
        AssignedTo: bugs@httpd.apache.org
        ReportedBy: edwin@cheatah.nl


The header file ap_config_auto.h is a bit flawed. It defines PACKAGE_NAME and
PACKAGE_VERSION, and similar constants. If one would like to compile their own
module, one could write something like:

#ifdef HAVE_CONFIG_H
#include <config.h>
#endif
#include <httpd.h>
#include <http_config.h>
#include <http_protocol.h>
#include <ap_config.h>

This would give compiler warnings (not errors) about PACKAGE_NAME being
re-defined. This is a bit sloppy and certainly not necessary. Hardly any C
program defines PACKAGE_NAME and similar constants outside config.h, and header
files should certainly not re-define them.

It would be better to have this in ap_config_auto.h:

#ifndef PACKAGE_BUGREPORT
# define PACKAGE_BUGREPORT ""
#endif

So it's not very critical, but it's also not nice to cause warnings that need
not be caused.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org