Review: Approve
Looks good.
wrt. dbpf. So this would look like:
> dbpf "some:pv" "1 2 3 4"
Did you consider switching dbpf() to use iocshArgArgv to capture the value element(s)?
This doesn't have to be the only argument. This seems more natural to me, and potentially
avoids some of the added string parsing code. I suppose this would be a disadvantage for
vxworks though?
Also, an example of this should be added to the help message for dbpf.
This is in dbIocRegister.c. I can do this after merge if needed.
--
https://code.launchpad.net/~dirk.zimoch/epics-base/+git/epics-base/+merge/386175
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.
- References:
- [Merge] ~dirk.zimoch/epics-base:fix_zero_size_arrays into epics-base:7.0 Dirk Zimoch via Core-talk
- Navigate by Date:
- Prev:
Jenkins build is back to normal : epics-pvData-mac #187 APS Jenkins via Core-talk
- Next:
Re: [Merge] ~dirk.zimoch/epics-base:fix_zero_size_arrays into epics-base:7.0 Dirk Zimoch via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
- Navigate by Thread:
- Prev:
[Merge] ~dirk.zimoch/epics-base:fix_zero_size_arrays into epics-base:7.0 Dirk Zimoch via Core-talk
- Next:
Re: [Merge] ~dirk.zimoch/epics-base:fix_zero_size_arrays into epics-base:7.0 Dirk Zimoch via Core-talk
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
2017
2018
2019
<2020>
2021
2022
2023
2024
|