]> granicus.if.org Git - postgresql/commitdiff
Add find_typedef comments for Linux.
authorBruce Momjian <bruce@momjian.us>
Tue, 18 Mar 2008 23:23:08 +0000 (23:23 +0000)
committerBruce Momjian <bruce@momjian.us>
Tue, 18 Mar 2008 23:23:08 +0000 (23:23 +0000)
src/tools/find_typedef

index ad14aeea48805c5578a1c1c8901a9edd98ad75f1..33e7fc677dce626d33de1ec4a41466851a751060 100755 (executable)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-# $PostgreSQL: pgsql/src/tools/find_typedef,v 1.10 2008/03/18 23:04:34 momjian Exp $
+# $PostgreSQL: pgsql/src/tools/find_typedef,v 1.11 2008/03/18 23:23:08 momjian Exp $
 
 # This script attempts to find all typedef's in the postgres binaries
 # by using 'nm' to report all typedef debugging symbols.
@@ -39,7 +39,11 @@ for DIR
 do     # if objdump -W is recognized, only one line of error should appear
        if [ `objdump -W 2>&1 | wc -l` -eq 1 ]
        then    # Linux
-               # unfortunately the Linux version doesn't show unreferenced typedefs
+               # Unfortunately the Linux version doesn't show unreferenced typedefs.
+               # The problem is that they are still in the source code so should be
+               # indented properly.  However, I think pgindent only cares about
+               # the typedef references, not the definitions, so I think it might
+               # be fine
                objdump -W "$DIR"/* |
                egrep -A3 '(DW_TAG_typedef|DW_TAG_structure_type|DW_TAG_union_type)' |
                awk ' $2 == "DW_AT_name" {print $NF}'