From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752258AbaKCORO (ORCPT ); Mon, 3 Nov 2014 09:17:14 -0500 Received: from cantor2.suse.de ([195.135.220.15]:52437 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751506AbaKCORL (ORCPT ); Mon, 3 Nov 2014 09:17:11 -0500 Date: Mon, 03 Nov 2014 15:17:08 +0100 Message-ID: From: Takashi Iwai To: SF Markus Elfring Cc: Jaroslav Kysela , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, trivial@kernel.org, Coccinelle Subject: Re: [PATCH resent] ALSA: emu10k1: Deletion of unnecessary checks before three function calls In-Reply-To: <54578CE0.1090201@users.sourceforge.net> References: <5307CAA2.8060406@users.sourceforge.net> <530A086E.8010901@users.sourceforge.net> <530A72AA.3000601@users.sourceforge.net> <530B5FB6.6010207@users.sourceforge.net> <530C5E18.1020800@users.sourceforge.net> <530CD2C4.4050903@users.sourceforge.net> <530CF8FF.8080600@users.sourceforge.net> <530DD06F.4090703@users.sourceforge.net> <5317A59D.4@users.sourceforge.net> <54567788.9060008@users.sourceforge.net> <54578CE0.1090201@users.sourceforge.net> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Mon, 03 Nov 2014 15:10:40 +0100, SF Markus Elfring wrote: > > > Your patch can't be applied cleanly due to your MUA breaking the > > lines. Please fix your MUA setup, or use an attachment if it's > > impossible, and resend the patch. > > Thanks for your feedback. > > Does this example show a conflict between long comments like > "snd_emu10k1_ctl_private_free( ... *kctl)" after patch ranges and line length > limitation for email eventually? Conflict? It's your MUA that is broken. > > Also, try to align the subject line with the relevant commits. See > > "git log sound/pci/emu10k1" > > I have attached my update suggestion with a slightly different commit title as > before. Is this variant acceptable? So, you couldn't fix your MUA? That's bad for you. Many maintainers dislike attachments and won't accept such patches. Couldn't you simply send a patch via git-send-email? In anyway, I applied the patch now. Thanks. Takashi