Nick Kew wrote:
> ObPedant:
>
>> 14:42:30 2007 @@ -113,8 +113,8 @@
>> /* we break the URL into host, port, uri */
>> if (APR_SUCCESS != apr_uri_parse_hostinfo(p, url, &uri)) {
>> - return ap_proxyerror(r, HTTP_BAD_REQUEST,
>> - apr_pstrcat(p, "URI cannot be parsed: ", url,
>> NULL));
>> + return ap_proxyerror(r, HTTP_BAD_REQUEST, apr_pstrcat(p,
>> + "URI cannot be parsed: ", url, NULL));
>
> The second arg to ap_pstrcat is presented as a new arg to ap_proxyerror.
> The old formatting was better.
+1 to niq's observation
- this is now quite illegible in terms of how it is doing what it does.
