Mailing List Archive

Re: svn commit: r1893643 - in /httpd/httpd/trunk: include/ap_mmn.h modules/dav/main/mod_dav.h modules/dav/main/props.c
On 9/26/21 2:27 PM, minfrin@apache.org wrote:
> Author: minfrin
> Date: Sun Sep 26 12:27:59 2021
> New Revision: 1893643
>
> URL: http://svn.apache.org/viewvc?rev=1893643&view=rev
> Log:
> Add dav_get_liveprop_element() to hide the implementation by which
> the element is passed.
>
> Modified:
> httpd/httpd/trunk/include/ap_mmn.h
> httpd/httpd/trunk/modules/dav/main/mod_dav.h
> httpd/httpd/trunk/modules/dav/main/props.c
>
> Modified: httpd/httpd/trunk/include/ap_mmn.h
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/include/ap_mmn.h?rev=1893643&r1=1893642&r2=1893643&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/include/ap_mmn.h (original)
> +++ httpd/httpd/trunk/include/ap_mmn.h Sun Sep 26 12:27:59 2021
> @@ -692,14 +692,15 @@
> * for 20210506.0 + 20210924.0, MINOR bump only for
> * adding ap_proxy_tunnel_conn_bytes_{in,out}().
> * 20210924.1 (2.5.1-dev) Add ap_proxy_fill_error_brigade()
> + * 20210926.0 (2.5.1-dev) Add dav_get_liveprop_element(), remove DAV_PROP_ELEMENT.
> */
>
> #define MODULE_MAGIC_COOKIE 0x41503235UL /* "AP25" */
>
> #ifndef MODULE_MAGIC_NUMBER_MAJOR
> -#define MODULE_MAGIC_NUMBER_MAJOR 20210924
> +#define MODULE_MAGIC_NUMBER_MAJOR 20210926
> #endif
> -#define MODULE_MAGIC_NUMBER_MINOR 1 /* 0...n */
> +#define MODULE_MAGIC_NUMBER_MINOR 0 /* 0...n */
>
> /**
> * Determine if the server's current MODULE_MAGIC_NUMBER is at least a
>
> Modified: httpd/httpd/trunk/modules/dav/main/mod_dav.h
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/dav/main/mod_dav.h?rev=1893643&r1=1893642&r2=1893643&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/dav/main/mod_dav.h (original)
> +++ httpd/httpd/trunk/modules/dav/main/mod_dav.h Sun Sep 26 12:27:59 2021
> @@ -1068,21 +1068,19 @@ DAV_DECLARE(long) dav_get_liveprop_ns_co
> DAV_DECLARE(void) dav_add_all_liveprop_xmlns(apr_pool_t *p,
> apr_text_header *phdr);
>
> -/*
> - ** When calling insert_prop(), the request element is associated with
> - ** the pool userdata attached to the resource. Access as follows:
> - **
> - ** apr_pool_userdata_get(&elem, DAV_PROP_ELEMENT, resource->pool);
> - **
> - */
> -#define DAV_PROP_ELEMENT "mod_dav-element"
> -

Hm, mod_dav.h is public. This IMHO means that removing the define would require a major bump.
Furthermore this move to props.c would not be backportable. But I guess this is no a big deal as the remainder of the patch is
backportable without any trouble.

Regards

RĂ¼diger
Re: svn commit: r1893643 - in /httpd/httpd/trunk: include/ap_mmn.h modules/dav/main/mod_dav.h modules/dav/main/props.c [ In reply to ]
On 10/7/21 12:00 PM, Ruediger Pluem wrote:
>
>
> On 9/26/21 2:27 PM, minfrin@apache.org wrote:
>> Author: minfrin
>> Date: Sun Sep 26 12:27:59 2021
>> New Revision: 1893643
>>
>> URL: http://svn.apache.org/viewvc?rev=1893643&view=rev
>> Log:
>> Add dav_get_liveprop_element() to hide the implementation by which
>> the element is passed.
>>
>> Modified:
>> httpd/httpd/trunk/include/ap_mmn.h
>> httpd/httpd/trunk/modules/dav/main/mod_dav.h
>> httpd/httpd/trunk/modules/dav/main/props.c
>>
>> Modified: httpd/httpd/trunk/include/ap_mmn.h
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/include/ap_mmn.h?rev=1893643&r1=1893642&r2=1893643&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/include/ap_mmn.h (original)
>> +++ httpd/httpd/trunk/include/ap_mmn.h Sun Sep 26 12:27:59 2021
>> @@ -692,14 +692,15 @@
>> * for 20210506.0 + 20210924.0, MINOR bump only for
>> * adding ap_proxy_tunnel_conn_bytes_{in,out}().
>> * 20210924.1 (2.5.1-dev) Add ap_proxy_fill_error_brigade()
>> + * 20210926.0 (2.5.1-dev) Add dav_get_liveprop_element(), remove DAV_PROP_ELEMENT.
>> */
>>
>> #define MODULE_MAGIC_COOKIE 0x41503235UL /* "AP25" */
>>
>> #ifndef MODULE_MAGIC_NUMBER_MAJOR
>> -#define MODULE_MAGIC_NUMBER_MAJOR 20210924
>> +#define MODULE_MAGIC_NUMBER_MAJOR 20210926
>> #endif
>> -#define MODULE_MAGIC_NUMBER_MINOR 1 /* 0...n */
>> +#define MODULE_MAGIC_NUMBER_MINOR 0 /* 0...n */
>>
>> /**
>> * Determine if the server's current MODULE_MAGIC_NUMBER is at least a
>>
>> Modified: httpd/httpd/trunk/modules/dav/main/mod_dav.h
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/dav/main/mod_dav.h?rev=1893643&r1=1893642&r2=1893643&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/modules/dav/main/mod_dav.h (original)
>> +++ httpd/httpd/trunk/modules/dav/main/mod_dav.h Sun Sep 26 12:27:59 2021
>> @@ -1068,21 +1068,19 @@ DAV_DECLARE(long) dav_get_liveprop_ns_co
>> DAV_DECLARE(void) dav_add_all_liveprop_xmlns(apr_pool_t *p,
>> apr_text_header *phdr);
>>
>> -/*
>> - ** When calling insert_prop(), the request element is associated with
>> - ** the pool userdata attached to the resource. Access as follows:
>> - **
>> - ** apr_pool_userdata_get(&elem, DAV_PROP_ELEMENT, resource->pool);
>> - **
>> - */
>> -#define DAV_PROP_ELEMENT "mod_dav-element"
>> -
>
> Hm, mod_dav.h is public. This IMHO means that removing the define would require a major bump.
> Furthermore this move to props.c would not be backportable. But I guess this is no a big deal as the remainder of the patch is
> backportable without any trouble.

Scratch this. I should read the patches better. You do a major bump above.

Regards

RĂ¼diger