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

Some methods missing #8

Closed
jcelerier opened this issue Mar 16, 2018 · 5 comments
Closed

Some methods missing #8

jcelerier opened this issue Mar 16, 2018 · 5 comments

Comments

@jcelerier
Copy link
Contributor

Namely:

  • jack_port_connected
  • jack_port_get_latency

Can be solved by adding:

#define jack_port_connected                 WJACK_port_connected
#define jack_port_get_latency               WJACK_port_get_latency

and

JPFUN(1, int,            port_connected, (const jack_port_t *p), (p), NULL)
JPFUN(1, jack_nframes_t, port_get_latency, (jack_port_t *p), (p), NULL)
@x42
Copy link
Owner

x42 commented Mar 16, 2018

jack_port_get_latency is deprecated and also does not work properly with recent jackd and most jack apps that have a latency callback. prefer jack_port_get_latency_range. Is there an application that needs it?

x42 added a commit that referenced this issue Mar 16, 2018
@jcelerier
Copy link
Contributor Author

jcelerier commented Mar 16, 2018

it's what's used in portaudio's JACK implementation (https://app.assembla.com/spaces/portaudio/git/source/master/src/hostapi/jack/pa_jack.c#ln598)

@x42
Copy link
Owner

x42 commented Mar 16, 2018

that code would actually benefit from the new API. portaudio internals do match jack's jack_port_get_latency_range API: min/max and capture/playback can be directly used to set all of
defaultLowOutputLatency, defaultHighOutputLatency , defaultLowInputLatency, defaultHighInputLatency (currently only input latency is set)

Anyway, what's the use-case of using weakjack for pulseaudio's jack-sink?

@jcelerier
Copy link
Contributor Author

jcelerier commented Mar 16, 2018

I'm hitting this bug in jack on windows: jackaudio/jack2#275 and wanted to try to update pa_jack.c to use weak_jack.
But actually, just using the JACK api directly from my own code seems to be much easier.

(note: I'm talking about portaudio, not pulseaudio)

@x42
Copy link
Owner

x42 commented Jul 25, 2018

+1 for using the JACK API directly!

@x42 x42 closed this as completed Jul 25, 2018
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