Fix CreateTableSpace() so it will compile without HAVE_SYMLINK.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Sep 2015 20:15:38 +0000 (16:15 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Sep 2015 20:15:38 +0000 (16:15 -0400)
This has been broken since 9.3 (commit 82b1b213cad3a69c to be exact),
which suggests that nobody is any longer using a Windows build system that
doesn't provide a symlink emulation.  Still, it's wrong on its own terms,
so repair.

YUriy Zhuravlev

src/backend/commands/tablespace.c

index ff0d904b7a865b32585a81d7790dcd79d3004ae4..7588b7ae264332a3bab13210d6b3541b88726516 100644 (file)
@@ -383,13 +383,14 @@ CreateTableSpace(CreateTableSpaceStmt *stmt)
 
    /* We keep the lock on pg_tablespace until commit */
    heap_close(rel, NoLock);
+
+   return tablespaceoid;
 #else                          /* !HAVE_SYMLINK */
    ereport(ERROR,
            (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
             errmsg("tablespaces are not supported on this platform")));
+   return InvalidOid;          /* keep compiler quiet */
 #endif   /* HAVE_SYMLINK */
-
-   return tablespaceoid;
 }
 
 /*