search

ls1mardyn / ls1-mardyn

ls1-MarDyn is a massively parallel Molecular Dynamics (MD) code for large systems. Its main target is the simulation of thermodynamics and nanofluidics. ls1-MarDyn is designed with a focus on performance and easy extensibility.
http://www.ls1-mardyn.de
Other
28 stars 14 forks source link

Unused code cleanup #282

Open cniethammer opened 7 months ago

cniethammer commented 7 months ago

Get rid of interfaces, methods, and functions that are not used anymore.

This should help improve code readability, compile times, and reduce binary size if compilers cannot optimize stuff away.

HomesGH commented 7 months ago

This issue is a good idea! There is a lot of mess in the code, especially regarding the thermostats, e.g. here and here. For the latter: Why is the warning only printed if unit tests are disabled? This FIXME for the thermostat is a bit worrying as this is in a central part of the code.

Also some output like this seems to be outdated.