Fix hard crash in case of a unwritable Latex Output file. Print warnign instead of hard stop

AUR/PKGBUILD

@@ -1,7 +1,7 @@
 # Maintainer: Mario Hüttel <mario (dot) huettel (!) gmx (dot) net>
 
 pkgname=gds-render
-pkgver=v1.2_10_g00d02b8
+pkgver=20191020.403.448de30
 pkgrel=1
 pkgdesc="Conversion tool for converting GDS layout files into TikZ Code and PDF"
 arch=('i686' 'x86_64')
@@ -15,8 +15,8 @@ sha1sums=('SKIP')
 
 pkgver () {
 	_date=`date +"%Y%m%d"`
-	cd "${srcdir}/${pkgname}-git/version"
-	echo "$(echo "$(sh ./generate-version-string.sh)" | sed 's/-/_/g')"
+	cd "${srcdir}/${pkgname}-git"
+	echo "$_date.$(git rev-list --count master).$(git rev-parse --short master)"
 }
 
 build () {

CMakeLists.txt

@@ -33,6 +33,8 @@ pkg_check_modules(CAIRO REQUIRED cairo)
 include_directories(${GLIB_INCLUDE_DIRS} ${GTK3_INCLUDE_DIRS} ${CAIRO_INCLUDE_DIRS} ${CMAKE_CURRENT_SOURCE_DIR}/include)
 add_subdirectory(plugins)
 
+add_compile_options(-Wall -Wextra -Wold-style-declaration -Wuninitialized -Wmaybe-uninitialized -Wunused-parameter)
+
 IF(CMAKE_BUILD_TYPE STREQUAL "Debug")
         message("${Yellow}Debug mode for translations used!${ColorReset}")
 	add_definitions(-DGETTEXT_PACKAGE=\"gds-render\" -DLOCALEDATADIR=\"${CMAKE_CURRENT_BINARY_DIR}/translations/output\")
@@ -42,7 +44,13 @@ else(CMAKE_BUILD_TYPE STREQUAL "Debug")
 	add_definitions(-DGETTEXT_PACKAGE=\"gds-render\" -DLOCALEDATADIR=\"/usr/share\")
 ENDIF(CMAKE_BUILD_TYPE STREQUAL "Debug")
 
+add_subdirectory(resources)
+add_subdirectory(doxygen)
+add_subdirectory(translations)
+add_subdirectory(version)
 
+link_directories(${GLIB_LINK_DIRS} ${GTK3_LINK_DIRS} ${CAIRO_LINK_DIRS})
+add_definitions(${GLIB2_CFLAGS_OTHER})
 
 aux_source_directory("widgets" LAYER_SOURCES)
 aux_source_directory("cell-selector" CELL_SELECTOR_SOURCES)
@@ -66,29 +74,15 @@ set(SOURCE_GENERATED
   ${CMAKE_CURRENT_BINARY_DIR}/resources/resources.c
 )
 
-link_directories(${GLIB_LINK_DIRS} ${GTK3_LINK_DIRS} ${CAIRO_LINK_DIRS})
 SET_SOURCE_FILES_PROPERTIES(${SOURCE_GENERATED} PROPERTIES GENERATED 1)
 
-add_subdirectory(test)
-
-add_compile_options(-Wall -Wextra -Wold-style-declaration -Wuninitialized -Wmaybe-uninitialized -Wunused-parameter)
-
-add_subdirectory(resources)
-add_subdirectory(doxygen)
-add_subdirectory(translations)
-add_subdirectory(version)
-
-link_directories(${GLIB_LINK_DIRS} ${GTK3_LINK_DIRS} ${CAIRO_LINK_DIRS})
-add_definitions(${GLIB2_CFLAGS_OTHER})
-
 add_executable(${PROJECT_NAME} ${SOURCE} ${SOURCE_GENERATED})
 add_dependencies(${PROJECT_NAME} glib-resources)
 add_dependencies(${PROJECT_NAME} version)
 add_dependencies(${PROJECT_NAME} translations)
 
+target_link_libraries(${PROJECT_NAME} ${GLIB_LDFLAGS} ${GTK3_LDFLAGS} ${CAIRO_LDFLAGS} m version ${CMAKE_DL_LIBS})
 install (TARGETS ${PROJECT_NAME} 
 	RUNTIME	
 		DESTINATION bin
 	)
-target_link_libraries(${PROJECT_NAME} ${GLIB_LDFLAGS} ${GTK3_LDFLAGS} ${CAIRO_LDFLAGS} m version ${CMAKE_DL_LIBS})
-

cell-selector/lib-cell-renderer.c

@@ -88,13 +88,11 @@ static void lib_cell_renderer_set_property(GObject      *object,
 		g_value_init(&val, G_TYPE_STRING);
 		g_value_set_string(&val, ((struct gds_library *)g_value_get_pointer(value))->name);
 		g_object_set_property(object, "text", &val);
-		g_value_unset(&val);
 		break;
 	case PROP_CELL:
 		g_value_init(&val, G_TYPE_STRING);
 		g_value_set_string(&val, ((struct gds_cell *)g_value_get_pointer(value))->name);
 		g_object_set_property(object, "text", &val);
-		g_value_unset(&val);
 		break;
 	case PROP_ERROR_LEVEL:
 		/* Set cell color according to error level */
@@ -102,7 +100,6 @@ static void lib_cell_renderer_set_property(GObject      *object,
 		convert_error_level_to_color(&color, g_value_get_uint(value));
 		g_value_set_boxed(&val, &color);
 		g_object_set_property(object, "foreground-rgba", &val);
-		g_value_unset(&val);
 		break;
 	default:
 		G_OBJECT_WARN_INVALID_PROPERTY_ID(object, param_id, pspec);

doxygen/Doxyconfig

@@ -1,4 +1,4 @@
-# Doxyfile 1.8.17
+# Doxyfile 1.8.16
 
 # This file describes the settings to be used by the documentation system
 # doxygen (www.doxygen.org) for a project.
@@ -309,7 +309,7 @@ OPTIMIZE_OUTPUT_SLICE  = NO
 # parses. With this tag you can assign which parser to use for a given
 # extension. Doxygen has a built-in mapping, but you can override or extend it
 # using this tag. The format is ext=language, where ext is a file extension, and
-# language is one of the parsers supported by doxygen: IDL, Java, JavaScript,
+# language is one of the parsers supported by doxygen: IDL, Java, Javascript,
 # Csharp (C#), C, C++, D, PHP, md (Markdown), Objective-C, Python, Slice,
 # Fortran (fixed format Fortran: FortranFixed, free formatted Fortran:
 # FortranFree, unknown formatted Fortran: Fortran. In the later case the parser
@@ -535,8 +535,8 @@ HIDE_UNDOC_MEMBERS     = NO
 HIDE_UNDOC_CLASSES     = NO
 
 # If the HIDE_FRIEND_COMPOUNDS tag is set to YES, doxygen will hide all friend
-# declarations. If set to NO, these declarations will be included in the
-# documentation.
+# (class|struct|union) declarations. If set to NO, these declarations will be
+# included in the documentation.
 # The default value is: NO.
 
 HIDE_FRIEND_COMPOUNDS  = NO
@@ -851,10 +851,8 @@ INPUT_ENCODING         = UTF-8
 # If left blank the following patterns are tested:*.c, *.cc, *.cxx, *.cpp,
 # *.c++, *.java, *.ii, *.ixx, *.ipp, *.i++, *.inl, *.idl, *.ddl, *.odl, *.h,
 # *.hh, *.hxx, *.hpp, *.h++, *.cs, *.d, *.php, *.php4, *.php5, *.phtml, *.inc,
-# *.m, *.markdown, *.md, *.mm, *.dox (to be provided as doxygen C comment),
-# *.doc (to be provided as doxygen C comment), *.txt (to be provided as doxygen
-# C comment), *.py, *.pyw, *.f90, *.f95, *.f03, *.f08, *.f, *.for, *.tcl, *.vhd,
-# *.vhdl, *.ucf, *.qsf and *.ice.
+# *.m, *.markdown, *.md, *.mm, *.dox, *.py, *.pyw, *.f90, *.f95, *.f03, *.f08,
+# *.f, *.for, *.tcl, *.vhd, *.vhdl, *.ucf, *.qsf and *.ice.
 
 FILE_PATTERNS          = *.c \
                          *.cc \
@@ -930,7 +928,7 @@ EXCLUDE_SYMLINKS       = NO
 # Note that the wildcards are matched against the file with absolute path, so to
 # exclude all test directories for example use the pattern */test/*
 
-EXCLUDE_PATTERNS       = */test/*
+EXCLUDE_PATTERNS       =
 
 # The EXCLUDE_SYMBOLS tag can be used to specify one or more symbol names
 # (namespaces, classes, functions, etc.) that should be excluded from the
@@ -1267,9 +1265,9 @@ HTML_TIMESTAMP         = NO
 
 # If the HTML_DYNAMIC_MENUS tag is set to YES then the generated HTML
 # documentation will contain a main index with vertical navigation menus that
-# are dynamically created via JavaScript. If disabled, the navigation index will
+# are dynamically created via Javascript. If disabled, the navigation index will
 # consists of multiple levels of tabs that are statically embedded in every HTML
-# page. Disable this option to support browsers that do not have JavaScript,
+# page. Disable this option to support browsers that do not have Javascript,
 # like the Qt help browser.
 # The default value is: YES.
 # This tag requires that the tag GENERATE_HTML is set to YES.
@@ -1557,14 +1555,8 @@ FORMULA_FONTSIZE       = 10
 
 FORMULA_TRANSPARENT    = YES
 
-# The FORMULA_MACROFILE can contain LaTeX \newcommand and \renewcommand commands
-# to create new LaTeX commands to be used in formulas as building blocks. See
-# the section "Including formulas" for details.
-
-FORMULA_MACROFILE      =
-
 # Enable the USE_MATHJAX option to render LaTeX formulas using MathJax (see
-# https://www.mathjax.org) which uses client side JavaScript for the rendering
+# https://www.mathjax.org) which uses client side Javascript for the rendering
 # instead of using pre-rendered bitmaps. Use this if you do not have LaTeX
 # installed or if you want to formulas look prettier in the HTML output. When
 # enabled you may also need to install MathJax separately and configure the path
@@ -1634,7 +1626,7 @@ MATHJAX_CODEFILE       =
 SEARCHENGINE           = YES
 
 # When the SERVER_BASED_SEARCH tag is enabled the search engine will be
-# implemented using a web server instead of a web client using JavaScript. There
+# implemented using a web server instead of a web client using Javascript. There
 # are two flavors of web server based searching depending on the EXTERNAL_SEARCH
 # setting. When disabled, doxygen will generate a PHP script for searching and
 # an index file used by the script. When EXTERNAL_SEARCH is enabled the indexing

doxygen/translation.dox

@@ -1,53 +0,0 @@
-/**
-
-@page translation Translations / Internationalization
-
-@section lang-available Available Languages
-
-The development, documentation and default application language of gds-render is English.
-Translations of GUI and console output exist for the following locales/languages
-
-- de German (Deutsch)
-- ja Japanese (日本語) (Work in progess)
-
-On how to add new languages, see @ref lang-add
-
-@section lang-internals Internals
-This progamm uses gettext for translations.
-
-@subsection lang-coding Make Strings Translatable in Code
-Only strings, that are marked as translatable are actually available for translation and shown in the user's language.
-All other text is displayed as written in the source files.
-
-@subsubsection lang-add-glade Make GUI Elements in Glade Files Translatable
-Text inside Glade's GUI files can be made translatable by checking the *translatable* option for the specific text property.
-The following image shows the label property of a button. Notice the checked Translatable option.
-@image html translate-glade.png
-@image latex translate-glade.png
-
-@subsubsection lang-add-c Make C Strings Translatable
-In order to make strings in C code translatable, the following header has to be included:
-@code
-#inlcude <glib/gi18n.h>
-@endcode
-This header provides a few macros useful for translation. The most important one is the function-like macro
-@code
-_("Text here")
-@endcode
-which is just a underline character.
-You can use this marco to encapsulate all static strings that shall be translated. The whole expression is still usable like any other string (except for compile time concatenation, like: "This is foo" " and this is bar")
-
-Example:
-@code
-printf(_("The value of foo is %d\n"), foo);
-@endcode
-
-Text that must not be translated can be encapsulated inside a N_ macro like
-@code
-N_("This has to stay in English")
-@endcode
-The N_ macro is basically an empty macro and just for visualization.
-
-@subsection lang-add Add a New Language
-
-*/

doxygen/versioning.dox

@@ -2,10 +2,10 @@
 @page versioning Version Number
 
 @section main-version Main Versioning Scheme
-The version number of this application consists of a given version in the format of 'v1.0.0' (formely only 2 digits).
-Where the first number indicates a major release and the second and third numbers indicate minor changes.
+The version number of this application consists of a given version in the format of 'v1.0'.
+Where the first number indicates a major release and the second number indicates minor changes.
 
-Versions, including release candidates and patch-levels, are tagged in git.
+Versions, including release candidates and path-levels, are tagged in git.
 
 @subsection rc Release Candidates
 Release candidates are software versions that seem stable and functional to become a new version but testing is not fully finished. These versions are marked with an '-rcX', where X is the number of the release candidate.
@@ -13,7 +13,7 @@ The 3rd release candidate of version 4.2 would be '*v4.2-rc3*'.
 Release candidates are in a frozen state. Only bugfixes that are necessary for functionality are applied to these versions before releasing the final version.
 
 @subsection patch-level Patch Levels
-If an already released version contains bugs that need to be fixed, the version number is not incremented. Insted a new version number with a patch-level is created. The patch-level is appended with a dash directly after the version number. The fist patch-level of version 3.5.2 would be: 'v3.5.2-1'.
+If an already released version contains bugs that need to be fixed, the version number is not incremented. Insted a new version number with a patch-level is created. The patch-level is appended with a dash directly after the version number. The fist patch-level of version 3.5 would be: 'v3.5-1'.
 
 
 @section git-version-num Git Based Version Number

gds-render-gui.c

@@ -187,7 +187,8 @@ static gboolean cell_store_filter_visible_func(GtkTreeModel *model, GtkTreeIter
 
 	gtk_tree_model_get(model, iter, CELL_SEL_CELL, &cell, CELL_SEL_LIBRARY, &lib, -1);
 
-	if (lib) {
+    /* Show always, if this is a pure lib entry */
+    if (lib && !cell) {
 		result = TRUE;
 		goto exit_filter;
 	}

gds-utils/gds-parser.c

@@ -80,7 +80,6 @@ enum gds_record {
 	STRANS = 0x1A01,
 	BOX = 0x2D00,
 	LAYER = 0x0D02,
-	DATATYPE = 0x0E02,
 	WIDTH = 0x0F03,
 	PATHTYPE = 0x2102,
 	COLROW = 0x1302,
@@ -868,7 +867,6 @@ int parse_gds_from_file(const char *filename, GList **library_list)
 		case LIBNAME:
 		case SNAME:
 		case LAYER:
-		case DATATYPE:
 		case STRNAME:
 			break;
 		default:
@@ -1005,16 +1003,6 @@ int parse_gds_from_file(const char *filename, GList **library_list)
 			}
 			GDS_INF("\t\tAdded layer %d\n", (int)current_graphics->layer);
 			break;
-		case DATATYPE:
-			if (!current_graphics) {
-				GDS_WARN("Datatype has to be defined inside graphics object. Probably unknown object. Implement it yourself!");
-				break;
-			}
-			current_graphics->datatype = gds_convert_signed_int16(workbuff);
-			if (current_graphics->datatype < 0)
-				GDS_WARN("Datatype negative!");
-			GDS_INF("\t\tAdded datatype %d\n", (int)current_graphics->datatype);
-			break;
 		case MAG:
 			if (rec_data_length != 8) {
 				GDS_WARN("Magnification is not an 8 byte real. Results may be wrong");

geometric/bounding-box.c

@@ -37,7 +37,7 @@
 #define MAX(a, b) (((a) > (b)) ? (a) : (b)) /**< @brief Return bigger number */
 #define ABS_DBL(a) ((a) < 0 ? -(a) : (a))
 
-void bounding_box_calculate_from_polygon(GList *vertices, conv_generic_to_vector_2d_t conv_func, union bounding_box *box)
+void bounding_box_calculate_polygon(GList *vertices, conv_generic_to_vector_2d_t conv_func, union bounding_box *box)
 {
 	double xmin = DBL_MAX, xmax = -DBL_MAX, ymin = DBL_MAX, ymax = -DBL_MAX;
 	struct vector_2d temp_vec;
@@ -68,7 +68,7 @@ void bounding_box_calculate_from_polygon(GList *vertices, conv_generic_to_vector
 	box->vectors.upper_right.y = ymax;
 }
 
-void bounding_box_update_with_box(union bounding_box *destination, union bounding_box *update)
+void bounding_box_update_box(union bounding_box *destination, union bounding_box *update)
 {
 	if (!destination || !update)
 		return;
@@ -151,6 +151,8 @@ void bounding_box_update_with_path(GList *vertices, double thickness,
 	GList *vertex_iterator;
 	struct vector_2d pt;
 
+	/* printf("Warning! Function %s not yet implemented correctly!\n", __func__); */
+
 	if (!vertices || !box)
 		return;
 
@@ -171,7 +173,7 @@ void bounding_box_update_with_path(GList *vertices, double thickness,
 	}
 }
 
-void bounding_box_update_with_point(union bounding_box *destination, conv_generic_to_vector_2d_t conv_func, void *pt)
+void bounding_box_update_point(union bounding_box *destination, conv_generic_to_vector_2d_t conv_func, void *pt)
 {
 	struct vector_2d point;
 
@@ -222,7 +224,7 @@ void bounding_box_apply_transform(double scale, double rotation_deg, bool flip_a
 		vector_2d_rotate(&input_points[i], rotation_deg * M_PI / 180.0);
 		vector_2d_scale(&input_points[i], scale);
 
-		bounding_box_update_with_point(box, NULL, &input_points[i]);
+		bounding_box_update_point(box, NULL, &input_points[i]);
 	}
 }
 

geometric/cell-geometrics.c

@@ -53,7 +53,7 @@ static void update_box_with_gfx(union bounding_box *box, struct gds_graphics *gf
 	case GRAPHIC_BOX:
 		/* Expected fallthrough */
 	case GRAPHIC_POLYGON:
-		bounding_box_calculate_from_polygon(gfx->vertices,
+		bounding_box_calculate_polygon(gfx->vertices,
 							(conv_generic_to_vector_2d_t)&convert_gds_point_to_2d_vector,
 							&current_box);
 		break;
@@ -74,7 +74,7 @@ static void update_box_with_gfx(union bounding_box *box, struct gds_graphics *gf
 	}
 
 	/* Update box with results */
-	bounding_box_update_with_box(box, &current_box);
+	bounding_box_update_box(box, &current_box);
 }
 
 void calculate_cell_bounding_box(union bounding_box *box, struct gds_cell *cell)
@@ -113,7 +113,7 @@ void calculate_cell_bounding_box(union bounding_box *box, struct gds_cell *cell)
 		temp_box.vectors.upper_right.y += sub_cell->origin.y;
 
 		/* update the parent's box */
-		bounding_box_update_with_box(box, &temp_box);
+		bounding_box_update_box(box, &temp_box);
 	}
 }
 

include/gds-render/gds-utils/gds-types.h

@@ -101,7 +101,7 @@ struct gds_graphics {
 	enum path_type path_render_type; /**< @brief Line cap */
 	int width_absolute; /**< @brief Width. Not used for objects other than paths */
 	int16_t layer; /**< @brief Layer the graphic object is on */
-	int16_t datatype; /**< @brief Data type of graphic object */
+	uint16_t datatype;
 };
 
 /**

include/gds-render/geometric/bounding-box.h

@@ -35,44 +35,15 @@
 #include <gds-render/geometric/vector-operations.h>
 #include <stdbool.h>
 
-/**
- * @brief Union describing a bounding box
- *
- * Two ways of accessing a bounding box are possible.
- *
- * Either, use the "named" vectors struct to specifically access the points
- * @code
- * 	lower_left = box.vectors.lower_left;
- * 	upper right = box.vectors.upper_right;
- * @endcode
- *
- * or use the iterable vector array:
- * @code
- * 	for (i = 0; i < 2; i++)
- * 		box.vector_array[i] = points[i];
- * @endcode
- */
 union bounding_box {
-	/**
-	 * @brief Location vectors of upper right and lower left bounding box points
-	 * @note Coordinate System is (y up | x right)
-	 */
+	/** Coordinate System is (y up | x right) */
 	struct _vectors {
-		/** @brief Lower left point of the bounding box */
 		struct vector_2d lower_left;
-		/** @brief Upper right point of the bounding box */
 		struct vector_2d upper_right;
 	} vectors;
-	/**
-	 * @brief Array of vectors representing a bounding box
-	 * @note This is more convenient for iterating
-	 */
 	struct vector_2d vector_array[2];
 };
 
-/*
- * @brief Pointer to a function that takes any pointer and converts this object to a vector_2d struct
- */
 typedef void (*conv_generic_to_vector_2d_t)(void *, struct vector_2d *);
 
 /**
@@ -81,14 +52,14 @@ typedef void (*conv_generic_to_vector_2d_t)(void *, struct vector_2d *);
  * @param conv_func Conversion function to convert vertices to vector_2d structs.
  * @param box Box to write to. This box is not updated! All previous data is discarded
  */
-void bounding_box_calculate_from_polygon(GList *vertices, conv_generic_to_vector_2d_t conv_func, union bounding_box *box);
+void bounding_box_calculate_polygon(GList *vertices, conv_generic_to_vector_2d_t conv_func, union bounding_box *box);
 
 /**
  * @brief Update an exisitng bounding box with another one.
  * @param destination Target box to update
  * @param update Box to update the target with
  */
-void bounding_box_update_with_box(union bounding_box *destination, union bounding_box *update);
+void bounding_box_update_box(union bounding_box *destination, union bounding_box *update);
 
 /**
  * @brief Prepare an empty bounding box.
@@ -105,7 +76,7 @@ void bounding_box_prepare_empty(union bounding_box *box);
  * @param conv_func Conversion function to convert \p pt to a vector_2d. May be NULL
  * @param pt Point to update bounding box with
  */
-void bounding_box_update_with_point(union bounding_box *destination, conv_generic_to_vector_2d_t conv_func, void *pt);
+void bounding_box_update_point(union bounding_box *destination, conv_generic_to_vector_2d_t conv_func, void *pt);
 
 /**
  * @brief Return all four corner points of a bounding box
@@ -131,7 +102,7 @@ void bounding_box_get_all_points(struct vector_2d *points, union bounding_box *b
  * @param rotation_deg Rotation of bounding box around the origin in degrees (counterclockwise)
  * @param flip_at_x Flip the boundig box on the x axis before rotating.
  * @param box Bounding box the operations should be applied to.
- * @note Keep in mind, that this bounding box is actually the bounding box of the rotated boundig box and not the object itself.
+ * @note Keep in mind, that this bounding boxy is actually the bounding box of the rotated boundig box and not the object itself.
  *       It might be too big.
  */
 void bounding_box_apply_transform(double scale, double rotation_deg, bool flip_at_x, union bounding_box *box);
@@ -142,9 +113,6 @@ void bounding_box_apply_transform(double scale, double rotation_deg, bool flip_a
  * @param thickness Thisckness of the path
  * @param conv_func Conversion function for vertices to vector_2d structs
  * @param box Bounding box to write results in.
- * @warning This function is not yet implemented correctly. Miter points of paths are not taken into account.
- *		If a path is the outmost object of your cell _and_ it is not parallel to one of the coordinate axes,
- *		the calculated bounding box size might be off. In other cases it should be reasonable close to the real bounding box.
  */
 void bounding_box_update_with_path(GList *vertices, double thickness, conv_generic_to_vector_2d_t conv_func, union bounding_box *box);
 

include/gds-render/layer/layer-settings.h

@@ -41,7 +41,7 @@ struct layer_info
 {
 	int layer; /**< @brief Layer number */
 	char *name; /**< @brief Layer name. */
-	int stacked_position; /**< @brief Position of layer in output @warning This parameter is not used by any renderer so far @note Lower is bottom, higher is top */
+	int stacked_position; ///< @brief Position of layer in output @warning This parameter is not used by any renderer so far @note Lower is bottom, higher is top
 	GdkRGBA color; /**< @brief RGBA color used to render this layer */
 	int render; /**< @brief true: Render to output */
 };

include/gds-render/output-renderers/external-renderer-interfaces.h

@@ -1,22 +1,3 @@
-/*
- * GDSII-Converter
- * Copyright (C) 2018  Mario Hüttel <mario.huettel@gmx.net>
- *
- * This file is part of GDSII-Converter.
- *
- * GDSII-Converter is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- * GDSII-Converter is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with GDSII-Converter.  If not, see <http://www.gnu.org/licenses/>.
- */
-
 #ifndef __EXTERNAL_RENDERER_INTERFACES_H__
 #define __EXTERNAL_RENDERER_INTERFACES_H__
 

output-renderers/cairo-renderer.c

@@ -275,20 +275,12 @@ static int cairo_renderer_render_cell_to_vector_file(GdsOutputRenderer *renderer
 		goto ret_parent;
 	}
 
-	/* We are now in a separate process just for rendering the output image.
-     * You may print a log message to the activity bar of the gui by writing a line
-     * teminated with '\n' to comm_pipe[1]. This will be handled by the parent process.
-     * Directly calling the update function 
-     *     gds_output_renderer_update_async_progress()
-     * does not have any effect because this is a separate process.
-     */
-  
-	/*
-     * Close stdin and (stdout and stderr may live on)
-     */
+	/* Close stdin and (stdout and stderr may live on) */
 	close(0);
+	//close(1);
 	close(comm_pipe[0]);
 
+
 	layers = (struct cairo_layer *)calloc(MAX_LAYERS, sizeof(struct cairo_layer));
 
 	/* Clear layers */

output-renderers/gds-output-renderer.c

@@ -100,11 +100,6 @@ static void gds_output_renderer_dispose(GObject *self_obj)
 	if (priv->output_file)
 		g_free(priv->output_file);
 
-	if (priv->idle_function_parameters.status_message) {
-		g_free(priv->idle_function_parameters.status_message);
-		priv->idle_function_parameters.status_message = NULL;
-	}
-
 	g_clear_object(&priv->layer_settings);
 
 	/* Chain up to parent class */

output-renderers/latex-renderer.c

@@ -370,7 +370,7 @@ static int latex_renderer_render_output(GdsOutputRenderer *renderer,
 						l_renderer->pdf_layers, l_renderer->tex_standalone, renderer);
 		fclose(tex_file);
 	} else {
-		g_error(_("Could not open LaTeX output file"));
+        g_warning(_("Could not open LaTeX output file"));
 	}
 
 	if (settings)

test/CMakeLists.txt

@@ -1,41 +0,0 @@
-project(gds-render-test)
-
-add_custom_target(test "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}" "-r compact" "-s" DEPENDS ${PROJECT_NAME})
-
-if(NOT WIN32)
-  string(ASCII 27 Esc)
-  set(ColorReset "${Esc}[m")
-  set(ColorBold  "${Esc}[1m")
-  set(Red         "${Esc}[31m")
-  set(Green       "${Esc}[32m")
-  set(Yellow      "${Esc}[33m")
-  set(Blue        "${Esc}[34m")
-  set(Magenta     "${Esc}[35m")
-  set(Cyan        "${Esc}[36m")
-  set(White       "${Esc}[37m")
-  set(BoldRed     "${Esc}[1;31m")
-  set(BoldGreen   "${Esc}[1;32m")
-  set(BoldYellow  "${Esc}[1;33m")
-  set(BoldBlue    "${Esc}[1;34m")
-  set(BoldMagenta "${Esc}[1;35m")
-  set(BoldCyan    "${Esc}[1;36m")
-  set(BoldWhite   "${Esc}[1;37m")
-endif()
-
-cmake_minimum_required(VERSION 2.8)
-find_package(PkgConfig REQUIRED)
-
-include_directories("${CMAKE_CURRENT_SOURCE_DIR}/catch-framework")
-
-aux_source_directory("geometric" GEOMETRIC_TEST_SOURCES)
-set(TEST_SOURCES
-	${GEOMETRIC_TEST_SOURCES}
-)
-
-set(DUT_SOURCES
-	"../geometric/vector-operations.c"
-)
-
-add_executable(${PROJECT_NAME} EXCLUDE_FROM_ALL "test-main.cpp" ${TEST_SOURCES} ${DUT_SOURCES})
-target_link_libraries(${PROJECT_NAME} ${GLIB_LDFLAGS} ${GTK3_LDFLAGS} ${CAIRO_LDFLAGS} m version ${CMAKE_DL_LIBS})
-

test/geometric/vector-operations.cpp

@@ -1,160 +0,0 @@
-#include <catch.hpp>
-#include <limits>
-
-extern "C" {
-#include <gds-render/geometric/vector-operations.h>
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_add", "[GEOMETRIC]")
-{
-	struct vector_2d res;
-	struct vector_2d a;
-	struct vector_2d b;
-
-	a.x = 1;
-	a.y = 2;
-
-	b.x = 2;
-	b.y = 6;
-
-	vector_2d_add(&res, &a, &b);
-
-	REQUIRE(res.x == Approx(a.x + b.x));
-	REQUIRE(res.y == Approx(a.y + b.y));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_calculate_angle_between", "[GEOMETRIC]")
-{
-	double angle;
-	struct vector_2d a;
-	struct vector_2d b;
-
-	a.x = 1;
-	a.y = 0;
-
-	b.x = 0;
-	b.y = 1;
-
-	angle = vector_2d_calculate_angle_between(&a, &a);
-	REQUIRE(angle == Approx(0.0));
-
-	angle = vector_2d_calculate_angle_between(&a, &b);
-	REQUIRE(angle == Approx(90.0 / 180.0 * M_PI));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_subtract", "[GEOMETRIC]")
-{
-	struct vector_2d res;
-	struct vector_2d a;
-	struct vector_2d b;
-
-	a.x = 1;
-	a.y = 2;
-
-	b.x = 2;
-	b.y = 6;
-
-	vector_2d_subtract(&res, &a, &b);
-
-	REQUIRE(res.x == Approx(a.x - b.x));
-	REQUIRE(res.y == Approx(a.y - b.y));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_abs", "[GEOMETRIC]")
-{
-	struct vector_2d c;
-	struct vector_2d a;
-	struct vector_2d b;
-	double a_len, b_len, c_len;
-
-	a.x = 1;
-	a.y = 0;
-
-	b.x = 0;
-	b.y = 2;
-
-	c.x = 3;
-	c.y = 4;
-
-	a_len = vector_2d_abs(&a);
-	b_len = vector_2d_abs(&b);
-	c_len = vector_2d_abs(&c);
-
-	REQUIRE(a_len == Approx(1.0));
-	REQUIRE(b_len == Approx(2.0));
-	REQUIRE(c_len == Approx(5.0));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_scalar_multipy", "[GEOMETRIC]")
-{
-	struct vector_2d c;
-	struct vector_2d a;
-	struct vector_2d b;
-	double mult;
-
-	a.x = 1;
-	a.y = 0;
-	b.x = 0;
-	b.y = 2;
-	mult = vector_2d_scalar_multipy(&a, &b);
-	REQUIRE(mult == Approx(0.0));
-	
-	a.x = 1;
-	a.y = 1;
-	b.x = 1;
-	b.y = 1;
-	mult = vector_2d_scalar_multipy(&a, &b);
-	REQUIRE(mult == Approx(2.0));	
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_normalize", "[GEOMETRIC]")
-{
-	struct vector_2d a;
-
-	a.x = 1;
-	a.y = 0;
-	vector_2d_normalize(&a);
-	REQUIRE(a.x == Approx(1.0));
-	REQUIRE(a.y == Approx(0.0));
-
-	a.x = 1;
-	a.y = -1;
-	vector_2d_normalize(&a);
-	REQUIRE(a.x == Approx(1.0/sqrt(2)));
-	REQUIRE(a.y == Approx(-1.0/sqrt(2)));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_rotate", "[GEOMETRIC]")
-{
-	struct vector_2d a;
-
-	a.x = 1;
-	a.y = 0;
-	vector_2d_rotate(&a, M_PI/2);
-	REQUIRE(a.x == Approx(0.0).scale(0.001));
-	REQUIRE(a.y == Approx(1.0));
-
-	a.x = 0;
-	a.y = 1;
-	vector_2d_rotate(&a, -M_PI/2);
-	vector_2d_rotate(&a, M_PI);
-	REQUIRE(a.x == Approx(-1.0));
-	REQUIRE(a.y == Approx(0.0).scale(0.001));
-}
-
-TEST_CASE("geometric/vector-operations/vector_2d_scale", "[GEOMETRIC]")
-{
-	struct vector_2d a;
-
-	a.x = 1;
-	a.y = 0;
-	vector_2d_scale(&a, 2.0);
-	REQUIRE(a.x == Approx(2.0));
-	REQUIRE(a.y == Approx(0.0));
-
-	a.x = 1;
-	a.y = -3;
-	vector_2d_scale(&a, 0.5);
-	REQUIRE(a.x == Approx(0.5));
-	REQUIRE(a.y == Approx(-1.5));
-}

test/test-main.cpp

@@ -1,2 +0,0 @@
-#define CATCH_CONFIG_MAIN
-#include "catch-framework/catch.hpp"

translations/.gitignore

@@ -1 +0,0 @@
-*.mo

translations/pot/po/de/gds-render.po

@@ -1,7 +1,7 @@
 # German translations for gds-render package.
 # Copyright (C) 2019 THE gds-render'S COPYRIGHT HOLDER
 # This file is distributed under the same license as the gds-render package.
-# Mario Hüttel <mario.huettel@gmx.net>, 2019.
+# マリオ <mario.huettel@gmx.net>, 2019.
 #
 msgid ""
 msgstr ""

translations/pot/po/ja/gds-render.po

@@ -1,460 +0,0 @@
-# Japanese translations for gds-render package.
-# Copyright (C) 2019 THE gds-render'S COPYRIGHT HOLDER
-# This file is distributed under the same license as the gds-render package.
-# Mario Hüttel <mario.huettel@gmx.net>, 2019.
-#
-msgid ""
-msgstr ""
-"Project-Id-Version: gds-render\n"
-"Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2019-12-16 18:27+0100\n"
-"PO-Revision-Date: 2020-01-16 22:54+0100\n"
-"Last-Translator: Mario Hüttel <mario.huettel@gmx.net>\n"
-"Language-Team: Japanese <translation-team-ja@lists.sourceforge.net>\n"
-"Language: ja\n"
-"MIME-Version: 1.0\n"
-"Content-Type: text/plain; charset=UTF-8\n"
-"Content-Transfer-Encoding: 8bit\n"
-"Plural-Forms: nplurals=1; plural=0;\n"
-"X-Generator: Poedit 2.2.4\n"
-
-#: ../main.c:97
-#, c-format
-msgid ""
-"\n"
-"\n"
-"Full git commit: %s"
-msgstr ""
-"\n"
-"\n"
-"git コミット: %s"
-
-#: ../main.c:294
-msgid " FILE - Convert GDS file <FILE> to graphic"
-msgstr " ファイル - ファイルを変換する"
-
-#: ../resources/about.glade:10
-msgid "!! Replaced during runtime !!"
-msgstr "!! ランタイムで代わる !!"
-
-#: ../command-line.c:205
-#, c-format
-msgid "%d reference loops found.\n"
-msgstr "%d 参照ループが見つかりました.\n"
-
-#: ../main.c:220
-msgid "About"
-msgstr "情報"
-
-#: ../main.c:290
-msgid "Argument string passed to render lib"
-msgstr "レンダリングに渡されるデタ"
-
-#: ../resources/main.glade:225
-msgid "Automatically color layers"
-msgstr "レイヤーを自動的に色付けする"
-
-#: ../resources/main.glade:255
-msgid "Automatically name layers"
-msgstr "レイヤーに自動的に名前を付ける"
-
-#: ../gds-utils/gds-tree-checker.c:71
-#, c-format
-msgid "Broken cell list item found in cell %s. Will continue.\n"
-msgstr "%sのセルで壊れたセルリストアイテムが見つかりました。続けます。\n"
-
-#: ../gds-utils/gds-tree-checker.c:57
-#, c-format
-msgid "Broken cell list item found. Will continue.\n"
-msgstr "壊れたセルリストアイテムが見つかりました。続けます。\n"
-
-#: ../output-renderers/cairo-renderer.c:409
-#, c-format
-msgid "Cairo export finished. It might still be buggy!\n"
-msgstr "Cairoの変換が終了しました.\n"
-
-#. Execute
-#: ../output-renderers/external-renderer.c:120
-msgid "Calling external renderer."
-msgstr "外部レンダラーの実行。"
-
-#: ../gds-render-gui.c:286 ../widgets/conv-settings-dialog.c:291
-msgid "Cancel"
-msgstr "キャンセル"
-
-#: ../gds-render-gui.c:244
-msgid "Cell"
-msgstr "セル"
-
-#: ../command-line.c:209
-#, c-format
-msgid "Cell is affected by reference loop. Abort!\n"
-msgstr "このセルに参照ループがあります。アボート！\n"
-
-#: ../main.c:284
-msgid "Cell to render"
-msgstr "レンダリングされるセル"
-
-#: ../command-line.c:215
-#, c-format
-msgid ""
-"Cell was not checked. This should not happen. Please report this issue. Will "
-"continue either way.\n"
-msgstr "セルのチェックはしませんでした。これは問題です。\n"
-
-#: ../command-line.c:202
-#, c-format
-msgid "Checking library %s failed.\n"
-msgstr "ライブラリ %s のチェックは失敗しました。\n"
-
-#: ../resources/dialog.glade:79
-msgid "Configure LaTeX as standalone document"
-msgstr ""
-
-#: ../resources/main.glade:109
-msgid "Convert selected cell"
-msgstr ""
-
-#: ../output-renderers/external-renderer.c:89
-#, c-format
-msgid ""
-"Could not load external library '%s'\n"
-"Detailed error is:\n"
-"%s\n"
-msgstr ""
-
-#: ../output-renderers/latex-renderer.c:373
-msgid "Could not open LaTeX output file"
-msgstr ""
-
-#: ../command-line.c:195
-#, c-format
-msgid "Couldn't find cell in first library!\n"
-msgstr ""
-
-#: ../command-line.c:81
-#, c-format
-msgid "Count of renderers %d does not match count of output file names %d\n"
-msgstr ""
-
-#: ../main.c:286
-msgid "Create PDF Layers (OCG)"
-msgstr ""
-
-#: ../main.c:285
-msgid "Create standalone TeX"
-msgstr ""
-
-#: ../main.c:280
-msgid "Divide output coordinates by <SCALE>"
-msgstr ""
-
-#: ../resources/layer-widget.glade:62
-msgid "Export Layer"
-msgstr ""
-
-#: ../output-renderers/cairo-renderer.c:366
-#, c-format
-msgid "Exporting layer %d to file\n"
-msgstr ""
-
-#: ../output-renderers/external-renderer.c:142
-msgid "External renderer finished."
-msgstr ""
-
-#. This should not happen
-#: ../output-renderers/cairo-renderer.c:271
-#, c-format
-msgid "Fatal error: Cairo Renderer: Could not spawn child process!"
-msgstr ""
-
-#: ../resources/main.glade:50
-msgid "GDS-Render"
-msgstr ""
-
-#: ../gds-render-gui.c:294
-msgid "GDSII-Files"
-msgstr ""
-
-#: ../resources/dialog.glade:18
-msgid "Generate LaTeX/TikZ output"
-msgstr ""
-
-#: ../resources/dialog.glade:93
-msgid "Generate PDF Layers"
-msgstr ""
-
-#: ../output-renderers/latex-renderer.c:253
-#, c-format
-msgid "Generating cell %s"
-msgstr ""
-
-#: ../resources/about.glade:12
-msgid "Git Repository"
-msgstr ""
-
-#: ../widgets/conv-settings-dialog.c:246
-#, c-format
-msgid "Height: %.3lf %sm"
-msgstr ""
-
-#: ../main.c:321
-#, c-format
-msgid "Ignored argument: %s"
-msgstr ""
-
-#: ../output-renderers/external-renderer.c:106
-#, c-format
-msgid ""
-"Init function not found in library:\n"
-"%s\n"
-msgstr ""
-
-#: ../output-renderers/cairo-renderer.c:321
-#, c-format
-msgid "Layer number too high / outside of spec.\n"
-msgstr ""
-
-#: ../output-renderers/cairo-renderer.c:369
-#, c-format
-msgid "Layer outside of spec.\n"
-msgstr ""
-
-#: ../widgets/layer-element.c:102
-#, c-format
-msgid "Layer: %d"
-msgstr ""
-
-#: ../gds-render-gui.c:241
-msgid "Library"
-msgstr ""
-
-#: ../resources/main.glade:68
-msgid "Load Mapping"
-msgstr ""
-
-#: ../resources/main.glade:73
-msgid "Load the current layer configuration from CSV"
-msgstr ""
-
-#: ../main.c:116
-#, c-format
-msgid "Logo could not be displayed: %s\n"
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:293
-msgid "No layer specification supplied."
-msgstr ""
-
-#: ../command-line.c:186
-#, c-format
-msgid "No library in library list. This should not happen.\n"
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:288
-msgid "No/invalid output file set."
-msgstr ""
-
-#: ../widgets/conv-settings-dialog.c:291
-msgid "OK"
-msgstr ""
-
-#: ../resources/main.glade:58
-msgid "Open GDS2 Database"
-msgstr ""
-
-#: ../gds-render-gui.c:287
-msgid "Open GDSII"
-msgstr ""
-
-#: ../gds-render-gui.c:284
-msgid "Open GDSII File"
-msgstr ""
-
-#: ../main.c:299
-#, c-format
-msgid "Option parsing failed: %s\n"
-msgstr ""
-
-#: ../widgets/conv-settings-dialog.c:255
-#, c-format
-msgid "Output Height: %u px"
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:283
-msgid "Output Renderer not valid."
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:304
-msgid "Output Renderer: Rendering function broken. This is a bug."
-msgstr ""
-
-#. Set the pixel sizes
-#: ../widgets/conv-settings-dialog.c:252
-#, c-format
-msgid "Output Width: %u px"
-msgstr ""
-
-#: ../main.c:282
-msgid "Output file path. Can be used multiple times."
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:298
-msgid "Output renderer called without cell to render."
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:74
-msgid "Output renderer does not define a render_output function!"
-msgstr ""
-
-#: ../main.c:283
-msgid "Path for Layer Mapping File"
-msgstr ""
-
-#: ../main.c:288
-msgid ""
-"Path to a custom shared object, that implements the necessary rendering "
-"functions"
-msgstr ""
-
-#: ../output-renderers/external-renderer.c:78
-#, c-format
-msgid "Path to shared object not set!\n"
-msgstr ""
-
-#: ../command-line.c:74
-#, c-format
-msgid "Please specify renderers and file names\n"
-msgstr ""
-
-#: ../command-line.c:104
-#, c-format
-msgid ""
-"Please specify shared object for external renderer. Will ignore this "
-"renderer.\n"
-msgstr ""
-
-#: ../main.c:277
-msgid "Print version"
-msgstr ""
-
-#: ../command-line.c:160
-#, c-format
-msgid "Probably missing argument. Check --help option\n"
-msgstr ""
-
-#: ../main.c:219
-msgid "Quit"
-msgstr ""
-
-#: ../widgets/activity-bar.c:104
-msgid "Ready"
-msgstr ""
-
-#: ../resources/dialog.glade:33
-msgid "Render PDF using Cairographics"
-msgstr ""
-
-#: ../resources/dialog.glade:48
-msgid "Render SVG using Cairographics (too buggy at the moment)"
-msgstr ""
-
-#: ../widgets/conv-settings-dialog.c:293
-msgid "Renderer Settings"
-msgstr ""
-
-#: ../output-renderers/gds-output-renderer.c:365
-msgid "Renderer already started asynchronously"
-msgstr ""
-
-#: ../main.c:279
-msgid "Renderer to use. Can be used multiple times."
-msgstr ""
-
-#: ../output-renderers/cairo-renderer.c:467
-msgid "Rendering Cairo Output..."
-msgstr ""
-
-#: ../gds-render-gui.c:554
-msgid "Rendering cell..."
-msgstr ""
-
-#: ../output-renderers/external-renderer.c:98
-#, c-format
-msgid ""
-"Rendering function not found in library:\n"
-"%s\n"
-msgstr ""
-
-#: ../resources/main.glade:86
-msgid "Save Mapping"
-msgstr ""
-
-#: ../resources/main.glade:23
-msgid "Save the current layer configuration to CSV"
-msgstr ""
-
-#: ../main.c:312
-#, c-format
-msgid "Scale < 1 not allowed. Setting to 1\n"
-msgstr ""
-
-#: ../resources/layer-widget.glade:51
-msgid "Select Layer Color and Opacity"
-msgstr ""
-
-#: ../resources/main.glade:240
-msgid "Select all layers for export"
-msgstr ""
-
-#: ../output-renderers/cairo-renderer.c:331
-#, c-format
-msgid "Size of layer %d%s%s%s: <%lf x %lf> @ (%lf | %lf)\n"
-msgstr ""
-
-#: ../resources/main.glade:195
-msgid "Sort layers ascending"
-msgstr ""
-
-#: ../resources/main.glade:210
-msgid "Sort layers descending"
-msgstr ""
-
-#: ../main.c:212
-#, c-format
-msgid ""
-"There is already an open instance. Will open second window in that "
-"instance.\n"
-msgstr ""
-
-#: ../main.c:244
-#, c-format
-msgid ""
-"This is gds-render, version: %s\n"
-"\n"
-"For a list of supported commands execute with --help option.\n"
-msgstr ""
-
-#: ../gds-utils/gds-tree-checker.c:188
-#, c-format
-msgid "Visited cell list should be empty. This is a bug. Please report this.\n"
-msgstr ""
-
-#: ../widgets/conv-settings-dialog.c:244
-#, c-format
-msgid "Width: %.3lf %sm"
-msgstr ""
-
-#: ../widgets/activity-bar.c:110
-msgid "Working..."
-msgstr ""
-
-#: ../main.c:96
-msgid ""
-"gds-render is a free tool for rendering GDS2 layout files into vector "
-"graphics."
-msgstr ""
-
-#: ../resources/layer-widget.glade:37
-msgid "label"
-msgstr ""