- Fixed a bug in the pure virtual functions

Bruno da Silva de Oliveira · Bruno da Silva de Oliveira · commit a933e458b302 · 2003-08-16T19:13:45.000Z
[SVN r19648]
diff --git a/pyste/NEWS b/pyste/NEWS
@@ -2,6 +2,9 @@
 Applied a patch by Gottfried Ganssauge that adds exception specifiers to 
 wrapper functions and pointer declarations. Thanks a lot Gottfried!!
 
+Applied a patch by Prabhu Ramachandran that fixes ae problem with the 
+pure virtual method generation. Thanks again Prabhu!
+
 10 August 2003
 Support for incremental generation of the code has been added. This changes
 how --multiple works; documentation of this new feature will follow. Thanks
diff --git a/pyste/TODO b/pyste/TODO
@@ -9,4 +9,4 @@
 
 - Virtual operators  
 
-- Apply Gottfried patches
+- args() support
diff --git a/pyste/src/Pyste/ClassExporter.py b/pyste/src/Pyste/ClassExporter.py
@@ -713,12 +713,13 @@ def DefaultImpl(method, param_names):
                 if method.abstract:
                     s = indent2 + 'PyErr_SetString(PyExc_RuntimeError, "pure virtual function called");\n' +\
                         indent2 + 'throw_error_already_set();\n' 
-                    if method.result.FullName() != 'void':
-                        s += indent2 + 'return %s();\n' % method.result.FullName()
+                    params = ', '.join(param_names)
+                    s += indent2 + '%s%s(%s);\n' % \
+                        (return_str, method.name, params)
                     return s
-                else:
+                else:                 
                     return indent2 + '%s%s(%s);\n' % \
-                    (return_str, method.FullName(), ', '.join(param_names)) 
+                        (return_str, method.FullName(), ', '.join(param_names)) 
             else:
                 # return a call for the wrapper
                 params = ', '.join(['this'] + param_names)
diff --git a/pyste/src/Pyste/pyste.py b/pyste/src/Pyste/pyste.py
@@ -43,7 +43,7 @@
 import time
 from declarations import Typedef
 
-__VERSION__ = '0.9.13'
+__VERSION__ = '0.9.14'
 
 def RecursiveIncludes(include):
     'Return a list containg the include dir and all its subdirectories'

Original file line number	Diff line number	Diff line change
`@@ -9,4 +9,4 @@`
`9`	`9`
`10`	`10`	`- Virtual operators`
`11`	`11`
`12`		`-- Apply Gottfried patches`
	`12`	`+- args() support`