summaryrefslogtreecommitdiffstats
path: root/valgrind.supp
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-10-03 17:54:08 +0200
committerDan McGee <dan@archlinux.org>2011-10-14 03:59:16 +0200
commit8605284e0d1b70d9845ca4c6362d7d451f503d32 (patch)
tree91068c025f1b0c38d14a13132208dddf2b5aee1e /valgrind.supp
parent9934052b54fd4dcd2339a2beb87ccc81b5e144bd (diff)
downloadpacman-8605284e0d1b70d9845ca4c6362d7d451f503d32.tar.gz
pacman-8605284e0d1b70d9845ca4c6362d7d451f503d32.tar.xz
Use puts() instead of no-op printf() where applicable
This replaces several printf calls of the following styles: printf("%s", ...); printf("some fixed string"); printf("x"); We can use either fputs() or putchar() here to do the same thing without incurring the overhead of the printf format parser. The biggest gain here comes when we are calling the print function in a loop repeatedly; notably when printing local package files. $ /usr/bin/time ./pacman-before -Ql | md5sum 0.25user 0.04system 0:00.30elapsed 98%CPU $ /usr/bin/time ./pacman-after -Ql | md5sum 0.17user 0.06system 0:00.25elapsed 94%CPU $ /usr/bin/time ./pacman-before -Qlq | md5sum 0.20user 0.05system 0:00.26elapsed 98%CPU $ /usr/bin/time ./pacman-after -Qlq | md5sum 0.15user 0.05system 0:00.23elapsed 93%CPU So '-Ql' shows a 17% improvement while '-Qlq' shows a 13% improvement on 382456 total files. Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'valgrind.supp')
0 files changed, 0 insertions, 0 deletions