What about doing:
 
 static const char *add_ignore_header(cmd_parms *parms, void *dummy,
-                                     const char *header)
+                                     char *header)
 {
     cache_server_conf *conf;
     char **new;
@@ -802,7 +802,7 @@
              * (When 'None' is passed, IGNORE_HEADERS_SET && nelts == 0.)
              */
             new = (char **)apr_array_push(conf->ignore_headers);
-            (*new) = (char *)header;
+            (*new) = header;

>>> [EMAIL PROTECTED] 10/25/04 10:32 AM >>>
Jean-Jacques Clar wrote:
> Thanks for reporting the problem Norm,
> The patch was submitted.

Is casting to (char *) really the right thing to do here?  I mean why
not just make new const so the cast isn't needed at all?  It seems like
a cast in this situation is a bit ugly...

-garrett

Reply via email to