You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ro...@apache.org on 2006/02/13 03:34:27 UTC

svn commit: r377292 - /httpd/httpd/trunk/modules/dav/main/mod_dav.c

Author: rooneg
Date: Sun Feb 12 18:34:26 2006
New Revision: 377292

URL: http://svn.apache.org/viewcvs?rev=377292&view=rev
Log:
Clean up some code in mod_dav.

Submitted by: Dan Rall <dlr collab.net>

* modules/dav/main/mod_dav.c
  (dav_error_response, dav_error_response_tag): Remove redundant assignment
   of r->status_line which is handled by basic_http_header_check().

Modified:
    httpd/httpd/trunk/modules/dav/main/mod_dav.c

Modified: httpd/httpd/trunk/modules/dav/main/mod_dav.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/dav/main/mod_dav.c?rev=377292&r1=377291&r2=377292&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/dav/main/mod_dav.c (original)
+++ httpd/httpd/trunk/modules/dav/main/mod_dav.c Sun Feb 12 18:34:26 2006
@@ -314,9 +314,6 @@
 {
     r->status = status;
 
-    /* ### I really don't think this is needed; gotta test */
-    r->status_line = ap_get_status_line(status);
-
     ap_set_content_type(r, "text/html");
 
     /* begin the response now... */
@@ -347,9 +344,6 @@
                                   dav_error *err)
 {
     r->status = err->status;
-
-    /* ### I really don't think this is needed; gotta test */
-    r->status_line = ap_get_status_line(err->status);
 
     ap_set_content_type(r, DAV_XML_CONTENT_TYPE);