Why does pgindent sometimes insert whitespace into the return type
part of a function definition?  Here's an example from the last
pgindent run:

RCS file: /home/projects/pgsql/cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v
retrieving revision 1.103
retrieving revision 1.104
diff -c -r1.103 -r1.104
*** pgsql/src/backend/optimizer/plan/createplan.c       2001/01/24 19:42:58     1.103
--- pgsql/src/backend/optimizer/plan/createplan.c       2001/03/22 03:59:36     1.104
***************
*** 1493,1499 ****
        return make_sort(sort_tlist, lefttree, numsortkeys);
  }
  
! Material *
  make_material(List *tlist, Plan *lefttree)
  {
        Material   *node = makeNode(Material);
--- 1495,1501 ----
        return make_sort(sort_tlist, lefttree, numsortkeys);
  }
  
! Material   *
  make_material(List *tlist, Plan *lefttree)
  {
        Material   *node = makeNode(Material);


			regards, tom lane

Tue, 05 Jun 2001 01:18:05 -0400
Tom Lane <tgl@sss.pgh.pa.us>
Another pgindent gripe

OK, fixed.  Not sure why indent likes to add the tab, but I had some
code to change that:

# move trailing * in function return type
        sed 's;^\([A-Za-z_][^   ]*\)[   ][      ]*\*$;\1 *;' |
                            ^^^^
Turns out the marked area was missing a space, and was only tab.  Making
it tab and space in the indicated brackets fixed the problem.  My guess
is that the failure was only happening when the function return type was
exaclty eight characters, like "Material".


> Why does pgindent sometimes insert whitespace into the return type
> part of a function definition?  Here's an example from the last
> pgindent run:
> 
> RCS file: /home/projects/pgsql/cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v
> retrieving revision 1.103
> retrieving revision 1.104
> diff -c -r1.103 -r1.104
> *** pgsql/src/backend/optimizer/plan/createplan.c       2001/01/24 19:42:58     1.103
> --- pgsql/src/backend/optimizer/plan/createplan.c       2001/03/22 03:59:36     1.104
> ***************
> *** 1493,1499 ****
>         return make_sort(sort_tlist, lefttree, numsortkeys);
>   }
>   
> ! Material *
>   make_material(List *tlist, Plan *lefttree)
>   {
>         Material   *node = makeNode(Material);
> --- 1495,1501 ----
>         return make_sort(sort_tlist, lefttree, numsortkeys);
>   }
>   
> ! Material   *
>   make_material(List *tlist, Plan *lefttree)
>   {
>         Material   *node = makeNode(Material);
> 
> 
> 			regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
> 

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Wed, 6 Jun 2001 00:51:56 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: Another pgindent gripe