Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readme unclear about supported listeners #53

Closed
jlmuir opened this issue Jan 23, 2020 · 1 comment
Closed

Readme unclear about supported listeners #53

jlmuir opened this issue Jan 23, 2020 · 1 comment

Comments

@jlmuir
Copy link
Contributor

jlmuir commented Jan 23, 2020

The file Readme.md contains the following item in the Features section:

Support of all listeners ChannelAccess supports: ConnectionListener, AccessRightListener, Value Listener (Monitor)

It's unclear to me what that means. Is it saying that it supports similar listening to what can be done in a JCA implementation?

JCA contains the interfaces ConnectionListener, AccessRightsListener, and MonitorListener (among others). If that's what this is referring to, then perhaps the item should read as follows:

Supports listening to all Channel Access events that can be listened to with a JCA implementation (e.g., similar to JCA's ConnectionListener, AccessRightListener, MonitorListener, etc.)

@simondelabici
Copy link
Contributor

@jlmuir

Thanks for the feedback and, yes, you are right the Readme file seems rather ambiguous. It wasn't clear to me, either, whether the comment referred to JCA or to the features supported by the channel access protocol itself.

I will modify the text to say what is explicitly supported directly, rather than what is supported by reference to some other (currently unspecified) implementations.

Thus:-

* Supports channel monitoring including mask capability to select events-of-interest (eg value, log, alarm 
and/or property changes).
* Supports the following listeners: ConnectionListener, AccessRightsListener.

simondelabici added a commit that referenced this issue Mar 17, 2020
Issue #59: Transition away from 'travis-ci.org' site to newer 'travis-ci.com' site.
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

No branches or pull requests

2 participants