NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd4a4c3d

committed

Suppress compiler warning in new jsonb_plperl code.

Some compilers are evidently pickier than others about whether Perl'sI32 typedef should be considered equivalent to int. Dodge the problemby using a separate variable; the prior coding was a bit confusing anyway.Per buildfarm. Note this does nothing to fix the test failures due toSV_to_JsonbValue not covering enough variable types.

1 parent242408d commitd4a4c3dCopy full SHA for d4a4c3d

File tree

1 file changed

-2

lines changed

contrib/jsonb_plperl
- jsonb_plperl.c

1 file changed

-2

lines changed

`‎contrib/jsonb_plperl/jsonb_plperl.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -150,16 +150,19 @@ HV_to_JsonbValue(HV obj, JsonbParseState *jsonb_state)`
`150`	`150`	`dTHX;`
`151`	`151`	`JsonbValuekey;`
`152`	`152`	`SV*val;`
	`153`	`+char*kstr;`
	`154`	`+I32klen;`
`153`	`155`
`154`	`156`	`key.type=jbvString;`
`155`	`157`
`156`	`158`	`pushJsonbValue(jsonb_state,WJB_BEGIN_OBJECT,NULL);`
`157`	`159`
`158`	`160`	`(void)hv_iterinit(obj);`
`159`	`161`
`160`		`-while ((val=hv_iternextsv(obj,&key.val.string.val,&key.val.string.len)))`
	`162`	`+while ((val=hv_iternextsv(obj,&kstr,&klen)))`
`161`	`163`	`{`
`162`		`-key.val.string.val=pnstrdup(key.val.string.val,key.val.string.len);`
	`164`	`+key.val.string.val=pnstrdup(kstr,klen);`
	`165`	`+key.val.string.len=klen;`
`163`	`166`	`pushJsonbValue(jsonb_state,WJB_KEY,&key);`
`164`	`167`	`(void)SV_to_JsonbValue(val,jsonb_state, false);`
`165`	`168`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd4a4c3d

File tree

1 file changed

1 file changed

`‎contrib/jsonb_plperl/jsonb_plperl.c`

0 commit comments