[alsa-devel] [PATCH 1/2] ALSA: soundcore: coding style fixes
Takashi Iwai
tiwai at suse.de
Thu Jun 5 16:12:37 CEST 2014
At Thu, 05 Jun 2014 15:59:03 +0200,
Daniel Mack wrote:
>
> On 06/05/2014 03:53 PM, Takashi Iwai wrote:
> > At Thu, 5 Jun 2014 14:12:25 +0200,
> > Daniel Mack wrote:
> >>
> >> sound/sounc_core.c has a larger number of style issues. Fix them to make the
> >> code more readable. No functional change.
> >>
> >> Signed-off-by: Daniel Mack <zonque at gmail.com>
> >
> > Thanks for the patches, but are you going to fix anything in these
> > codes along with them? Such coding style fixes just stir git history,
> > which make often annoying for backporting or reviewing. So, basically
> > I'd take such cleanup patches only if the real fixes will come
> > together.
>
> Yeah, I know that code is legacy. I was just skimming over the sources
> to trace a bug report on alsa-user, and stumbled over so many style
> issues that made reading the code harder than necessary that I couldn't
> resist fixing them :)
Ah, you're looking at usb disconnection issue? I just saw the thread
now...
> Hence, I don't know yet. Maybe there will be a fix as well at some late
> point. Feel free to ignore the patches for now. I'll stash and resend
> them along with real fixes in case there are any.
OK, then let's postpone.
thanks,
Takashi
More information about the Alsa-devel
mailing list