Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Apache: Dev

Re: svn commit: r823337 - in /httpd/httpd/trunk: CHANGES include/ap_mmn.h include/http_core.h modules/loggers/mod_logio.c server/scoreboard.c

 

 

Apache dev RSS feed   Index | Next | Previous | View Threaded


sf at sfritsch

Oct 8, 2009, 2:45 PM

Post #1 of 3 (465 views)
Permalink
Re: svn commit: r823337 - in /httpd/httpd/trunk: CHANGES include/ap_mmn.h include/http_core.h modules/loggers/mod_logio.c server/scoreboard.c

On Thursday 08 October 2009, sf [at] apache wrote:
> --- httpd/httpd/trunk/include/ap_mmn.h (original)
> +++ httpd/httpd/trunk/include/ap_mmn.h Thu Oct 8 21:42:13 2009
> @@ -198,15 +198,17 @@
> * 20090401.3 (2.3.3-dev) Added DAV options provider to mod_dav.h
> * 20090925.0 (2.3.3-dev) Added server_rec::context and added
> *server_rec * param to
> ap_wait_or_timeout()
> + * 20090925.1 (2.3.3-dev) Add optional function
> ap_logio_get_last_bytes() to + * mod_logio
> *
> */
>
> #define MODULE_MAGIC_COOKIE 0x41503234UL /* "AP24" */
>
> #ifndef MODULE_MAGIC_NUMBER_MAJOR
> -#define MODULE_MAGIC_NUMBER_MAJOR 20090401
> +#define MODULE_MAGIC_NUMBER_MAJOR 20090925
> #endif
> -#define MODULE_MAGIC_NUMBER_MINOR 2 /* 0...n
> */ +#define MODULE_MAGIC_NUMBER_MINOR 1 /*
> 0...n */
>

I updated MODULE_MAGIC_NUMBER_MAJOR to match the comments. I hope this
is correct?


wrowe at rowe-clan

Oct 8, 2009, 3:00 PM

Post #2 of 3 (429 views)
Permalink
Re: svn commit: r823337 - in /httpd/httpd/trunk: CHANGES include/ap_mmn.h include/http_core.h modules/loggers/mod_logio.c server/scoreboard.c [In reply to]

Stefan Fritsch wrote:
> On Thursday 08 October 2009, sf [at] apache wrote:
>> --- httpd/httpd/trunk/include/ap_mmn.h (original)
>> +++ httpd/httpd/trunk/include/ap_mmn.h Thu Oct 8 21:42:13 2009
>> @@ -198,15 +198,17 @@
>> * 20090401.3 (2.3.3-dev) Added DAV options provider to mod_dav.h
>> * 20090925.0 (2.3.3-dev) Added server_rec::context and added
>> *server_rec * param to
>> ap_wait_or_timeout()
>> + * 20090925.1 (2.3.3-dev) Add optional function
>> ap_logio_get_last_bytes() to + * mod_logio
>> *
>> */
>>
>> #define MODULE_MAGIC_COOKIE 0x41503234UL /* "AP24" */
>>
>> #ifndef MODULE_MAGIC_NUMBER_MAJOR
>> -#define MODULE_MAGIC_NUMBER_MAJOR 20090401
>> +#define MODULE_MAGIC_NUMBER_MAJOR 20090925
>> #endif
>> -#define MODULE_MAGIC_NUMBER_MINOR 2 /* 0...n
>> */ +#define MODULE_MAGIC_NUMBER_MINOR 1 /*
>> 0...n */
>>
>
> I updated MODULE_MAGIC_NUMBER_MAJOR to match the comments. I hope this
> is correct?

Nope - adding a function doesn't cost us MMN MAJOR (and never delete MMN_MINOR,
it just falls back to zero).

MMN MAJOR jumps every time we encounter a change to an exported API which users
might be calling, which would result in erroneous calls to the function from a
module at an older build. It similarly changes when the fields of an existing
public data structure change (but not when they are extended).


sf at sfritsch

Oct 8, 2009, 3:09 PM

Post #3 of 3 (433 views)
Permalink
Re: svn commit: r823337 - in /httpd/httpd/trunk: CHANGES include/ap_mmn.h include/http_core.h modules/loggers/mod_logio.c server/scoreboard.c [In reply to]

On Friday 09 October 2009, William A. Rowe, Jr. wrote:
> >> * 20090925.0 (2.3.3-dev) Added server_rec::context and added *server_rec
> >> * param to ap_wait_or_timeout()
> >> + * 20090925.1 (2.3.3-dev) Add optional function ap_logio_get_last_bytes() to
> >> + * mod_logio
> >> *
> >> */
> >>
> >> #define MODULE_MAGIC_COOKIE 0x41503234UL /* "AP24" */
> >>
> >> #ifndef MODULE_MAGIC_NUMBER_MAJOR
> >> -#define MODULE_MAGIC_NUMBER_MAJOR 20090401
> >> +#define MODULE_MAGIC_NUMBER_MAJOR 20090925
> >> #endif
> >> -#define MODULE_MAGIC_NUMBER_MINOR 2 /* 0...n */
> >> +#define MODULE_MAGIC_NUMBER_MINOR 1 /* 0...n */
> >
> >
> > I updated MODULE_MAGIC_NUMBER_MAJOR to match the comments. I hope
> > this is correct?
>
> Nope - adding a function doesn't cost us MMN MAJOR (and never
> delete MMN_MINOR, it just falls back to zero).

Sorry, that's not what I meant (and kmail sucks at quoting preformatted
mails).

The comments indicated MMN 20090925.0 but the code was at 20090401.2.
Therefore I assumed this was just forgotten and jumped to 20090925.1.

>
> MMN MAJOR jumps every time we encounter a change to an exported API
> which users might be calling, which would result in erroneous
> calls to the function from a module at an older build. It
> similarly changes when the fields of an existing public data
> structure change (but not when they are extended).

Apache dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.