nviz2/wxGUI: nviz module log improved

git-svn-id: https://svn.osgeo.org/grass/grass/trunk@32237 15284696-431f-4ddb-bdfa-cd5b030d7da7

gui/wxpython/gui_modules/goutput.py

@@ -372,7 +372,7 @@ class GMStdout:
             self.gmstc.StartStyling(p1, 0xff)
             self.gmstc.SetStyling(p2 - p1 + 1, self.gmstc.StyleOutput)
 
-class GMStderr:
+class GMStderr(object):
     """GMConsole standard error output
 
     Based on FrameOutErr.py

gui/wxpython/gui_modules/nviz.py

@@ -94,7 +94,7 @@ class GLWindow(MapWindow, glcanvas.GLCanvas):
         #
         # create nviz instance
         #
-        self.nvizClass = wxnviz.Nviz()
+        self.nvizClass = wxnviz.Nviz(sys.stderr)
 
         #
         # set current display
@@ -349,14 +349,12 @@ class GLWindow(MapWindow, glcanvas.GLCanvas):
                 if type == 'raster':
                     self.LoadRaster(item)
             except gcmd.NvizError, e:
-                print >> sys.stderr, "Nviz: %s" % (self.nvizClass.GetErrorMsg())
                 print >> sys.stderr, "Nviz: " + e.message
 
             try:
                 if type == 'vector':
                     self.LoadVector(item)
             except gcmd.NvizError, e:
-                print >> sys.stderr, "Nviz: %s" % (self.nvizClass.GetErrorMsg())
                 print >> sys.stderr, "Nviz: " + e.message
 
         stop = time.time()

gui/wxpython/nviz/init.cpp

@@ -20,16 +20,16 @@
 
 static void swap_gl();
 static int print_error(const char *, const int);
-static std::string *error;
+static PyObject *logStream;
 
 /*!
   \brief Initialize Nviz class instance
 */
-Nviz::Nviz()
+Nviz::Nviz(PyObject *log)
 {
     G_gisinit(""); /* GRASS functions */
 
-    error = new std::string();
+    logStream = log;
 
     G_set_error_routine(&print_error);
     G_set_verbose(0); // TODO: read progress info
@@ -57,7 +57,7 @@ Nviz::~Nviz()
     data = NULL;
     glCanvas = NULL;
 
-    delete error;
+    logStream = NULL;
 }
 
 /*!
@@ -135,22 +135,19 @@ void Nviz::SetBgColor(const char *color_str)
     return;
 }
 
-std::string Nviz::GetErrorMsg()
-{
-    if (error) {
-	std::string retStr(*error);
-	error->clear();
-	return retStr;
-    }
-
-    return std::string();
-}
-
 int print_error(const char *msg, const int type)
 {
-    if (error) {
-	error->append(msg);
-	error->append("\n");
+    char *fmsg;
+
+    if (logStream) {
+	fmsg = (char *) G_malloc (strlen(msg) + 6);
+	sprintf (fmsg, "Nviz: %s", msg);
+    
+	PyFile_WriteString(fmsg, logStream);
+	G_free((void *) fmsg);
+    }
+    else {
+	fprintf(stderr, "Nviz: %s\n", msg);
     }
 
     return 0;

gui/wxpython/nviz/nviz.h

@@ -1,7 +1,6 @@
 #ifndef WXNVIZ_H
 #define WXNVIZ_H
 
-#include <string>
 #include <vector>
 
 extern "C" {
@@ -25,6 +24,8 @@ extern "C" {
 
 #include <wx/glcanvas.h>
 
+#include <Python.h>
+
 class Nviz
 {
 private:
@@ -37,13 +38,11 @@ private:
 
 public:
     /* constructor */
-    Nviz();
+    Nviz(PyObject *);
 
     /* destructor */
     ~Nviz();
 
-    std::string GetErrorMsg();
-
     /* change_view.cpp */
     int ResizeWindow(int, int);
     std::vector<double> SetViewDefault();

gui/wxpython/nviz/nviz.i

@@ -17,6 +17,4 @@ namespace std {
    %template(IntVecIntMap) map<int, vector<int> >;
 }
 
-%include "std_string.i"
-
 %include "nviz.h"