NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitcb0bcf4

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 parentc02bc63 commitcb0bcf4Copy full SHA for cb0bcf4

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
`@@ -473,39 +473,33 @@ pg_size_pretty(PG_FUNCTION_ARGS)`
`473`	`473`	`int64size=PG_GETARG_INT64(0);`
`474`	`474`	`charbuf[64];`
`475`	`475`	`int64limit=10*1024;`
`476`		`-int64mult=1;`
	`476`	`+int64limit2=limit*2-1;`
`477`	`477`
`478`		`-if (size<limit*mult)`
	`478`	`+if (size<limit)`
`479`	`479`	`snprintf(buf,sizeof(buf),INT64_FORMAT" bytes",size);`
`480`	`480`	`else`
`481`	`481`	`{`
`482`		`-mult *=1024;`
`483`		`-if (size<limit*mult)`
	`482`	`+size >>=9;/* keep one extra bit for rounding */`
	`483`	`+if (size<limit2)`
`484`	`484`	`snprintf(buf,sizeof(buf),INT64_FORMAT" kB",`
`485`		`- (size+mult /2) /mult);`
	`485`	`+ (size+1) /2);`
`486`	`486`	`else`
`487`	`487`	`{`
`488`		`-mult *=1024;`
`489`		`-if (size<limit*mult)`
	`488`	`+size >>=10;`
	`489`	`+if (size<limit2)`
`490`	`490`	`snprintf(buf,sizeof(buf),INT64_FORMAT" MB",`
`491`		`- (size+mult /2) /mult);`
	`491`	`+ (size+1) /2);`
`492`	`492`	`else`
`493`	`493`	`{`
`494`		`-mult *=1024;`
`495`		`-if (size<limit*mult)`
	`494`	`+size >>=10;`
	`495`	`+if (size<limit2)`
`496`	`496`	`snprintf(buf,sizeof(buf),INT64_FORMAT" GB",`
`497`		`- (size+mult /2) /mult);`
	`497`	`+ (size+1) /2);`
`498`	`498`	`else`
`499`	`499`	`{`
`500`		`-/* Here we have to worry about avoiding overflow */`
`501`		`-int64val;`
`502`		`-`
`503`		`-mult *=1024;`
`504`		`-val=size /mult;`
`505`		`-if ((size %mult) >= (mult /2))`
`506`		`-val++;`
	`500`	`+size >>=10;`
`507`	`501`	`snprintf(buf,sizeof(buf),INT64_FORMAT" TB",`
`508`		`-val);`
	`502`	`+(size+1) /2);`
`509`	`503`	`}`
`510`	`504`	`}`
`511`	`505`	`}`

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!

Commitcb0bcf4

File tree

1 file changed

1 file changed

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

0 commit comments