Fix redundant density symmetrization in OFDFT

[Cstandardlib]

Reminder

• I have read AGENTS.md and docs/developers_guide/agent_governance.md.
• I have linked an issue or explained why this PR does not need one.
• I have added adequate unit tests and/or case tests, or explained why not.
• I have listed the exact verification commands run and their results.
• I have described user-visible behavior changes, including INPUT parameter changes.
• I have explained core-module impact for ESolver, HSolver, ElecState, Hamilt, Operator, Psi, or other source/ changes.
• I have requested any needed governance exception below.

Linked Issue

Fix #7724

Unit Tests and/or Case Tests for my changes

• No tests added.

What's changed?

• Remove redundant rho_symmetrize operation whose results will soon be overwritten in esolver_of.cpp.
• Now no unnecessary symmetrization is performed for ofdft with init_chg!=file, saving up to 95% of total time.

Governance Notes

• INPUT/docs changes: No changes.

[Copilot]

Pull request overview

This PR targets the OFDFT initialization hotspot reported in #7724 by avoiding a redundant density symmetrization step when the initial (non-file) density is immediately replaced with a uniform density in ESolver_OF::before_opt.

Changes:

• Guard Symmetry_rho::symmetrize_rho(...) so it only runs when init_chg == "file" in ESolver_OF::before_opt.
• Add an in-code note explaining why symmetrization is skipped for non-file initialization.
• Minor whitespace/formatting cleanups in esolver_of.cpp.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

[mohanchen]

LGTM