avoid copying vectors as arguments and pass a const reference instead

This commit is contained in:
Axel Kohlmeyer
2025-06-25 19:55:10 -04:00
parent 7d86d7f89a
commit b093094d5e
3 changed files with 3 additions and 3 deletions

View File

@ -73,7 +73,7 @@ class FixElectrodeConp : public Fix {
enum class VarStyle { CONST, EQUAL, UNSET }; enum class VarStyle { CONST, EQUAL, UNSET };
virtual void update_psi(); virtual void update_psi();
virtual void pre_update() {}; virtual void pre_update() {};
virtual void recompute_potential(std::vector<double>, std::vector<double>){}; virtual void recompute_potential(const std::vector<double> &, const std::vector<double> &) {};
virtual std::vector<double> constraint_projection(std::vector<double>); virtual std::vector<double> constraint_projection(std::vector<double>);
virtual std::vector<double> constraint_correction(std::vector<double>); virtual std::vector<double> constraint_correction(std::vector<double>);
virtual void compute_macro_matrices(); virtual void compute_macro_matrices();

View File

@ -113,7 +113,7 @@ std::vector<double> FixElectrodeConq::constraint_projection(std::vector<double>
Recompute group potential as average for output if using cg algo Recompute group potential as average for output if using cg algo
------------------------------------------------------------------------- */ ------------------------------------------------------------------------- */
void FixElectrodeConq::recompute_potential(std::vector<double> b, std::vector<double> q_local) void FixElectrodeConq::recompute_potential(const std::vector<double> &b, const std::vector<double> &q_local)
{ {
const int n = b.size(); const int n = b.size();
auto a = ele_ele_interaction(q_local); auto a = ele_ele_interaction(q_local);

View File

@ -34,7 +34,7 @@ class FixElectrodeConq : public FixElectrodeConp {
public: public:
FixElectrodeConq(class LAMMPS *, int, char **); FixElectrodeConq(class LAMMPS *, int, char **);
void update_psi() override; void update_psi() override;
void recompute_potential(std::vector<double>, std::vector<double>) override; void recompute_potential(const std::vector<double> &, const std::vector<double> &) override;
std::vector<double> constraint_projection(std::vector<double>) override; std::vector<double> constraint_projection(std::vector<double>) override;
std::vector<double> constraint_correction(std::vector<double>) override; std::vector<double> constraint_correction(std::vector<double>) override;