Changeset: 0ca25de68fa3 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=0ca25de68fa3
Modified Files:
        geom/monetdb5/sfcgal.c
Branch: sfcgal
Log Message:

Improve error messages


diffs (168 lines):

diff --git a/geom/monetdb5/sfcgal.c b/geom/monetdb5/sfcgal.c
--- a/geom/monetdb5/sfcgal.c
+++ b/geom/monetdb5/sfcgal.c
@@ -513,7 +513,7 @@ sfcgal_from_geom(str *ret, const GEOSGeo
             break;
 
         default:
-            *ret = createException(MAL, "geom_to_sfcgal", "Unknown geometry 
type");
+            *ret = createException(MAL, "geom_from_sfcgal", "Unknown geometry 
type");
             return NULL;
     }
 }
@@ -652,7 +652,7 @@ geom_to_sfcgal(sfcgal_geometry_t **res, 
             break;
 
         default:
-            ret = createException(MAL, "geom2cgal", "Unknown geometry type");
+            ret = createException(MAL, "geom_to_sfcgal", "Unknown geometry 
type");
             *res = NULL;
     }
 
@@ -684,29 +684,29 @@ geom_sfcgal_extrude(wkb **res, wkb **geo
 
     if ( !(inGeos = wkb2geos(*geom)) ) {
         *res = NULL;
-               return createException(MAL, "geom.Extrude", "wkb2geos failed");
+               return createException(MAL, "geom_sfcgal_extrude", "wkb2geos 
failed");
     }
     if ((ret = wkbGetSRID(&srid, geom)) != MAL_SUCCEED) {
                *res = NULL;
-        msg = createException(MAL, "geom.Extrude", "wkbGetSRID failed: %s", 
ret);
+        msg = createException(MAL, "geom_sfcgal_extrude", "wkbGetSRID failed: 
%s", ret);
         GDKfree(ret);
                return msg;
     }
 
        if (( ret = geom_to_sfcgal(&inGeom, inGeos)) != MAL_SUCCEED) {
                *res = NULL;
-        msg = createException(MAL, "geom.Extrude", "geom_to_sfcgal failed:%s", 
ret);
+        msg = createException(MAL, "geom_sfcgal_extrude", "geom_to_sfcgal 
failed:%s", ret);
         GDKfree(ret);
                return msg;
        }
        if (!(outGeom = sfcgal_geometry_extrude (inGeom, *ex, *ey, *ez))) {
                *res = NULL;
-               return createException(MAL, "geom.Extrude", 
"sfcgal_geometry_extrude failed");
+               return createException(MAL, "geom_sfcgal_extrude", 
"sfcgal_geometry_extrude failed");
        }
 
     if ( ( ret = sfcgal_to_geom(&outGeos, outGeom, 0, srid, 0)) != 
MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Extrude", "GEOSExtrude 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_extrude", 
"sfcgal_to_geom failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
@@ -730,29 +730,29 @@ geom_sfcgal_straightSkeleton(wkb **res, 
 
     if ( !(inGeos = wkb2geos(*geom)) ) {
                *res = NULL;
-               return createException(MAL, "geom.StraightSkeleton", "wkb2geos 
failed");
+               return createException(MAL, "geom_sfcgal_straightSkeleton", 
"wkb2geos failed");
     }
     if ((ret = wkbGetSRID(&srid, geom)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Tesselate", "wkbGetSRID 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_straightSkeleton", 
"wkbGetSRID failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
 
        if ( (ret = geom_to_sfcgal(&inGeom, inGeos)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.StraightSkeleton", 
"geom_to_sfcgal failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_straightSkeleton", 
"geom_to_sfcgal failed:%s", ret);
         GDKfree(ret);
         return msg;
        }
        if (!(outGeom = sfcgal_geometry_straight_skeleton(inGeom))) {
                *res = NULL;
-               return createException(MAL, "geom.StraightSkeleton", 
"sfcgal_geometry_straight_skeleton failed");
+               return createException(MAL, "geom_sfcgal_straightSkeleton", 
"sfcgal_geometry_straight_skeleton failed");
        }
 
     if ( (ret = sfcgal_to_geom(&outGeos, outGeom, 0, srid, 0)) != MAL_SUCCEED) 
{
                *res = NULL;
-               msg = createException(MAL, "geom.Extrude", "sfcgal_to_geom 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_straightSkeleton", 
"sfcgal_to_geom failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
@@ -776,29 +776,29 @@ geom_sfcgal_tesselate(wkb **res, wkb **g
 
     if ( !(inGeos = wkb2geos(*geom)) ) {
                *res = NULL;
-               return createException(MAL, "geom.Tesselate", "wkb2geos 
failed");
+               return createException(MAL, "geom_sfcgal_tesselate", "wkb2geos 
failed");
     }
     if ( (ret = wkbGetSRID(&srid, geom)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Tesselate", "wkbGetSRID 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_tesselate", "wkbGetSRID 
failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
 
        if ( (ret = geom_to_sfcgal(&inGeom, inGeos)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Tesselate", "geom_to_sfcgal 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_tesselate", 
"geom_to_sfcgal failed:%s", ret);
         GDKfree(ret);
         return msg;
        }
        if (!(outGeom = sfcgal_geometry_tesselate(inGeom))) {
                *res = NULL;
-               return createException(MAL, "geom.Tesselate", 
"sfcgal_geometry_tesselate failed");
+               return createException(MAL, "geom_sfcgal_tesselate", 
"sfcgal_geometry_tesselate failed");
        }
 
     if (  (ret = sfcgal_to_geom(&outGeos, outGeom, 0, srid, 0)) != 
MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Tesselate", "sfcgal_to_geom 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_tesselate", 
"sfcgal_to_geom failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
@@ -822,31 +822,35 @@ geom_sfcgal_triangulate2DZ(wkb **res, wk
 
     if ( !(inGeos = wkb2geos(*geom)) ) {
                *res = NULL;
-               return createException(MAL, "geom.Triangulate2DZ", "wkb2geos 
failed");
+               return createException(MAL, "geom_sfcgal_triangulate2DZ", 
"wkb2geos failed");
     }
     if ( (ret = wkbGetSRID(&srid, geom)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Triangulate2DZ", "wkbGetSRID 
failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_triangulate2DZ", 
"wkbGetSRID failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
 
        if ( (ret = geom_to_sfcgal(&inGeom, inGeos)) != MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Triangulate2DZ", 
"geom_to_sfcgal failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_triangulate2DZ", 
"geom_to_sfcgal failed:%s", ret);
         GDKfree(ret);
         return msg;
        }
 
        if (!(outGeom = sfcgal_geometry_triangulate_2dz(inGeom))) {
-               *res = NULL;
-               return createException(MAL, "geom.Tesselate", 
"sfcgal_geometry_triangulate_2dz failed");
+        /*TODO:Make sure you can jump over it*/
+               //*res = NULL;
+               //return createException(MAL, "geom_sfcgal_triangulate2DZ", 
"sfcgal_geometry_triangulate_2dz failed");
+               if ((*res = wkbNULLcopy()) == NULL)
+                       throw(MAL, "geom_sfcgal_triangulate2DZ", 
MAL_MALLOC_FAIL);
+               return MAL_SUCCEED;
        }
 
 
     if (  (ret = sfcgal_to_geom(&outGeos, outGeom, 0, srid, *flag)) != 
MAL_SUCCEED) {
                *res = NULL;
-               msg = createException(MAL, "geom.Triangulate2DZ", 
"sfcgal_to_geom failed:%s", ret);
+               msg = createException(MAL, "geom_sfcgal_triangulate2DZ", 
"sfcgal_to_geom failed:%s", ret);
         GDKfree(ret);
         return msg;
     }
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to