Re: [pulseaudio-discuss] [PATCH] build: fix Mac OS X configure process

2012-11-03 Thread rong deng
2012/11/4 Colin Guthrie : > 'Twas brillig, and Deng Zhengrong at 19/07/12 10:41 did gyre and gimble: >> The original header file doesn't exist on Lion (10.7.4). >> --- >> configure.ac |5 - >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >

Re: [pulseaudio-discuss] [PATCH] tests: make 'check' optional

2012-10-04 Thread rong deng
Thanks a lot for the follow-ups. 2012/10/4 Tanu Kaskinen : > On Mon, 2012-08-20 at 16:24 +0800, rong deng wrote: >> ping. > > Sorry to keep you waiting. I've pushed the patch now (I needed it in my > own packaging work). Thanks for the cont

Re: [pulseaudio-discuss] PulseAudio on OSX

2012-09-14 Thread rong deng
Hi, 2012/9/13 Jared Sohn : > I want to use PulseAudio to separately control the volume of processes > on OSX. Is this possible, and if so what is the best way for doing > this? > > I have found some instructions at > http://www.freedesktop.org/wiki/Software/PulseAudio/Ports/OSX and also > have fo

[pulseaudio-discuss] thank you all for a nice summer of code

2012-08-30 Thread rong deng
Hi PulseAudio community, It's a great honour for me to be selected as this year's google summer of code on PulseAudio project and now GSoC is nearing the end. I would say it's a nice experience for me. I'm involved in the mailing list discussion, get used to talk on IRC room, I've never done this

Re: [pulseaudio-discuss] [PATCH] resampler: Fix crash if 'auto' resampler chooses ffmpeg with variable rate

2012-08-24 Thread rong deng
2012/8/25 rong deng : > Besides, I once profiled and it showed the trivial resampler is much > faster, esp on resource limited embedded boards. Hmm, the output sound > is not so much bad... to be correct, i shouldn't have said "faster", actuall

Re: [pulseaudio-discuss] [PATCH] resampler: Fix crash if 'auto' resampler chooses ffmpeg with variable rate

2012-08-24 Thread rong deng
2012/8/24 Tanu Kaskinen : > On Wed, 2012-08-22 at 19:31 +0530, Arun Raghavan wrote: >> On Wed, 2012-08-22 at 15:42 +0200, Frédéric Dalleau wrote: >> [...] >> > diff --git a/src/pulsecore/resampler.c b/src/pulsecore/resampler.c >> > index 17f1783..8199bc4 100644 >> > --- a/src/pulsecore/resampler.c

Re: [pulseaudio-discuss] [PATCH 0/2] change pa_bool_t to bool

2012-08-23 Thread rong deng
2012/8/23 Thomas Martitz : > Am 22.08.2012 06:56, schrieb Tanu Kaskinen: >> >> David said: >> >> >> "I don't think there is any reason to not do s/pa_bool_t/bool/g. >> Probably the reason for this might be historical, as some compilers may >> have supported some C99 features but not all of them (Mi

Re: [pulseaudio-discuss] [PATCH 0/2] change pa_bool_t to bool

2012-08-22 Thread rong deng
2012/8/22 Tanu Kaskinen : > On Wed, 2012-08-22 at 08:01 +0800, rong deng wrote: >> 2012/8/22 Tanu Kaskinen : >> > Hi Deng, >> > >> > On Sat, 2012-07-07 at 22:49 +0800, Deng Zhengrong wrote: >> >> As discussed, this might not handle the microsoft c

Re: [pulseaudio-discuss] running in high priority

2012-08-22 Thread rong deng
2012/8/22 Tanu Kaskinen : > On Thu, 2012-08-16 at 04:55 +0200, Sladjan Ri wrote: >> Hi, >> >> is it beneficial to run in high priority at all? I tested and found >> stuttering with the src-high-quality resampling, with nice level -11 >> and CPU went up to 70% on one core (gnome-system-monitor). >>

Re: [pulseaudio-discuss] [PATCH 0/2] change pa_bool_t to bool

2012-08-21 Thread rong deng
2012/8/22 Tanu Kaskinen : > Hi Deng, > > On Sat, 2012-07-07 at 22:49 +0800, Deng Zhengrong wrote: >> As discussed, this might not handle the microsoft compilers well. But let me >> post this series of patches and think out a better way to handle it. Or... we >> don't change it at all. > > The patch

Re: [pulseaudio-discuss] [PATCH] tests: make 'check' optional

2012-08-20 Thread rong deng
ping. 2012/8/13 rong deng : > 2012/8/13 Deng Zhengrong : >> --- >> configure.ac| 17 +++-- >> src/Makefile.am | 14 ++ >> 2 files changed, 29 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac &g

Re: [pulseaudio-discuss] How to decrease cpu usage?

2012-08-14 Thread rong deng
2012/8/15 Tanu Kaskinen : > On Tue, 2012-08-14 at 20:49 -0600, Kelly Anderson wrote: >> In profiling with oprofile on CuBox the code that is hogging cpu is the >> Speex resampler. > > The original reporter had high cpu use also when pulseaudio was supposed > to be idle. Do you have the same problem

Re: [pulseaudio-discuss] introduce the new feature of ring buffer log and ask for comments

2012-08-14 Thread rong deng
2012/8/14 David Henningsson : > On 08/14/2012 08:19 AM, rong deng wrote: >> >> Hi David, >> >> Thanks for your comments, see my inlined comments below: >> >> 2012/8/14 David Henningsson : >>> >>> On 08/13/2012 0

Re: [pulseaudio-discuss] introduce the new feature of ring buffer log and ask for comments

2012-08-13 Thread rong deng
Hi David, Thanks for your comments, see my inlined comments below: 2012/8/14 David Henningsson : > On 08/13/2012 04:59 PM, rong deng wrote: >> >> Ask for comments >> >> >> We design this feature to try to be useful for users and developers. We&

Re: [pulseaudio-discuss] [PATCH] card-restore: Add the ability to save and restore the maximum volume

2012-08-13 Thread rong deng
2012/8/14 Tanu Kaskinen : > On Mon, 2012-08-13 at 15:44 +0800, rong deng wrote: >> 2012/8/13 Tanu Kaskinen : >> > On Fri, 2012-08-03 at 15:12 +0800, Deng Zhengrong wrote: >> >> @@ -336,7 +351,10 @@ static void subscribe_callback(pa_core *c, >> >

[pulseaudio-discuss] introduce the new feature of ring buffer log and ask for comments

2012-08-13 Thread rong deng
Hi list, What it is == This is a new feature which plugs into the current logging system to enhance the debugging experiences for users and developers. It would log all level's messages in a separate place, therefore it have the following benefits: - The most important of this feature i

Re: [pulseaudio-discuss] [PATCH] tests: make 'check' optional

2012-08-13 Thread rong deng
2012/8/13 Deng Zhengrong : > --- > configure.ac| 17 +++-- > src/Makefile.am | 14 ++ > 2 files changed, 29 insertions(+), 2 deletions(-) > > diff --git a/configure.ac b/configure.ac > index ffb2a35..43ccf44 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -579

Re: [pulseaudio-discuss] [PATCH] card-restore: Add the ability to save and restore the maximum volume

2012-08-13 Thread rong deng
2012/8/13 Tanu Kaskinen : > On Fri, 2012-08-03 at 15:12 +0800, Deng Zhengrong wrote: >> module-card-restore now saves the maximum volume. >> >> This change includes a entry version bump. >> --- >> src/modules/module-card-restore.c | 26 +++--- >> 1 files changed, 23 insertion

Re: [pulseaudio-discuss] [PATCH] tests: make 'check' optional

2012-08-09 Thread rong deng
2012/8/9, David Henningsson : > Seems to do what it should. Sorry if I was a little harsh on you > yesterday, it's just that bringing in new dependencies is nothing to > take lightly. It brings work upon others, e g packagers. Most > dependencies also means that size increases, which means that on

Re: [pulseaudio-discuss] [PATCH v2 1/6] build-sys: add `check` test framework

2012-08-08 Thread rong deng
2012/8/9, Tanu Kaskinen : > On Wed, 2012-08-08 at 22:16 +0800, rong deng wrote: >> 2012/8/8, David Henningsson : >> > But what about a user of e g Linux From Scratch or other minimalist >> > distros, who just wants to set up and build PulseAudio as quick as >>

Re: [pulseaudio-discuss] [PATCH v2 1/6] build-sys: add `check` test framework

2012-08-08 Thread rong deng
2012/8/8, David Henningsson : > On 08/08/2012 03:21 PM, rong deng wrote: >> 2012/8/8, David Henningsson : >>> On 07/17/2012 08:45 AM, Deng Zhengrong wrote: >>>> + check test framework >>>> + >>>> +PKG_CHECK_MODULES(LIBCHECK, [ c

Re: [pulseaudio-discuss] [PATCH] build-sys: Link utf8-test to libpulsecommon.

2012-08-08 Thread rong deng
2012/8/8, Chan-yeol Park : > Hi > > On 08/08/2012 09:44 PM, rong deng wrote: >> 2012/8/8, Chan-yeol Park : >>> Hi Tanu, >>> >>> On 08/06/2012 09:07 PM, Tanu Kaskinen wrote: >>>> Fixes this: >>>> /usr/bin/ld: utf8_test-utf8-test.o: u

Re: [pulseaudio-discuss] [PATCH v2 1/6] build-sys: add `check` test framework

2012-08-08 Thread rong deng
2012/8/8, David Henningsson : > On 07/17/2012 08:45 AM, Deng Zhengrong wrote: >> + check test framework >> + >> +PKG_CHECK_MODULES(LIBCHECK, [ check ]) >> +AC_SUBST(LIBCHECK_CFLAGS) >> +AC_SUBST(LIBCHECK_LIBS) >> + > > Is it possible to make this build dependency optional? If so, please do

Re: [pulseaudio-discuss] [PATCH] build-sys: Link utf8-test to libpulsecommon.

2012-08-08 Thread rong deng
2012/8/8, Chan-yeol Park : > Hi Tanu, > > On 08/06/2012 09:07 PM, Tanu Kaskinen wrote: >> Fixes this: >> /usr/bin/ld: utf8_test-utf8-test.o: undefined reference to symbol >> 'pa_log_set_level' >> /usr/bin/ld: note: 'pa_log_set_level' is defined in DSO >> /home/takaskin/dev/pulseaudio/src/.libs/libp

Re: [pulseaudio-discuss] [PATCH] card-restore: Add the ability to save and restore the maximum volume

2012-08-05 Thread rong deng
2012/8/3 poljar (Damir Jelić) : > On Fri, Aug 03, 2012 at 08:25:18PM +0800, rong deng wrote: >> Hi Poljar, >> >> Sorry for top posting. >> >> This patch is based on your github's commit. Of course, it doesn't >> work yet because not everything

Re: [pulseaudio-discuss] [PATCH] card-restore: Add the ability to save and restore the maximum volume

2012-08-03 Thread rong deng
Hi Poljar, Sorry for top posting. This patch is based on your github's commit. Of course, it doesn't work yet because not everything is weaved together... However I'd like to give others a chance to see what i'm doing and whether i'm on the right track. 2012/8/3 Deng Zhengrong : > module-card-re

Re: [pulseaudio-discuss] [PATCH v2] tests: enable to test standalone pulseaudio daemon

2012-07-29 Thread rong deng
Hi Arun, 2012/7/24, Arun Raghavan : > Hi Deng, > > On Mon, 2012-07-23 at 13:12 +0800, Deng Zhengrong wrote: >> With this fix, `check-daemon` doesn't need a system-wide running >> pulseaudio >> anymore. >> >> The method to use is to invoke `make check-daemon` under `src/` and it >> just >> works! :

[pulseaudio-discuss] status report

2012-07-29 Thread rong deng
Hi, This week, I continue the conversion to make the tests program use the new 'check' framework. Now it's almost done, only with one or two exceptions which are not suitable for this framework. This week, I also discussed with pojar to work on the max volume project, I'm reading the related code

Re: [pulseaudio-discuss] I want to know about how to communication one module with another module

2012-07-25 Thread rong deng
Hi Zhen, 2012/7/24 Zhen Fu : > Hi all developer, > > > > I want to know about how to communication one module with another module. Is > it use asyncmg? Thank you! I'm not the expert here, but here's my two cents. :) In theory, two modules should not communicate directly with each other. For a mo

Re: [pulseaudio-discuss] [PATCH] tests: enable to test standalone pulseaudio daemon

2012-07-22 Thread rong deng
2012/7/23 Arun Raghavan : > On Thu, 2012-07-19 at 16:50 +0800, Deng Zhengrong wrote: >> With this fix, `check-daemon` doesn't need a system-wide running pulseaudio >> anymore. >> >> The method to use is to invoke `make check-daemon` under `src/` and it just >> works! :) >> --- >> src/Makefile.am

Re: [pulseaudio-discuss] [PATCH] build: fix Mac OS X configure process

2012-07-22 Thread rong deng
2012/7/23 Arun Raghavan : > On Thu, 2012-07-19 at 16:41 +0800, Deng Zhengrong wrote: >> The original header file doesn't exist on Lion (10.7.4). >> --- >> configure.ac |5 - >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index 0290fa0.

Re: [pulseaudio-discuss] GSOC: bluetooth latency - status report [7/?]

2012-07-20 Thread rong deng
2012/7/21 poljar (Damir Jelić) : > Hi mailing list. > > After the midterm break I've started extending pavucontrol for the GUI > part of my GSOC project. A very bad implementation is on my github page. > > A better version is on the way and should land by the end of the week. > After that I'm plani

[pulseaudio-discuss] weekly status report for logging and testing

2012-07-19 Thread rong deng
Hi list, This week, I've been focusing on the test framework, so a total of 6 patches are sent to the mailing list. Besides, I'm setting things up to let us be able to test standalone daemon. Now the script is done and I've already sent the patch out to the mailing list. So to use this script, sim

Re: [pulseaudio-discuss] [PATCH] core: add more verbose error info

2012-07-17 Thread rong deng
2012/7/17, Arun Raghavan : > On Tue, 2012-07-10 at 08:32 +0800, Deng Zhengrong wrote: >> I'm debugging an issue, and find that it's more useful for users to know >> which >> directory hasn't been created instead of just knowing error code. >> --- > > Pulled, with a minor whitespace addition to sati

[pulseaudio-discuss] GSoC weekly report logging and test framework

2012-07-13 Thread rong deng
Hi list, This week, I've sent several small patches, during the review process, I've also learnt the way to add version number to patches. :) However, these patches are still in review, I'll ask committers to commit these patches later. For the test framework, according to Arun's suggestion, I've

Re: [pulseaudio-discuss] [PATCH] bluetooth: add a parenthesis around pa_streq()

2012-07-11 Thread rong deng
Hi Frederic, I should have CC-ed you for this patch. Please review! Thanks! 2012/7/11 Deng Zhengrong : > I know that pa_streq() is defined with an extra parenthesis, but everytime I > look at it, it makes me nervous. :D > > So it's better to add the parenthesis here. > --- > src/modules/bluetoo

[pulseaudio-discuss] [PATCH] bluetooth: add a parenthesis around pa_streq()

2012-07-11 Thread rong deng
>From 6df0bd0e06c6dd35b66d7ffe03001c0d9e9b8b5d Mon Sep 17 00:00:00 2001 From: Deng Zhengrong Date: Wed, 11 Jul 2012 15:28:34 +0800 Subject: [PATCH] bluetooth: add a parenthesis around pa_streq() I know that pa_streq() is defined with an extra parenthesis, but everytime I look at it, it makes me n

Re: [pulseaudio-discuss] OT: Adapting the subject tag `PATCH` in Git (was: [PATCH] man/pulse-cli-syntax: document option `set-log-target`)

2012-07-10 Thread rong deng
2012/7/10 Paul Menzel : > Am Dienstag, den 10.07.2012, 22:17 +0800 schrieb rong deng: >> 2012/7/10 David Henningsson : >> > On 07/09/2012 07:01 PM, rong deng wrote: >> >> >> >> How to add the nicely "v2" just before the email subject? :) >>

Re: [pulseaudio-discuss] [PATCH] man/pulse-cli-syntax: document option `set-log-target`

2012-07-10 Thread rong deng
2012/7/10 David Henningsson : > On 07/09/2012 07:01 PM, rong deng wrote: >> >> How to add the nicely "v2" just before >> the email subject? :) > > > You edit the patch file with your favorite editor right before you send it > with "git send-email&qu

Re: [pulseaudio-discuss] Wrong time (was: [PATCH] core: add more verbose error info)

2012-07-09 Thread rong deng
Hi, 2012/7/10 Paul Menzel : > Dear Deng, > > > Am Dienstag, den 10.07.2012, 08:32 +0800 schrieb Deng Zhengrong: > > […] > >> Tue, 10 Jul 2012 08:32:54 +0800 (10.07.2012 02:32:54) > > your system time is not set up correctly. Thanks for pointing it out, because I'm dual booting with Linux and Wind

Re: [pulseaudio-discuss] [PATCH] man/pulse-cli-syntax: document option `set-log-target`

2012-07-09 Thread rong deng
The new version is sent. BTW: How to add the nicely "v2" just before the email subject? :) 2012/7/10 Deng Zhengrong : > --- > man/pulse-cli-syntax.5.xml.in | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/man/pulse-cli-syntax.5.xml.in b/man/pulse-cli-syntax.5.xml.in > index 9d3f278..

Re: [pulseaudio-discuss] [PATCH] man: add man documents to set-log-target

2012-07-09 Thread rong deng
Hi Paul, 2012/7/10 Paul Menzel : > Dear Rong, > > > thank you for the patch. > > > Am Montag, den 09.07.2012, 23:52 +0800 schrieb dzrongg: > > Please set up Git with your first and last name. > > $ git config --global user.name "Rong Deng" > &

Re: [pulseaudio-discuss] [PATCH] core: add ring buffer for log

2012-07-09 Thread rong deng
2012/7/9 Peter Meerwald : > >> >> The usage model for this ring buffer is that we can peek into what's >> >> going on >> >> under the hood without restarting pulseaudio daemon, we enable this via >> >> logging >> >> all level's messages. >> > >> > what is the expected memory usage? 512*200 bytes

Re: [pulseaudio-discuss] [PATCH] core: add ring buffer for log

2012-07-09 Thread rong deng
Hi Peter, 2012/7/9 Peter Meerwald : > Hello Deng Zhengrong, > >> The usage model for this ring buffer is that we can peek into what's going on >> under the hood without restarting pulseaudio daemon, we enable this via >> logging >> all level's messages. > > what is the expected memory usage? 512*

[pulseaudio-discuss] GSoC weekly status report logging and testing facilities (Deng Zhengrong)

2012-07-07 Thread rong deng
Hi list, This week, I've crafted the basic test framework and use it to implement a cpu test for intel x86 architecture, I've uploaded this patch to this mailing list. So hopefully I'll translate more and more test cases using this new test framework. I've also helped Tanu to change pa_bool_t to

Re: [pulseaudio-discuss] Transitioning from pa_bool_t to C99 bool

2012-07-05 Thread rong deng
2012/7/5 Arun Raghavan : > On Thu, 2012-07-05 at 15:19 +0800, rong deng wrote: >> 2012/7/2 David Henningsson : >> > I don't think there is any reason to not do s/pa_bool_t/bool/g. Probably >> > the >> > reason for this might be historical, as some c

Re: [pulseaudio-discuss] Transitioning from pa_bool_t to C99 bool

2012-07-05 Thread rong deng
2012/7/2 David Henningsson : > I don't think there is any reason to not do s/pa_bool_t/bool/g. Probably the > reason for this might be historical, as some compilers may have supported > some C99 features but not all of them (Microsoft compilers come to mind). Does pulseaudio work on Windows? If it

Re: [pulseaudio-discuss] change pa_bool_t to bool

2012-07-05 Thread rong deng
2012/7/4 rong deng : > Hi Tanu, > > 2012/7/2 Tanu Kaskinen : >> On Sun, 2012-07-01 at 23:46 +0800, Deng Zhengrong wrote: >>> According to Tanu's suggestion, we should change pa_bool_t to bool. And >>> here >>> are my patches. Now when you do &quo

Re: [pulseaudio-discuss] [PATCH] add a new log target that enables to create new log file if it exists

2012-07-05 Thread rong deng
2012/7/2 David Henningsson : > I installed the new version, added > extra-arguments == --log-target=newfile:/tmp/pulse.log --log-level=debug > > ...and rebooted, and after that there were three files, one for the logged > in user, and two for the lightdm user - one being successful, the other > sta

Re: [pulseaudio-discuss] change pa_bool_t to bool

2012-07-03 Thread rong deng
Hi Tanu, 2012/7/2 Tanu Kaskinen : > On Sun, 2012-07-01 at 23:46 +0800, Deng Zhengrong wrote: >> According to Tanu's suggestion, we should change pa_bool_t to bool. And here >> are my patches. Now when you do "git grep pa_bool_t", here's nothing... >> Please >> help to review. :) > > Thanks a lot

Re: [pulseaudio-discuss] [PATCH] add a new log target that enables to create new log file if it exists

2012-07-01 Thread rong deng
Hi David, Could you help to review this patch? Does this patch solve your feature request about logging? If this patch is OK for you, could you please help to commit this patch for me? :D Thanks! 2012/7/1 Deng Zhengrong : > --- >  src/daemon/cmdline.c     |    4 ++-- >  src/daemon/daemon-conf.c |

Re: [pulseaudio-discuss] [PATCH 1/8] utils: change pa_bool_t to bool

2012-07-01 Thread rong deng
Hi list, According to Tanu's suggestion, I've changed pa_bool_t to bool. Now, when we do "git grep pa_bool_t", here's nothing. :) Please help to review. Thanks! 2012/7/1 Deng Zhengrong : > --- >  src/utils/pacat.c |   10 +- >  src/utils/pacmd.c |    2 +- >  src/utils/pactl.c |   10 +-

[pulseaudio-discuss] about the size of the mail (Was Your message to pulseaudio-discuss awaits moderator approval)

2012-07-01 Thread rong deng
Hi list, Can we make the limit a little bigger? When we do some overall refactoring, we often hit the limit of the patch size... Thanks! 2012/7/1 : > Your mail to 'pulseaudio-discuss' with the subject > >     [PATCH 6/8] modules: change pa_bool_t to bool > > Is being held until the list moderat

[pulseaudio-discuss] GSoC: status report: logging and testing facilities

2012-06-29 Thread rong deng
Hi list, This week, I've been investigating existing test frameworks such as check, cmockery, gtests etc. Some of quite powerful, but they're implemented in C++ language, some are simple enough for our projects. However, after some investigation, I decided to write our own test framework, which wo

Re: [pulseaudio-discuss] Logging startup of the first instance...

2012-06-28 Thread rong deng
2012/6/29 David Henningsson : > On 06/29/2012 07:48 AM, rong deng wrote: >> >> Hi David, >> >> Sorry for the top posting, I've added the O_EXCL flag and please >> review again. Thanks! > > > I meant that if opening with O_EXCL fails, you should redo

Re: [pulseaudio-discuss] Logging startup of the first instance...

2012-06-28 Thread rong deng
Hi David, Sorry for the top posting, I've added the O_EXCL flag and please review again. Thanks! 2012/6/28 rong deng : > 2012/6/28 David Henningsson : >> On 06/28/2012 09:41 AM, rong deng wrote: >>> >>> Hi David, >>> >>> I've come up

Re: [pulseaudio-discuss] [PATCH] utils: add help info for 'set-log-target'

2012-06-28 Thread rong deng
2012/6/28 Arun Raghavan : > On Sat, 2012-06-23 at 19:15 +0800, Deng Zhengrong wrote: >> --- >>  src/utils/pacmd.c |    1 + >>  1 files changed, 1 insertions(+), 0 deletions(-) > > Thanks. Pulled this and the gcov patch. Could you update the pacmd man > page as well? Sure. I'll send another patch a

Re: [pulseaudio-discuss] Logging startup of the first instance...

2012-06-28 Thread rong deng
2012/6/28 David Henningsson : > On 06/28/2012 09:41 AM, rong deng wrote: >> >> Hi David, >> >> I've come up with an idea to accommodate this situation, tell me if >> it's useful for you or there's a better solution. :) > > > I think this s

Re: [pulseaudio-discuss] Logging startup of the first instance...

2012-06-28 Thread rong deng
reate the PATH.5 as our log file target. Any comments are welcome. The patch is attached. 2012/6/11 David Henningsson : > On 06/11/2012 02:35 AM, rong deng wrote: >> >> 2012/6/4 David Henningsson: >>> >>> Sometimes I have users who complain about some problem,

Re: [pulseaudio-discuss] [PATCH] utils: add help info for 'set-log-target'

2012-06-26 Thread rong deng
Hi Arun, Please help review this patch, I forgot to add the help message to this set-log-target command in my last patch. Thanks! 2012/6/23 Deng Zhengrong : > --- >  src/utils/pacmd.c |    1 + >  1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/src/utils/pacmd.c b/src/utils/pacm

Re: [pulseaudio-discuss] status report from Deng Zhengrong Jun 15th

2012-06-22 Thread rong deng
2012/6/21 David Henningsson : > On 06/15/2012 04:24 PM, rong deng wrote: >> >> 2. write a first version of ring buffer log support. Now we have a per >> thread log pool. So therefore different threads would not be blocked >> for a long time. However, as in the cur

[pulseaudio-discuss] status report from Deng Zhengrong June 22

2012-06-22 Thread rong deng
Hi list, I'm a bit busy with my school project this week. However, I've been starting to look at the test framework. There's no unified test framework in pulse audio code right now. As most of its code are written in a casual way and some of the tests in tests/ directory are only meant to be read

[pulseaudio-discuss] status report from Deng Zhengrong Jun 15th

2012-06-15 Thread rong deng
Hi list, This week, I've mainly done two things: 1. integrate the coverage test tool 'gcov' into our pulseaudio compiling system and make it work. With my hack into Makefile and configure.ac, if we use './configure --enable-gcov' and then make and make install, the pulseaudio program would create

Re: [pulseaudio-discuss] Logging startup of the first instance...

2012-06-10 Thread rong deng
2012/6/4 David Henningsson : > Sometimes I have users who complain about some problem, but after a > "pulseaudio -k" the problem is gone. Therefore asking them to restart > pulseaudio with - switch will not give any information about the error. > >  * I could log to syslog (the default), but on

[pulseaudio-discuss] gsoc 2012 status report about logging and testing

2012-06-08 Thread rong deng
Hi list, This week I've finished the implementation of log category support. The core part incorporate changes in log.c and log.h, but the most part of this patch is to add the category definition to each c file. The interface is quite simple, we just need to add one line similar to this "#define

Re: [pulseaudio-discuss] [PATCH] fix the wrong parameter sequence in pax11publish

2012-06-08 Thread rong deng
2012/6/8 Arun Raghavan > On Tue, 2012-06-05 at 20:30 +0800, rong deng wrote: > > Hi Arun, > > > > Could you help review this patch? > > Done, thanks. BTW, please avoid top-posting and HTML email (GMail allows > you to compose plain-text messages as well). > &

Re: [pulseaudio-discuss] [PATCH 1/2] add log category

2012-06-07 Thread rong deng
diff --git a/src/daemon/caps.c b/src/daemon/caps.c index 3759388..49a0690 100644 --- a/src/daemon/caps.c +++ b/src/daemon/caps.c @@ -20,6 +20,8 @@ USA. ***/ +#define PA_LOG_CATEGORY_DEFAULT "daemon" + The above #define is duplicated below in various files.

[pulseaudio-discuss] fix wrong parameter sequence

2012-06-05 Thread rong deng
Please review. Thanks! 0001-fix-the-wrong-parameter-sequence-in-pax11publish.patch Description: Binary data ___ pulseaudio-discuss mailing list pulseaudio-discuss@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Re: [pulseaudio-discuss] [PATCH] fix the wrong parameter sequence in pax11publish

2012-06-05 Thread rong deng
Hi Arun, Could you help review this patch? 2012/6/3 Deng Zhengrong > --- > src/utils/pax11publish.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/utils/pax11publish.c b/src/utils/pax11publish.c > index 0f9a030..16cf866 100644 > --- a/src/utils/pax11publish

[pulseaudio-discuss] status report of gsoc

2012-06-01 Thread rong deng
Hi PulseAudio community, This is Deng Zhengrong, a member of this year's GSoC. I'll send the status report every Friday, as this is my first status report, I'll write about what I've done till now. I've mainly focused on enhancement of logging facilities lately. One part is about adding a categor

Re: [pulseaudio-discuss] [PATCH] add set-log-target command for pacmd

2012-05-28 Thread rong deng
Hi, Can someone review this? Is it OK to be accepted? 2012/5/25 rong deng > > 2012/5/25 Dalleau, Frederic > > Hi Rong Deng, >> >> >> >> This helps to change log target on the fly. >>> >>> >>> >> You don't

Re: [pulseaudio-discuss] [PATCH] add set-log-target command for pacmd

2012-05-25 Thread rong deng
2012/5/25 Dalleau, Frederic > Hi Rong Deng, > > > > This helps to change log target on the fly. >> >> >> > You don't need to copy and truncate file_path as a truncated file path is > useless ! > Instead you can use directly m+5. If the path is too

[pulseaudio-discuss] [PATCH] add set-log-target command for pacmd

2012-05-24 Thread rong deng
Please review! This helps to change log target on the fly. 0001-add-set-log-target-command-for-pacmd.patch Description: Binary data ___ pulseaudio-discuss mailing list pulseaudio-discuss@lists.freedesktop.org http://lists.freedesktop.org/mailman/listin

[pulseaudio-discuss] [PATCH] use pa_streq instead of plain strcmp in daemon-conf.c

2012-05-24 Thread rong deng
Please review and hopefully got accepted. :) I did a simple grep and saw there are lots of other usages of strcmp, but let's translate it bit by bit... --deng 0001-use-pa_streq-instead-of-plain-strcmp.patch Description: Binary data ___ pulseaudio-disc

Re: [pulseaudio-discuss] About My Blog Space

2012-05-24 Thread rong deng
2012/5/22 Colin Guthrie > If you would like a little face icon to go with your posts (e.g. see > what Arun's posts look like) please email me a suitable png file. Size > isn't too important but about 80x100px is probably about right. > > OK. I'll send you one if I find a suitable one. > Also, I

[pulseaudio-discuss] About My Blog Space

2012-05-21 Thread rong deng
Hi Col, According to your advice, I'm setting my blog up, to track my status and some thoughts during this GSoC project. The URL is http://zhengrong.wordpress.com I'll make everything related to PulseAudio into the category "PulseAudio", so hopefully it would only output the correct RSS. There's

[pulseaudio-discuss] about log level configuration

2012-05-16 Thread rong deng
Hi list, I'm implementing 'category' feature in pulseaudio, the 'category' feature mimics the feature in gstreamer, i.e. we can use something like 'ENV_LOG_DEBUG=loopback:5 pulseaudio' to setup the level of module loopback to 5. During my implementation, I found there's already an environment var

[pulseaudio-discuss] My Blog URL for GSoC 2012

2012-05-01 Thread rong deng
Hi Colin, I've setup one blog space here: http://pulseaudio.blog.com and later I'll use this blog to publish my status report and my current work. Please help add this url into pulseaudio's planet. As long as it's added to pulseaudio's planet, the blog's content would automatically be sent to thi

[pulseaudio-discuss] thank you letter

2012-04-24 Thread rong deng
Hi PulseAudio community, It's my first close contact with this community and it's quite positive, it's a great honor for me to join this year's Google Summer of Code. I'll start coding these days. To keep you informed of my status, I would write a biweekly (if it's too long a time, I can write we

[pulseaudio-discuss] Fwd: [PATCH] proof of concept: use pactl log to get the buffer log

2012-04-02 Thread rong deng
I clicked "Reply" instead of "Reply to All", So I forward my reply mail to mailing list again. Sorry for the noises. -- 已转发邮件 -- 发件人: rong deng 日期: 2012年4月3日 上午10:41 主题: Re: [pulseaudio-discuss] [PATCH] proof of concept: use pactl log to get the buffer lo

Re: [pulseaudio-discuss] [PATCH] use CLOCK_MONOTONIC_COARSE to reduce the system call

2012-03-31 Thread rong deng
在 2012年3月30日 下午12:48,Arun Raghavan 写道: > Hi, > > On Thu, 2012-03-29 at 21:31 +0800, Deng Zhenrong wrote: > > According to the description below: > > > > http://lkml.org/lkml/2009/7/17/258 > > > > There are two benefits: > > a) it doesn't need to access the hardware. > > b) avoid the syscal

[pulseaudio-discuss] more thoughts about my GSoC plan

2012-03-25 Thread rong deng
Hi all, I would like to apply for logging/testing project. Here's my attempt to write more about my thoughts on how this should be done: First about logging: 1. pulseaudio has already had some support for log level, but it may need some improvements. 2. the category support is not yet. for this

Re: [pulseaudio-discuss] express my interests in applying GSoC 2012

2012-03-22 Thread rong deng
Thanks all for replying and giving suggestions. I've sent two patches in another thread, and I've already setup git send-email, cool! It's way easy to send patches on command lines. Personally, I'd like to use PRI-prefix, because it looks more standard. But casting and stick it with %llx should be

Re: [pulseaudio-discuss] express my interests in applying GSoC 2012

2012-03-21 Thread rong deng
在 2012年3月21日 下午3:21,David Henningsson 写道: > On 03/20/2012 03:48 PM, rong deng wrote: > >> Hi all, >> >> This is Deng Zhenrong from Shanghai China, I've already used Linux as my >> main desktop for several years. Now I would like to express my interests >&

[pulseaudio-discuss] express my interests in applying GSoC 2012

2012-03-20 Thread rong deng
Hi all, This is Deng Zhenrong from Shanghai China, I've already used Linux as my main desktop for several years. Now I would like to express my interests in applying pulseaudio's projects for this year's GSoC in this mail. Glancing through the ideas, I find I'm interested in several few already.