Make retrace_call void()

diff --git a/glretrace_main.cpp b/glretrace_main.cpp
index 922ec303b01e0f72e7976a3aaa5d90ef45868baa..4fd8304ccac6a613897be393fdef485ae29d82f8 100644 (file)
--- a/glretrace_main.cpp
+++ b/glretrace_main.cpp
@@ -158,15 +158,16 @@ static void display(void) {
         const std::string &name = call->name();
         bool skipCall = false;
 
+        if (retrace::verbosity >= 1) {
+            std::cout << *call;
+            std::cout.flush();
+        }
+
         if ((name[0] == 'w' && name[1] == 'g' && name[2] == 'l') ||
             (name[0] == 'g' && name[1] == 'l' && name[2] == 'X')) {
             // XXX: We ignore the majority of the OS-specific calls for now
             if (name == "glXSwapBuffers" ||
                 name == "wglSwapBuffers") {
-                if (retrace::verbosity >= 1) {
-                    std::cout << *call;
-                    std::cout.flush();
-                };
                 frame_complete(call->no);
                 if (double_buffer)
                     drawable->swapBuffers();

diff --git a/retrace.hpp b/retrace.hpp
index 916cf3823c2b920df2a8d3888a3e346fc9ce1c73..05cef5970ed88cad50efb8a7bb8d338076efbc0d 100644 (file)
--- a/retrace.hpp
+++ b/retrace.hpp
@@ -82,7 +82,7 @@ public:
 extern int verbosity;
 
 
-bool retrace_call(Trace::Call &call);
+void retrace_call(Trace::Call &call);
 
 
 } /* namespace retrace */

diff --git a/retrace.py b/retrace.py
index 27d0afe8879c94e6bb013ee34e4b33d9a5c7159b..6c688b92d4941facd0b31aac46eb7f92f583218a 100644 (file)
--- a/retrace.py
+++ b/retrace.py
@@ -246,14 +246,9 @@ class Retracer:
             if function.sideeffects:
                 self.retrace_function(function)
 
-        print 'bool retrace::retrace_call(Trace::Call &call) {'
+        print 'void retrace::retrace_call(Trace::Call &call) {'
         print '    const char *name = call.name().c_str();'
         print
-        print '    if (retrace::verbosity >= 1) {'
-        print '        std::cout << call;'
-        print '        std::cout.flush();'
-        print '    };'
-        print
 
         func_dict = dict([(function.name, function) for function in functions])
 
@@ -261,13 +256,12 @@ class Retracer:
             function = func_dict[function_name]
             if function.sideeffects:
                 print '        retrace_%s(call);' % function.name
-            print '        return true;'
+            print '        return;'
     
         string_switch('name', func_dict.keys(), handle_case)
 
         print '    if (retrace::verbosity >= 0)'
         print '        std::cerr << call.no << ": warning: unknown call " << call.name() << "\\n";'
-        print '    return false;'
         print '}'
         print