Re: Declared but no defined functions - Mailing list pgsql-hackers

From Ashwin Agrawal
Subject Re: Declared but no defined functions
Date
Msg-id CALfoeiugb01cCQHRQ+DdTNCWeV=ihFHcuWtc=aEVQE3ou3DTtw@mail.gmail.com
Whole thread Raw
In response to Re: Declared but no defined functions  (Masahiko Sawada <sawada.mshk@gmail.com>)
Responses Re: Declared but no defined functions  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers

On Sat, Jul 6, 2019 at 4:32 PM Masahiko Sawada <sawada.mshk@gmail.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.

pgsql-hackers by date:

Previous
From: Julien Rouhaud
Date:
Subject: Re: progress report for ANALYZE
Next
From: Julien Rouhaud
Date:
Subject: Re: Add parallelism and glibc dependent only options to reindexdb