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

Provide an isRunning method #534

Closed
nvaccessAuto opened this issue Jan 21, 2010 · 3 comments
Closed

Provide an isRunning method #534

nvaccessAuto opened this issue Jan 21, 2010 · 3 comments
Assignees
Milestone

Comments

@nvaccessAuto
Copy link

Reported by Q on 2010-01-21 21:30
The NVDA controller interface should provide a method to easily determine if there is an active instance of NVDA. A simple bool would suffice.

@nvaccessAuto
Copy link
Author

Comment 2 by mdcurran on 2010-01-21 22:31
Very happy to add one. However out of interest, how exactly would your app use this.
The way we have designed NVDAControllerClient is so that you can safely call the methods no matter if its running or not. The return code will be non-0 if there was an error communicating with NVDA.
So I guess I just want to be sure (for best performance on your side) that you're not just going to call one NVDA method if it is running. If so, you would have called two methods rather than one.
Also if we did add an isRunning method, it would return 0 if it was running, and an error code (non-zero) if it was not. Is this ok?
Thats just part of how we've chosen to use RPC etc.

@nvaccessAuto
Copy link
Author

Comment 3 by bramd (in reply to comment 2) on 2010-01-21 22:43
Replying to mdcurran:

Very happy to add one. However out of interest, how exactly would your app use this.

Q, I and others work on the Qwitter client for Twitter. We support various speech outputs and have to determine which one to use. If a speech output module is unavailable, aka screenreader is not running, we'll skip it and select the next one.

Of course, how you implement this is your choice. Just do what is consistent with the rest of your interface.

@nvaccessAuto
Copy link
Author

Comment 4 by mdcurran on 2010-01-22 07:36
Implemented in r3480.
nvdaController_testIfRunning()
Like all of the other functions, returns 0 on success (in this case, NVDA is running), non-zero otherwize.
Changes:
State: closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants