From: Marcin Slusarz <marcin.slusarz@gmail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: hch@infradead.org, linux-kernel@vger.kernel.org,
bfennema@falcon.csc.calpoly.edu, jack@suse.cz
Subject: Re: [PATCH] udf: convert some macros to functions
Date: Tue, 8 Jan 2008 07:52:43 +0100 [thread overview]
Message-ID: <20080108065239.GA23875@joi> (raw)
In-Reply-To: <20080107152815.acab7678.akpm@linux-foundation.org>
On Mon, Jan 07, 2008 at 03:28:15PM -0800, Andrew Morton wrote:
> On Mon, 7 Jan 2008 21:44:26 +0100
> Marcin Slusarz <marcin.slusarz@gmail.com> wrote:
>
> > > There's some overly long lines here and some odd style, this should look
> > > more like:
> > These long lines were split later in "[PATCH 1/7] udf: fix coding style"
>
> Confused. How can patch 1/n come "later" than this one?
>
> Your patches actually came through as "[patch] udf: ...". ie: no
> sequence numbering.
Because there are 2 patchsets. First in third version and second in first version.
I think I join them to not confuse anyone...
> > (but I will fix it in next version of this patch).
>
> Please fix the sequence numbering too, thanks.
Yeah. I forgot to do this before submission.
Marcin
next prev parent reply other threads:[~2008-01-08 6:51 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-06 0:44 [PATCH 0/6] udf: improve code related to super_block v3 marcin.slusarz
2008-01-06 0:44 ` [PATCH] udf: fix coding style of super.c marcin.slusarz
2008-01-07 13:33 ` Jan Kara
2008-01-06 0:44 ` [PATCH] udf: remove some ugly macros marcin.slusarz
2008-01-06 0:44 ` [PATCH] udf: convert UDF_SB_ALLOC_PARTMAPS macro to udf_sb_alloc_partition_maps function marcin.slusarz
2008-01-06 0:44 ` [PATCH] udf: check if udf_load_logicalvol failed marcin.slusarz
2008-01-07 12:19 ` Christoph Hellwig
2008-01-06 0:44 ` [PATCH] udf: convert some macros to functions marcin.slusarz
2008-01-07 12:26 ` Christoph Hellwig
2008-01-07 20:44 ` Marcin Slusarz
2008-01-07 23:28 ` Andrew Morton
2008-01-08 6:52 ` Marcin Slusarz [this message]
2008-01-08 10:09 ` Jan Kara
2008-01-06 0:44 ` [PATCH] udf: fix sparse warnings (shadowing & mismatch between declaration and definition) marcin.slusarz
2008-01-07 12:26 ` Christoph Hellwig
2008-01-06 0:52 ` [PATCH 0/6] udf: improve code related to super_block v3 Marcin Slusarz
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=20080108065239.GA23875@joi \
--to=marcin.slusarz@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=bfennema@falcon.csc.calpoly.edu \
--cc=hch@infradead.org \
--cc=jack@suse.cz \
--cc=linux-kernel@vger.kernel.org \
/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.