Commit Graph

19 Commits

Author SHA1 Message Date
Ian Bell
9eb3eb8db1 Run clang-format with claude code and fix VS warnings (#2629)
* Run clang-format with claude code and fix VS warnings

* More clang-format

* And the tests too

* Cleanup from clang-tidy

* More constness and modernization

* Cleanup and modernization
2025-10-05 11:02:51 -04:00
Ian Bell
456d365428 Only initlalize the config if not already initialized 2025-06-14 09:38:30 -04:00
Ian Bell
710737db9e First initialization order fix from coverity 2025-06-14 09:37:20 -04:00
Ian Bell
45fdfaa8ea Config should use unordered_map
Safe change, doubles the speed of lookups
2025-04-28 20:30:11 -04:00
Julien Marrec
05c8cf503b Lint: use automated tooling to reformat C++ and CMakeLists files (#2103)
* Add initial clang tidy / clang format config files

* Clang format the entire codebase

```
find ./src -regextype posix-extended -regex '.*\.(cpp|hpp|c|h|cxx|hxx)$' | xargs clang-format-12 -style=file -i -fallback-style=none
find ./include -regextype posix-extended -regex '.*\.(cpp|hpp|c|h|cxx|hxx)$' | xargs clang-format-12 -style=file -i -fallback-style=none
find ./Web -regextype posix-extended -regex '.*\.(cpp|hpp|c|h|cxx|hxx)$' | xargs clang-format-12 -style=file -i -fallback-style=none
find ./dev -regextype posix-extended -regex '.*\.(cpp|hpp|c|h|cxx|hxx)$' | xargs clang-format-12 -style=file -i -fallback-style=none
find ./wrappers -regextype posix-extended -regex '.*\.(cpp|hpp|c|h|cxx|hxx)$' | xargs clang-format-12 -style=file -i -fallback-style=none
```

* Add a .cmake-format file and reformat CmakeLists.txt with it

https://github.com/cheshirekow/cmake_format

* Add a clang-format workflow


only runs on PRs, only on touched files
2022-03-31 10:51:48 -04:00
ian
84dcbda935 Allow REFPROP error threshold to be changed; closes #1847 2019-08-11 21:43:29 -06:00
Ian Bell
b4b2bae981 Fix errors identified by Xcode static analysis 2019-01-12 21:47:22 -07:00
Ian Bell
93a1729ff7 Fix bug with conditional 2019-01-11 18:58:28 -07:00
Jorrit Wronski
db6ad8d435 Force unloading of REFPROP upon path change (#1738) 2018-10-24 20:04:03 +02:00
Ian Bell
d61c365215 Parse refprop HMX.BNC file and load coefficients (#1304)
* Add parser for HMX.BNC from REFPROP; closes #1294

Also move configuration variables into main pybind11 module

* Add the ability to over-write departure functions and binary interaction parameters
2016-10-30 21:26:05 -06:00
Ian Bell
be6d2ffdbb Add config keys to pybind11 interface; closes #1299 2016-10-27 21:15:23 -06:00
Ian Bell
2bbb6d068c Also add high-level interface to pybind11 interface 2016-10-21 19:42:23 -06:00
Ian Bell
9f0fa0e47c Added docs for configuration keys, and functions to C++ and python to make this possible; closes #761 2015-08-20 15:57:21 -05:00
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