NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commitfd2e2d0

committed

Rewrite pg_size_pretty() to avoid compiler bug.

Convert it to use successive shifts right instead of increasing a divisor.This is probably a tad more efficient than the original coding, and it'snicer-looking than the previous patch because we don't need a special caseto avoid overflow in the last branch. But the real reason to do it is toavoid a Solaris compiler bug, as per results from buildfarm member moa.

1 parentc49e4ae commitfd2e2d0Copy full SHA for fd2e2d0

File tree

1 file changed

+13

-19

lines changed

src/backend/utils/adt
- dbsize.c

1 file changed

+13

-19

lines changed

`‎src/backend/utils/adt/dbsize.c`

Lines changed: 13 additions & 19 deletions

Original file line number	Diff line number	Diff line change
`@@ -466,39 +466,33 @@ pg_size_pretty(PG_FUNCTION_ARGS)`
`466`	`466`	`int64size=PG_GETARG_INT64(0);`
`467`	`467`	`charbuf[64];`
`468`	`468`	`int64limit=10*1024;`
`469`		`-int64mult=1;`
	`469`	`+int64limit2=limit*2-1;`
`470`	`470`
`471`		`-if (size<limit*mult)`
	`471`	`+if (size<limit)`
`472`	`472`	`snprintf(buf,sizeof(buf),INT64_FORMAT" bytes",size);`
`473`	`473`	`else`
`474`	`474`	`{`
`475`		`-mult *=1024;`
`476`		`-if (size<limit*mult)`
	`475`	`+size >>=9;/* keep one extra bit for rounding */`
	`476`	`+if (size<limit2)`
`477`	`477`	`snprintf(buf,sizeof(buf),INT64_FORMAT" kB",`
`478`		`- (size+mult /2) /mult);`
	`478`	`+ (size+1) /2);`
`479`	`479`	`else`
`480`	`480`	`{`
`481`		`-mult *=1024;`
`482`		`-if (size<limit*mult)`
	`481`	`+size >>=10;`
	`482`	`+if (size<limit2)`
`483`	`483`	`snprintf(buf,sizeof(buf),INT64_FORMAT" MB",`
`484`		`- (size+mult /2) /mult);`
	`484`	`+ (size+1) /2);`
`485`	`485`	`else`
`486`	`486`	`{`
`487`		`-mult *=1024;`
`488`		`-if (size<limit*mult)`
	`487`	`+size >>=10;`
	`488`	`+if (size<limit2)`
`489`	`489`	`snprintf(buf,sizeof(buf),INT64_FORMAT" GB",`
`490`		`- (size+mult /2) /mult);`
	`490`	`+ (size+1) /2);`
`491`	`491`	`else`
`492`	`492`	`{`
`493`		`-/* Here we have to worry about avoiding overflow */`
`494`		`-int64val;`
`495`		`-`
`496`		`-mult *=1024;`
`497`		`-val=size /mult;`
`498`		`-if ((size %mult) >= (mult /2))`
`499`		`-val++;`
	`493`	`+size >>=10;`
`500`	`494`	`snprintf(buf,sizeof(buf),INT64_FORMAT" TB",`
`501`		`-val);`
	`495`	`+(size+1) /2);`
`502`	`496`	`}`
`503`	`497`	`}`
`504`	`498`	`}`

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!

Commitfd2e2d0

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/dbsize.c`

0 commit comments