Commitff4b5c5

committed

do not use recursion in Cover() - ~2% performance

1 parent3283aed commitff4b5c5Copy full SHA for ff4b5c5

File tree

-8

lines changed

-8

lines changed

Lines changed: 6 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -656,15 +656,13 @@ Cover(DocRepresentation doc, uint32 len, QueryRepresentation qr,`
`656`	`656`	`Extention*ext)`
`657`	`657`	`{`
`658`	`658`	`DocRepresentation*ptr;`
`659`		`-intlastpos=ext->pos;`
	`659`	`+intlastpos;`
`660`	`660`	`inti;`
`661`		`-boolfound= false;`
	`661`	`+boolfound;`
`662`	`662`
`663`		`-/*`
`664`		`- * since this function recurses, it could be driven to stack overflow.`
`665`		`- * (though any decent compiler will optimize away the tail-recursion.`
`666`		`- */`
`667`		`-check_stack_depth();`
	`663`	`+restart:`
	`664`	`+lastpos=ext->pos;`
	`665`	`+found= false;`
`668`	`666`
`669`	`667`	`memset(qr->operandexist,0,sizeof(bool)*qr->lenght);`
`670`	`668`
`@@ -742,7 +740,7 @@ Cover(DocRepresentation doc, uint32 len, QueryRepresentation qr,`
`742`	`740`	`}`
`743`	`741`
`744`	`742`	`ext->pos++;`
`745`		`-returnCover(doc,len,qr,ext);`
	`743`	`+gotorestart;`
`746`	`744`	`}`
`747`	`745`
`748`	`746`	`staticDocRepresentation*`

Comments

(0)