From: Rene Herman <rene.herman@keyaccess.nl>
To: "John L. Utz III" <john.utz@dmx.com>
Cc: Takashi Iwai <tiwai@suse.de>,
ALSA Developers <alsa-devel@alsa-project.org>
Subject: Re: PATCH - vt1618 7.1 Audio
Date: Wed, 20 Aug 2008 21:54:23 +0200 [thread overview]
Message-ID: <48AC766F.1010608@keyaccess.nl> (raw)
In-Reply-To: <op.uf685pnb647oa9@jutz-gnto64>
On 20-08-08 21:16, John L. Utz III wrote:
> anyway, we can argue about it again after i get a checkpatch
> compliant patch to submit....
Let's get a headstart...
Note that Hungarian is not a private Takashi Iwai or ALSA dislike; it's
rather specifically against the Linux kernel coding style (although the
actual CodingStyle document only mentions Hungarian for function names).
Most contributors have had to adjust a few local habits and in the end,
it's generally worth it. If all is good and well, open source code is
read many more times than it's written after all and any (significant)
style inconsistency costs every reader just a little mental hickup.
Review resources are one of the more scarcely available ones in this
development process, so a consistent style definitely pays.
Sure, sure, I also try ty get away with things, but Hungarians hickup
rather loudly.
Rene.
next prev parent reply other threads:[~2008-08-20 19:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-20 0:17 PATCH - vt1618 7.1 Audio John L. Utz III
2008-08-20 8:28 ` Takashi Iwai
2008-08-20 17:50 ` John L. Utz III
2008-08-20 19:05 ` Rene Herman
2008-08-20 19:16 ` John L. Utz III
2008-08-20 19:54 ` Rene Herman [this message]
2008-08-20 20:50 ` John L. Utz III
2008-08-20 21:00 ` Rene Herman
2008-08-20 19:21 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48AC766F.1010608@keyaccess.nl \
--to=rene.herman@keyaccess.nl \
--cc=alsa-devel@alsa-project.org \
--cc=john.utz@dmx.com \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.