EEL_USER is true when eelPMESWITCH is true
The monster Verlet kernel patch included
#define EEL_USER(e) ((e) == eelUSER || (e) == eelPMEUSER || (e) == (eelPMESWITCH))
I think it should be
#define EEL_USER(e) ((e) == eelUSER || (e) == eelPMEUSER || (e) == (eelPMEUSERSWITCH))
Corrected definition of EEL_USER
Looks like a copy-paste typo occured at some point in the nbnxn
development. EEL_USER is only used in runner.c, and only to issue a
fatal error with user-supplied non-bonded potentials. That clearly
should be not applicable to eelPMESWITCH.
#4 Updated by Teemu Murtola almost 6 years ago
- Status changed from Fix uploaded to Resolved
- Affected version - extra info set to 4.6-4.6.2
Was fixed in https://gerrit.gromacs.org/#/c/2459/ for 4.6.3 (not possible to set that target version any longer). For some reason, the commit wasn't linked here.