Commited740b0

committed

Remove redundant test.

The condition "context_start < context_end" is strictly weakerthan "context_end - context_start >= 50", so we don't need both.Oversight in commitffd3944, noted by tanghy.fnst.In passing, line-wrap a nearby test to make it more readable.Discussion:https://postgr.es/m/OS0PR01MB61137C4054774F44E3A9DC89FBC69@OS0PR01MB6113.jpnprd01.prod.outlook.com

1 parent11c1239 commited740b0Copy full SHA for ed740b0

File tree

-2

lines changed

-2

lines changed

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -652,7 +652,7 @@ report_json_context(JsonLexContext *lex)`
`652`	`652`	`context_end=lex->token_terminator;`
`653`	`653`
`654`	`654`	`/* Advance until we are close enough to context_end */`
`655`		`-while (context_end-context_start >=50&&context_start<context_end)`
	`655`	`+while (context_end-context_start >=50)`
`656`	`656`	`{`
`657`	`657`	`/* Advance to next multibyte character */`
`658`	`658`	`if (IS_HIGHBIT_SET(*context_start))`
`@@ -680,7 +680,9 @@ report_json_context(JsonLexContext *lex)`
`680`	`680`	`* suffixing "..." if not ending at end of line.`
`681`	`681`	`*/`
`682`	`682`	`prefix= (context_start>line_start) ?"..." :"";`
`683`		`-suffix= (lex->token_type!=JSON_TOKEN_END&&context_end-lex->input<lex->input_length&&context_end!='\n'&&context_end!='\r') ?"..." :"";`
	`683`	`+suffix= (lex->token_type!=JSON_TOKEN_END&&`
	`684`	`+context_end-lex->input<lex->input_length&&`
	`685`	`+context_end!='\n'&&context_end!='\r') ?"..." :"";`
`684`	`686`
`685`	`687`	`returnerrcontext("JSON data, line %d: %s%s%s",`
`686`	`688`	`lex->line_number,prefix,ctxt,suffix);`

Comments

(0)