Welcome! Log In Create A New Profile

Advanced

Re: [nginx] svn commit: r4327 - trunk/src/core

Maxim Dounin
December 06, 2011 11:28AM
Hello!

On Tue, Dec 06, 2011 at 04:23:34PM +0000, Valery Kholodkov wrote:

>
> FYI This function is used in upload module:
>
> https://github.com/vkholodkov/nginx-upload-module/blob/2.2/ngx_http_upload_module.c#L2656

No, it's in #else part of the:

#if defined nginx_version && nginx_version >= 8025

The prototype for the function was nuked in 0.8.25 during named
captures introduction (and corresponding regexp changes).

Maxim Dounin

>
> ----- vbart@nginx.com wrote:
> > Author: vbart
> > Date: 2011-12-06 15:49:40 +0000 (Tue, 06 Dec 2011)
> > New Revision: 4327
> >
> > Modified:
> > trunk/src/core/ngx_regex.c
> > Log:
> > Removed unused function ngx_regex_capture_count().
> > The function has been unused since r3326.
> >
> >
> > Modified: trunk/src/core/ngx_regex.c
> > ===================================================================
> > --- trunk/src/core/ngx_regex.c 2011-12-06 13:23:37 UTC (rev 4326)
> > +++ trunk/src/core/ngx_regex.c 2011-12-06 15:49:40 UTC (rev 4327)
> > @@ -137,23 +137,6 @@
> >
> >
> > ngx_int_t
> > -ngx_regex_capture_count(ngx_regex_t *re)
> > -{
> > - int rc, n;
> > -
> > - n = 0;
> > -
> > - rc = pcre_fullinfo(re, NULL, PCRE_INFO_CAPTURECOUNT, &n);
> > -
> > - if (rc < 0) {
> > - return (ngx_int_t) rc;
> > - }
> > -
> > - return (ngx_int_t) n;
> > -}
> > -
> > -
> > -ngx_int_t
> > ngx_regex_exec_array(ngx_array_t *a, ngx_str_t *s, ngx_log_t *log)
> > {
> > ngx_int_t n;
>
> --
> Regards,
> Valery Kholodkov
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[nginx] svn commit: r4327 - trunk/src/core

Anonymous User 1338 December 06, 2011 10:50AM

Re: [nginx] svn commit: r4327 - trunk/src/core

Valery Kholodkov 512 December 06, 2011 11:24AM

Re: [nginx] svn commit: r4327 - trunk/src/core

Maxim Dounin 443 December 06, 2011 11:28AM

Re: [nginx] svn commit: r4327 - trunk/src/core

Valery Kholodkov 566 December 06, 2011 11:36AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 296
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready