Add a useless return statement to suppress a warning seen with some

tglsfdc · tglsfdc · commit f1c87830b5e3 · 2007-10-22T01:02:22.000Z
versions of gcc (I'm seeing it with Apple's gcc 4.0.1).  I think the
reason we did not see this before was that the assert() macros in the
regex code were all no-ops till recently.
diff --git a/src/backend/regex/regc_lex.c b/src/backend/regex/regc_lex.c
@@ -28,7 +28,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.5 2005/10/15 02:49:24 momjian Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.6 2007/10/22 01:02:22 tgl Exp $
  *
  */
 
@@ -1071,6 +1071,7 @@ brenext(struct vars * v,
 	}
 
 	assert(NOTREACHED);
+	return 0;
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@`
`28`	`28`	`* OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF`
`29`	`29`	`* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.`
`30`	`30`	`*`
`31`		`- * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.5 2005/10/15 02:49:24 momjian Exp $`
	`31`	`+ * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.6 2007/10/22 01:02:22 tgl Exp $`
`32`	`32`	`*`
`33`	`33`	`*/`
`34`	`34`
`@@ -1071,6 +1071,7 @@ brenext(struct vars * v,`
`1071`	`1071`	`}`
`1072`	`1072`
`1073`	`1073`	`assert(NOTREACHED);`
	`1074`	`+ return 0;`
`1074`	`1075`	`}`
`1075`	`1076`
`1076`	`1077`	`/*`