Code review comment for ~bhill/epics-base:pvAccess-envDefs.h

Revision history for this message
mdavidsaver (mdavidsaver) wrote :

> I got the defaults from pvAccessConfig.xlsx

Where did you find this file? I don't think I've come across it before.

I like the idea of allowing users to check the PVA settings in the same way as the CA settings. The unfortunate reality is that these two are managed quite differently. cf. pv/configuration.h

I don't think it would be a good idea to merge this without corresponding changes to pvAccessCPP. To do so would potentially confuse users by showing incorrect values when defaults are being override in code, of when environment variables are ignored altogether.

review: Needs Fixing

« Back to merge proposal