From: | Ashwin Agrawal <aagrawal(at)pivotal(dot)io> |
---|---|
To: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Declared but no defined functions |
Date: | 2019-07-08 18:57:18 |
Message-ID: | CALfoeiugb01cCQHRQ+DdTNCWeV=ihFHcuWtc=aEVQE3ou3DTtw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jul 6, 2019 at 4:32 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
wrote:
> Indeed. I've tried to search again with the following script and got
> more such functions.
>
> for func in `git ls-files | egrep "\w+\.h$" | xargs cat | egrep -v
> "(^typedef)|(DECLARE)|(BKI)" | egrep "^(extern )*[\_0-9A-Za-z]+
> [\_\*0-9a-zA-Z]+ ?\(.+\);$" | sed -e "s/\(^extern \)*[\_0-9A-Za-z]\+
> \([\_0-9A-Za-z\*]\+\) \{0,1\}(.*);$/\2(/g" | sed -e "s/\*//g"`
> do
> if [ "`git grep "$func" -- "*.c" | wc -l`" -lt 1 ];then
> echo $func
> fi
> done
>
>
Do we wish to make this a tool and have it in src/tools, either as part of
find_static tool after renaming that one to more generic name or
independent script.
From | Date | Subject | |
---|---|---|---|
Next Message | Julien Rouhaud | 2019-07-08 19:08:43 | Re: Add parallelism and glibc dependent only options to reindexdb |
Previous Message | Julien Rouhaud | 2019-07-08 18:47:01 | Re: progress report for ANALYZE |