Skip to content

Add -S option to pvget/pvmonitor to stringify byte arrays - #222

Merged
mdavidsaver merged 1 commit into
epics-base:masterfrom
JJL772:pr-add-S-stringify
Sep 11, 2026
Merged

mdavidsaver merged 1 commit into
epics-base:masterfrom
JJL772:pr-add-S-stringify

Conversation

@JJL772

@JJL772 JJL772 commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

This emulates the behavior of the caget -S option. Byte arrays are displayed as strings.

Example (without -S):

$ pvget SIOC:TST:SYS0:APP_DIR
SIOC:TST:SYS0:APP_DIR 2026-03-17 23:09:56.242  [47,109,101,100,105,97,47,66,105,103,68,114,105,118,101,47,80,114,111,106,101,99,116,115,47,101,112,105,99,115,47,105,111,99,47,97,116,108,97,115,45,103,105,116,47,105,111,99,66,111,111,116,47,115,105,111,99,45,116,115,116,45,115,121,115,48,0]

Example (with -S):

$ pvget -S SIOC:TST:SYS0:APP_DIR
SIOC:TST:SYS0:APP_DIR 2026-03-17 22:39:05.669  /media/BigDrive/Projects/epics/ioc/atlas-git/iocBoot/sioc-tst-sys0

pvData PR: epics-base/pvDataCPP#105

@AppVeyorBot

Copy link
Copy Markdown

❌ Build pvAccessCPP 1.0.151 failed (commit 742e286f1f by @JJL772)

@JJL772
JJL772 force-pushed the pr-add-S-stringify branch from 3a2d753 to a1a7e37 Compare June 30, 2026 16:16
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@AppVeyorBot

Copy link
Copy Markdown

❌ Build pvAccessCPP 1.0.157 failed (commit b25bdedd80 by @JJL772)

@mdavidsaver

Copy link
Copy Markdown
Member

Following epics-base/pvDataCPP#105, are any changes needed to this PR?

@JJL772

JJL772 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Nope- this is good to go as well

@mdavidsaver
mdavidsaver merged commit ea48b85 into epics-base:master Sep 11, 2026
1 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants