Thread

Commits

  1. Remove useless "return;" lines

  1. remove useless returns

    Alvaro Herrera <alvherre@2ndquadrant.com> — 2019-11-28T14:46:53Z

    Here's a silly insomnia-inspired little patch that removes useless
    "return;" lines from some routines.  (I left some alone, because they
    seemed to be there more for documentation purposes, such as the ones in
    from_char_set_mode and from_char_set_int; also regcomp.c since the
    pedigree there is unclear.)
    
    This seems pretty uncontroversial, so I'm not thinking of waiting over
    the US holidays to get opinions about it ...
    
    -- 
    Álvaro Herrera                PostgreSQL Expert, https://www.2ndQuadrant.com/
    
  2. Re: remove useless returns

    Tom Lane <tgl@sss.pgh.pa.us> — 2019-11-28T15:26:15Z

    Alvaro Herrera <alvherre@2ndquadrant.com> writes:
    > Here's a silly insomnia-inspired little patch that removes useless
    > "return;" lines from some routines.
    
    +1, I always thought that was poor style.
    
    			regards, tom lane