Closed mdavidsaver closed 3 years ago
:x: Build pvxs 1.0.279 failed (commit https://github.com/mdavidsaver/pvxs/commit/e0477fbc36 by @mdavidsaver)
:white_check_mark: Build pvxs 1.0.288 completed (commit https://github.com/mdavidsaver/pvxs/commit/730f0ddcad by @mdavidsaver)
I'm creating this PR mainly as a venue to mention several (I think minor) API changes in the big set of changes driven by my porting P4P to use PVXS (including the gateway component). It will likely remain open until I'm closer to finalizing that project.
So far these are:
Several argument types change from
Value&&
toconst Value&
. This should not be a breaking change as an implicit cast is possible. It may trigger warnings about now unnecessarystd::move()
. This includes:SharedPV::post()
and friends, and several of the client*Builder
methods.The signature of the
SharedPV
callbacksonFirstConnect()
andonLastDisconnect()
are changed to add a new argumentSharedPV&
. This is added for consistency with theonPut()
andonRPC()
callbacks which already had this convience.The
server::Config
andclient::Config
classes add a new static methodfromEnv()
which is identical to thefrom_env()
method. The originalfrom_env()
will later be removed.Some exception types have changed. I've tried to maintain consistency, but likely haven't been 100% successful.
Setting an invalid hostname in
$EPICS_PVAS_INTF_ADDR_LIST
changes from a warning to an error. It didn't seem friendly to silently bind to0.0.0.0
when a user is explicitly asking for something else.