Commit Graph

6 Commits

Author SHA1 Message Date
mikekaganski
28691e8e75 const, ref and iterator optimization
1. Another set of arguments optimized:
* args-by-val converted to args-by-ref
* in some cases, reverse has been done, if that is better: if the
object's copy is modified in the func, it's better to avoid extra copy
from arg to local var
2. some const functions marked as such (this is just a beginning)
3. iterators were replaced with const_iterators where applicable
4. catches that catch std::exception& changed to catch (...) - that is
safer; if exception handling is restructured, this will have to be
reconsidered anyway
5. removed some basic structures' constructors; changed them to hold
const char*s to avoid unnecessary string constructions;
6. in some places, moved variable declarations to their definitions, to
avoid calling default constructors and then assigning
7. removed some unnecessary shared_pointers in favor of local objects;
8. in FORTRAN-style functions, added const specifiers to input doubles;
9. fixed a place where values were inserted into a map while iterating
through it
10. fixed is_valid_*_derivative: they could accept incorrect values and
throw when upper index is less than lower
2015-03-01 10:51:01 +10:00
Ian Bell
6b312b7fa7 Expose the configuration functions through SWIG - closes #326
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-12-10 01:25:43 -05:00
Ian Bell
362a84254f Squelch some compiler warnings
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-27 12:08:44 -05:00
Ian Bell
be69e81916 configuration now works with the X() macros of http://www.drdobbs.com/the-new-c-x-macros/184401387
This is a very nice solution, though a bit confusing at first.

Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-19 00:11:20 -05:00
Ian Bell
6f4a4fcc93 getting/setting configuration values from python works
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-18 23:33:32 -05:00
Ian Bell
9ab6d2721e Updated configuration setting and getting - ready for json integration at the python level, finally
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-18 22:40:46 -05:00