Merge ~bhill/epics-base:EPICS-and-CA-version-from-CA-tools into ~epics-core/epics-base/+git/epics-base:3.15
Proposed by
Bruce Hill
Status: | Merged |
---|---|
Merged at revision: | 937878e0a98e6835f272a22f9e03a342ae1a6e72 |
Proposed branch: | ~bhill/epics-base:EPICS-and-CA-version-from-CA-tools |
Merge into: | ~epics-core/epics-base/+git/epics-base:3.15 |
Diff against target: |
153 lines (+26/-5) 4 files modified
src/ca/client/tools/caget.c (+6/-1) src/ca/client/tools/cainfo.c (+7/-1) src/ca/client/tools/camonitor.c (+7/-2) src/ca/client/tools/caput.c (+6/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
mdavidsaver | Approve | ||
Andrew Johnson | Approve | ||
Review via email: mp+358142@code.launchpad.net |
Description of the change
Adds a -V command line argument to caget, cainfo, camonitor, and caput to display the EPICS version string and the CA protocol version.
To post a comment you must log in.
I like the idea of '-V', but some cleanup is needed. There is no reason to change all of the includes (back) from brackets '<>' to quotes '""'. Also, please maintain consistent indentation! Switch statements are hard enough to parse as it is.