Changes between Version 26 and Version 27 of SwiggingHlsvd

Timestamp:

Jul 14, 2009, 5:08:24 PM (10 years ago)

Author:

flip

Comment:

--

SwiggingHlsvd

@@ -317 +317 @@
 ndiv = hlsvd_wrapper.new_ndiv()
 }}}
-
+{{{
+#!rst
 The ``ctypes.byref()`` call is informative. It says that the C interface demands that I pass by reference, but I don't need access to the pointer here in Python. That tells anyone reading this code that I probably don't care what the called function does with the value, hinting that it is input-only. Again, I wouldn't need this at all if the f2c-generated code was tweaked to eliminate the unnecessary pass-by-reference for the input parameters.
 
 Also note that the code above includes some value assignments, like this line which assigns the value .256 to tstep:
-::
-
-    tstep = ctypes.c_float(.256)
-
+}}}
+{{{
+#!python
+tstep = ctypes.c_float(.256)
+}}}
 
 In my ideal Python example above I ignored assigning values to the variables for brevity's sake. And in order to make a fair comparison of SWIG's code to the ideal, I didn't include assignment in that code either. But to fairly compare SWIG and ctypes I need to mention assignment here, because it's simpler with ctypes. The code to assign a value to tstep via the SWIG interface looks like this:
-::
-
-    tstep = hlsvd_wrapper.new_tstep()
-    hlsvd_wrapper.tstep_assign(tstep, .256)
-
-
-Arrays are also considerably uglier in SWIG. First consider the ctypes version (inputs is a list that I read from a disk file):
-::
-
-    yi = InputFloatArrayType()
-    for i, f in enumerate(inputs):
-        yi[i] = inputs[i]
-
+
+{{{
+#!python
+tstep = hlsvd_wrapper.new_tstep()
+hlsvd_wrapper.tstep_assign(tstep, .256)
+}}}
+
+Arrays are also considerably uglier in SWIG. First consider the ctypes version (in which inputs is a list that I read from a disk file):
+
+{{{
+#!python
+yi = InputFloatArrayType()
+for i, f in enumerate(inputs):
+    yi[i] = inputs[i]
+}}}
 
 And now look at the SWIG version:
-::
-
-    yi = hlsvd_wrapper.new_float_array(2048)
-    for i in range(0, 2048):
-        hlsvd_wrapper.float_array_setitem(yi, i, inputs[i])
-
-
+{{{
+#!python
+yi = hlsvd_wrapper.new_float_array(2048)
+for i in range(0, 2048):
+    hlsvd_wrapper.float_array_setitem(yi, i, inputs[i])
+}}}
+
+{{{
+#!rst
 Last but not least, note that the code above is 95% of what I needed to use ctypes to call ``hlsvd()``. There's no interface file, no SWIG-ing step, and no compile & link steps. Just the Python standard library, that's it.
 
@@ -375 +381 @@
 | http://www.netlib.org/f2c/f2c.1
 
-
-
-
-}}}
+}}}