nikhilgupta10 / GridLAB-D

Other
1 stars 0 forks source link

#1006 Implement module API functions as inline calls rather than macros, #2717

Closed nikhilgupta10 closed 7 years ago

nikhilgupta10 commented 7 years ago

05/13/08 20:54:08 changed by d3g637 /u00b6

About half done using #ifdef cplusplus test.

Curiously, Doxygen doesn't seem to notice the change and so doesn't document the API propertly. Will have to investigate why later.

06/02/08 11:04:37 changed by d3g637 /u00b6
Partially completed, but really can be completed until all modules are implemented as C++ code. Tape is still C code.

,

nikhilgupta10 commented 7 years ago

nikhilgupta10 imported these comments from Sourceforge: ntenney: * type changed from defect to enhancement ,

dchassin:Nikhil added this comment from #831 for testing purposes. The following was implemented in ticket:830 as a solution that seems to work ok.

--- configure.ac    (revision 4702)
+++ configure.ac    (working copy)
@@ -476,8 +476,9 @@
             [gld_cv_matlab_ldpath=`$MATLAB -e | grep ^DYLD_LIBRARY_PATH= | cut -f2 -d=`
 ])])])
 AS_IF([test /x$MATLAB\ != x],
-    [MATLAB_CPPFLAGS=\-I$MATLAB/extern/include\n    -     MATLAB_LDFLAGS=\-L${MATLAB_LDPATH//:/ -L}\n    +    [MATLAB=`$MATLAB -e | grep ^MATLAB= | cut -f2 -d=`
+     MATLAB_CPPFLAGS=\-I$MATLAB/extern/include\n    +     MATLAB_LDFLAGS=\-L${gld_cv_matlab_ldpath//:/ -L}\n          MATLAB_LIBS=\-lmx -leng\n          GLD_CPPFLAGS=\$GLD_CPPFLAGS -DHAVE_MATLAB\n          HAVE_MATLAB=yes],

,

dchassin: * status changed from new to accepted ,

dchassin: * status changed from accepted to closed

Done in changeset:105

,