Re: [PATCH] sun-cs4231: code improvements (2nd version)

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

 



At Mon, 3 Sep 2007 18:41:44 +0200,
Krzysztof Helt wrote:
> 
> From: Krzysztof Helt <krzysztof.h1@xxxxx>
> 
> This patch does some code improvements to make
> driver (both code and binary) shorter.
> It also make use of card->private_data pointer to
> store chip information.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1@xxxxx>

Thanks, now applied to HG tree.


> ---
> 
> Do I understand correctly, that lines:
> 
> if ((err = function()) < 0) 
> 
> are passe now?

Yes.

> They are so abudant in the alsa code, that I thought it is some kind of template code.

Yep, it was in an example code.  And we have still many of them.
But such a clean up is a low-priority task, so I haven't done much
yet.  Still, it's better not to have such lines in a new patch, at
least.


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/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