[PATCH] Improve the "set log functionality"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/20/2013 10:28 AM, Shuai Fan wrote:
> This patch is my first attempt to Improve the "set log functionality".
>
> Any suggestions?

Hi Shuai!

I'm not sure what kind of suggestions you're looking for, but first 
suggestion is to use "git send-email" to send your patches instead of 
attaching them, because that makes commenting easier.

Also, have you actually tested the code? It looks like it would set the 
log target for the pactl client, not the server which pactl connects to, 
which is quite unexpected.

Also, did you copy-paste pa_log_parse_target from somewhere else? If so 
you should make the old code use your new function.

>
>
> Best wishes,
>      Shuai
>
> 0001-Improve-set-log-target-functionality.patch
>


-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux