mirror of
				https://github.com/python/cpython.git
				synced 2025-11-04 11:49:12 +00:00 
			
		
		
		
	gh-109631: Allow interruption of short repeated regex matches (GH-109867)
Counting for signal checking now continues in new match from the point where it ended in the previous match instead of starting from 0.
This commit is contained in:
		
							parent
							
								
									7c61a361fc
								
							
						
					
					
						commit
						8ac2085b80
					
				
					 3 changed files with 8 additions and 2 deletions
				
			
		| 
						 | 
					@ -0,0 +1,3 @@
 | 
				
			||||||
 | 
					:mod:`re` functions such as :func:`re.findall`, :func:`re.split`,
 | 
				
			||||||
 | 
					:func:`re.search` and :func:`re.sub` which perform short repeated matches
 | 
				
			||||||
 | 
					can now be interrupted by user.
 | 
				
			||||||
| 
						 | 
					@ -95,6 +95,7 @@ typedef struct {
 | 
				
			||||||
    size_t data_stack_base;
 | 
					    size_t data_stack_base;
 | 
				
			||||||
    /* current repeat context */
 | 
					    /* current repeat context */
 | 
				
			||||||
    SRE_REPEAT *repeat;
 | 
					    SRE_REPEAT *repeat;
 | 
				
			||||||
 | 
					    unsigned int sigcount;
 | 
				
			||||||
} SRE_STATE;
 | 
					} SRE_STATE;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
typedef struct {
 | 
					typedef struct {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -564,7 +564,7 @@ SRE(match)(SRE_STATE* state, const SRE_CODE* pattern, int toplevel)
 | 
				
			||||||
    Py_ssize_t alloc_pos, ctx_pos = -1;
 | 
					    Py_ssize_t alloc_pos, ctx_pos = -1;
 | 
				
			||||||
    Py_ssize_t ret = 0;
 | 
					    Py_ssize_t ret = 0;
 | 
				
			||||||
    int jump;
 | 
					    int jump;
 | 
				
			||||||
    unsigned int sigcount=0;
 | 
					    unsigned int sigcount = state->sigcount;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    SRE(match_context)* ctx;
 | 
					    SRE(match_context)* ctx;
 | 
				
			||||||
    SRE(match_context)* nextctx;
 | 
					    SRE(match_context)* nextctx;
 | 
				
			||||||
| 
						 | 
					@ -1567,8 +1567,10 @@ exit:
 | 
				
			||||||
    ctx_pos = ctx->last_ctx_pos;
 | 
					    ctx_pos = ctx->last_ctx_pos;
 | 
				
			||||||
    jump = ctx->jump;
 | 
					    jump = ctx->jump;
 | 
				
			||||||
    DATA_POP_DISCARD(ctx);
 | 
					    DATA_POP_DISCARD(ctx);
 | 
				
			||||||
    if (ctx_pos == -1)
 | 
					    if (ctx_pos == -1) {
 | 
				
			||||||
 | 
					        state->sigcount = sigcount;
 | 
				
			||||||
        return ret;
 | 
					        return ret;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
    DATA_LOOKUP_AT(SRE(match_context), ctx, ctx_pos);
 | 
					    DATA_LOOKUP_AT(SRE(match_context), ctx, ctx_pos);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    switch (jump) {
 | 
					    switch (jump) {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue