13 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
84bdcb1aee Add configuration key for maximum table directory size
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2015-02-27 22:12:59 -07:00
Ian Bell
77046321b8 Allow user to specify another HMX.BNC path for REFPROP
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2015-02-25 08:22:34 -07:00
Ian Bell
7cdf05cb32 Hide getter/setter from SWIG as well.
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-12-12 18:51:11 -05:00
Ian Bell
93726173b8 Try to hide rapidjson header to swig for some wrappers that get confused
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-12-10 23:39:17 -05: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
bab22184c9 Fixed handling of string configuration keys
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-12-09 13:19:38 -05:00
Ian Bell
0115553119 Allow user to specify the REFPROP root path for fluids and mixtures
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-12-09 12:21:32 -05:00
Ian Bell
f3b96109fd By default, temperatures within 1 uK of the critical point will now be treated as the critical point
Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-29 15:37:54 -05:00
Ian Bell
a73cfd4c49 I'm not sure why, but this fixes VS2012+
Closes https://github.com/CoolProp/CoolProp/issues/230

Signed-off-by: Ian Bell <ian.h.bell@gmail.com>
2014-11-23 21:48:24 -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