diff --git a/src/helper-fns.h b/src/helper-fns.h
index b4976bad44f044cca86ab35561f5a95f43e3ff80..43c90063b28252cb2a6ca1eeca2c47e2a640fcc9 100644 (file)
--- a/src/helper-fns.h
+++ b/src/helper-fns.h
* Released under GNU GPL, read the file 'COPYING' for more information
*/
+#include <string.h>
+#include <vector>
#include <sstream>
-inline double helperfns_read_number(gchar const *value) {
+// calling helperfns_read_number(string, false), it's not obvious, what
+// that false stands for. helperfns_read_number(string, HELPERFNS_NO_WARNING)
+// can be more clear.
+#define HELPERFNS_NO_WARNING false
+
+/* Setting warning to false disables conversion error warnings from
+ * this function. This can be useful in places, where the input type
+ * is not known beforehand. For example, see sp_feColorMatrix_set in
+ * sp-fecolormatrix.cpp */
+inline double helperfns_read_number(gchar const *value, bool warning = true) {
if (!value) return 0;
char *end;
double ret = g_ascii_strtod(value, &end);
if (*end) {
- g_warning("Unable to convert \"%s\" to number", value);
+ if (warning) {
+ g_warning("Unable to convert \"%s\" to number", value);
+ }
// We could leave this out, too. If strtod can't convert
// anything, it will return zero.
ret = 0;
inline std::vector<gdouble> helperfns_read_vector(const gchar* value, int size){
std::vector<gdouble> v(size, (gdouble) 0);
std::istringstream is(value);
- for(int i = 0; i < size && (is >> v[i]); i++);
+ for(int i = 0; i < size && (is >> v[i]); i++){};
+ return v;
+}
+
+inline std::vector<gdouble> helperfns_read_vector(const gchar* value){
+ std::vector<gdouble> v;
+ std::istringstream is(value);
+ gdouble d;
+ while (is >> d){
+ v.push_back(d);
+ }
return v;
}