Fix thread-unsafe lazy initialization in ExpDeproj::mass()

[Copilot]

ExpDeproj::mass() used lazy initialization guarded only by a size check, making it unsafe for concurrent calls before the grid was populated.

Changes

• include/ExpDeproj.H: Move initialize() call into the constructor, eliminating lazy init entirely.
• exputil/ExpDeproj.cc: Remove the now-redundant mv.size() == 0 guard from mass().

// Before
ExpDeproj(int n=4000) : ngrid(n) {}  // grid populated on first mass() call

double ExpDeproj::mass(double R) {
  if (mv.size() == 0) initialize();  // race condition under concurrent access
  ...
}

// After
ExpDeproj(int n=4000) : ngrid(n) { initialize(); }  // grid ready at construction

double ExpDeproj::mass(double R) {
  // no init check needed
  ...
}

Grid is now computed once at construction time, before any thread can invoke mass().

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.