On Thu, Apr 21, 2011 at 11:21 AM, Colin Guthrie <gmane at colin.guthr.ie> wrote: > 'Twas brillig, and Margarita Olaya at 21/04/11 17:16 did gyre and gimble: >> Hi Tanu, >> >> On Thu, Apr 21, 2011 at 12:12 AM, Tanu Kaskinen <tanuk at iki.fi> wrote: >>> On Thu, 2011-04-21 at 10:13 +0530, Arun Raghavan wrote: >>>> On Wed, 2011-04-20 at 23:26 -0500, Margarita Olaya wrote: >>>>> Hi, >>>>> >>>>> The initial implementation of ?jack detection is using threads and >>>>> simple file operations like open, close and read currently I'm looking >>>>> into using pa_threads and glib functions. First step It is simple >>>>> enough e.g change threads by pa_threads and for file operations I have >>>>> >>>>> u->fd = g_open(u->device_id, O_RDONLY) >>>>> ioch = g_io_channel_unix_new(u->fd); >>>>> >>>>> then use g_io_channel_read_chars() instead of read. >>>>> >>>>> I'm wonder if it is the beast approach or I'm missing any PA function >>>>> that fits better for this purpose. >>>> >>>> I'm not aware of any PA code that uses glib for this. module-pipe-sink >>>> seems to do most of the things you need so that might serve as a good >>>> starting point. >>> >>> Also, the server side code doesn't use glib for anything - the glib >>> dependency has been avoided on purpose. (I don't know or remember the >>> details why Lennart has been avoiding glib - is it only to keep the >>> amount of dependencies low, or are there some other reasons why PA >>> shouldn't use glib.) >> >> If that is the case, It looks that it is better to keep the simple >> file operations, right? Otherwise I'll add a glib dependency. > > Yes, please use the various pa_ functions already in use for other basic > file operations. No glib deps in the core PA or modules (module-gconf is > an exception but even it will die away soon - for various values of "soon"). > ok, thanks, then I'll go with pa_open, pa_read and pa_close, Thanks, Margarita > Col > > > > -- > > Colin Guthrie > gmane(at)colin.guthr.ie > http://colin.guthr.ie/ > > Day Job: > ?Tribalogic Limited [http://www.tribalogic.net/] > Open Source: > ?Mageia Contributor [http://www.mageia.org/] > ?PulseAudio Hacker [http://www.pulseaudio.org/] > ?Trac Hacker [http://trac.edgewall.org/] > > _______________________________________________ > pulseaudio-discuss mailing list > pulseaudio-discuss at mail.0pointer.de > https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss >
