Re: [PATCH 0/4] bitpos expansion summary reloaded

[Jan Kratochvil <jan.kratochvil@redhat.com>]

On Wed, 03 Oct 2012 15:11:55 +0200, Siddhesh Poyarekar wrote:
[...]
> make 2> build-warnings.out

Probably only in gdb/ directory otherwise it would create more needless output
to deal with.


[...]
> I then processed the two files with this wonderful looking command:
> 
> diff -u build-master.out build-bitpos.out |
> 	perl -e 'while(<>){s{^(.)(.*)}{print "$2$1\n"}egm}' | sort |
> 	perl -e 'while(<>){s{^([^\]]+\])(.)$}{print "$2$1\n"}egm}' |
> 	grep "^[-+]" | ./checkreport2.pl | grep -v "^----" |
> 	grep -v "^+++" | grep "^+" |
> 	perl -e \
> 	'while(<>){s{^([^\/]+/)+([^:]+:[^:]+):.*}{print "$2\n"}egm}' |
> 	while read loc; do
> 		grep -q "$loc" splint-bitpos3.locdiff.processed

This 'grep' has too relaxed search string, for example warnings in "main.c"
(there are some 'conversion' warnings) will match _29_ other files creating
false FOUND lines and therefore fogetting about possibly valid type safety
conversion problems.

There should be sure some fgrep -q "($loc:" or so.


Thanks,
Jan


break-interp-main.c
dmsym_main.c
dw2-cp-infcall-ref-static-main.c
dw2-entry-value-main.c
dw2-inline-param-main.c
dw2-noloc-main.c
dw2-param-error-main.c
dw2-ref-missing-frame-main.c
dw2-unresolved-main.c
gcore-relro-main.c
gdb1555-main.c
jit-dlmain.c
jit-main.c
main.c
mi-main.c
mips16-thunks-inmain.c
mips16-thunks-main.c
mips16-thunks-sinmain.c
print-file-var-main.c
shmain.c
skip-solib-main.c
solib-display-main.c
solib-list-main.c
solib-main.c
solib-overlap-main.c
solib-symbol-main.c
tls-main.c
tls-var-main.c
type-opaque-main.c