Changeset: 674895f8e497 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=674895f8e497
Added Files:
gdk/gdk_geomlogger.h
Modified Files:
gdk/Makefile.ag
gdk/gdk_logger.c
gdk/gdk_logger.h
geom/monetdb5/geom.c
geom/monetdb5/geom.h
geom/monetdb5/geom_upgrade.c
sql/backends/monet5/sql_upgrades.c
sql/common/sql_types.c
Branch: default
Log Message:
Move geom-specific logger declarations to separate include file.
diffs (163 lines):
diff --git a/gdk/Makefile.ag b/gdk/Makefile.ag
--- a/gdk/Makefile.ag
+++ b/gdk/Makefile.ag
@@ -29,6 +29,7 @@ lib_gdk = {
gdk_posix.c gdk_logger.c gdk_sample.c xoshiro256starstar.h \
gdk_private.h gdk_delta.h gdk_logger.h gdk_posix.h \
gdk_system.h gdk_system_private.h gdk_tm.h gdk_storage.h \
+ gdk_geomlogger.h \
gdk_group.c \
gdk_imprints.c gdk_imprints.h \
gdk_join.c gdk_project.c \
diff --git a/gdk/gdk_geomlogger.h b/gdk/gdk_geomlogger.h
new file mode 100644
--- /dev/null
+++ b/gdk/gdk_geomlogger.h
@@ -0,0 +1,23 @@
+/*
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * Copyright 1997 - July 2008 CWI, August 2008 - 2020 MonetDB B.V.
+ */
+
+#ifndef _GEOMLOGGER_H_
+#define _GEOMLOGGER_H_
+
+typedef int (*geomcatalogfix_fptr)(void *, int);
+gdk_export void geomcatalogfix_set(geomcatalogfix_fptr);
+gdk_export geomcatalogfix_fptr geomcatalogfix_get(void);
+
+typedef str (*geomsqlfix_fptr)(int);
+gdk_export void geomsqlfix_set(geomsqlfix_fptr);
+gdk_export geomsqlfix_fptr geomsqlfix_get(void);
+
+gdk_export void geomversion_set(void);
+gdk_export bool geomversion_get(void);
+
+#endif /* _GEOMLOGGER_H_ */
diff --git a/gdk/gdk_logger.c b/gdk/gdk_logger.c
--- a/gdk/gdk_logger.c
+++ b/gdk/gdk_logger.c
@@ -126,10 +126,12 @@ typedef struct logformat_t {
typedef enum {LOG_OK, LOG_EOF, LOG_ERR} log_return;
+#include "gdk_geomlogger.h"
+
/* When reading an old format database, we may need to read the geom
* Well-known Binary (WKB) type differently. This variable is used to
* indicate that to the function wkbREAD during reading of the log. */
-static int geomisoldversion;
+static bool geomisoldversion;
static gdk_return bm_commit(logger *lg);
static gdk_return tr_grow(trans *tr);
@@ -2239,7 +2241,7 @@ logger_load(int debug, const char *fn, c
goto error;
/* done reading the log, revert to "normal" behavior */
- geomisoldversion = 0;
+ geomisoldversion = false;
}
return GDK_SUCCEED;
@@ -3314,9 +3316,11 @@ geomsqlfix_get(void)
void
geomversion_set(void)
{
- geomisoldversion = 1;
+ geomisoldversion = true;
}
-int geomversion_get(void)
+
+bool
+geomversion_get(void)
{
return geomisoldversion;
}
diff --git a/gdk/gdk_logger.h b/gdk/gdk_logger.h
--- a/gdk/gdk_logger.h
+++ b/gdk/gdk_logger.h
@@ -132,15 +132,4 @@ gdk_export log_bid logger_find_bat(logge
gdk_export gdk_return logger_upgrade_bat(logger *lg, const char *name, char
tpe, oid id)
__attribute__ ((__warn_unused_result__));
-typedef int (*geomcatalogfix_fptr)(void *, int);
-gdk_export void geomcatalogfix_set(geomcatalogfix_fptr);
-gdk_export geomcatalogfix_fptr geomcatalogfix_get(void);
-
-typedef str (*geomsqlfix_fptr)(int);
-gdk_export void geomsqlfix_set(geomsqlfix_fptr);
-gdk_export geomsqlfix_fptr geomsqlfix_get(void);
-
-gdk_export void geomversion_set(void);
-gdk_export int geomversion_get(void);
-
#endif /*_LOGGER_H_*/
diff --git a/geom/monetdb5/geom.c b/geom/monetdb5/geom.c
--- a/geom/monetdb5/geom.c
+++ b/geom/monetdb5/geom.c
@@ -12,6 +12,7 @@
*/
#include "geom.h"
+#include "gdk_logger.h"
#include "mal_exception.h"
int TYPE_mbr;
@@ -2064,6 +2065,8 @@ geoGetType(char **res, int *info, int *f
/* returns a pointer to a nil-mbr. */
static mbr mbrNIL; /* to be filled in */
+#include "gdk_geomlogger.h"
+
str
geom_prelude(void *ret)
{
diff --git a/geom/monetdb5/geom.h b/geom/monetdb5/geom.h
--- a/geom/monetdb5/geom.h
+++ b/geom/monetdb5/geom.h
@@ -23,8 +23,6 @@
#include <math.h>
#include <time.h>
-#include "gdk_logger.h"
-
#ifdef WIN32
#ifndef LIBGEOM
#define geom_export extern __declspec(dllimport)
diff --git a/geom/monetdb5/geom_upgrade.c b/geom/monetdb5/geom_upgrade.c
--- a/geom/monetdb5/geom_upgrade.c
+++ b/geom/monetdb5/geom_upgrade.c
@@ -12,6 +12,7 @@
#include "geom.h"
+#include "gdk_logger.h"
static char *
N(char *buf, const char *pre, const char *schema, const char *post)
diff --git a/sql/backends/monet5/sql_upgrades.c
b/sql/backends/monet5/sql_upgrades.c
--- a/sql/backends/monet5/sql_upgrades.c
+++ b/sql/backends/monet5/sql_upgrades.c
@@ -21,6 +21,7 @@
#include "rel_semantic.h"
#include "rel_unnest.h"
#include "rel_optimizer.h"
+#include "gdk_geomlogger.h"
#include "rel_remote.h"
#include "mal_authorize.h"
diff --git a/sql/common/sql_types.c b/sql/common/sql_types.c
--- a/sql/common/sql_types.c
+++ b/sql/common/sql_types.c
@@ -23,6 +23,7 @@
#ifdef HAVE_HGE
#include "mal.h" /* for have_hge */
#endif
+#include "gdk_geomlogger.h" /* for geomcatalogfix_get */
list *aliases = NULL;
list *types = NULL;
_______________________________________________
checkin-list mailing list
[email protected]
https://www.monetdb.org/mailman/listinfo/checkin-list