Re: [PATCH] us428control support for mixxx

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

 



yes I will change the indent, and resend you the patch. Thanks for your code review.

On 10/4/06, Takashi Iwai <tiwai@xxxxxxx > wrote:
At Sat, 30 Sep 2006 18:06:53 +0200,
CTAF ctaf wrote:
>
> This patch allow to change the mapping of the us428, the new mapping is used with
> mixxx.
> To use the new mapping:
> ./us428control -m mixxx
>
> I already posted an older version on the bug tracker, but I had'nt have response for
> it.
> thanks.
>
> Signed-off-by: Cedric GESTES < goctaf@xxxxxxxxx>

Thanks.  Through a quick look, the changes seem OK to me.
But, could you fix the indent before merging to the tree?
Karsten's original code uses 8-char tabs for indentation while you're
using 2-chars.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux