Add find_typedef comments for Linux.
authorBruce Momjian
Tue, 18 Mar 2008 23:23:08 +0000 (23:23 +0000)
committerBruce Momjian
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}'