public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <rpurdie@openedhand.com>
To: Nitin Gupta <nitingupta910@gmail.com>
Cc: akpm <akpm@linux-foundation.org>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH -mm] Add LZO1X compression support to the kernel
Date: Thu, 07 Jun 2007 09:27:36 +0100	[thread overview]
Message-ID: <1181204856.6086.13.camel@localhost.localdomain> (raw)
In-Reply-To: <4cefeab80706062141u5fea846ane51c90dc89732db5@mail.gmail.com>

On Thu, 2007-06-07 at 10:11 +0530, Nitin Gupta wrote:
> Your code now looks nice and clean. But I don't know what you want. I
> already spent lot of time on version 7 I posted and contains all those
> corrections which were suggested for my earlier version. I cannot ask
> you to look into possible problems (if any) in my code now since  you
> are not interested in that anyway. So, please continue this
> duplication.

What I (and others) repeatedly asked for was an explanation of why there
were the differences between the bytecode from your version and the
bytecode from mine. This was never explained by you and I was left to do
this myself. Sadly, doing so meant some duplication.

Anyhow, I think I can now explain the differences. Now we have two
versions, its a question of how to proceed. Basically one needs to be
merged into the other. I will try and compare the two patches and see
what differences we have.

Regards,

Richard



  reply	other threads:[~2007-06-07  8:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-06 17:26 [PATCH -mm] Add LZO1X compression support to the kernel Richard Purdie
2007-06-07  4:41 ` Nitin Gupta
2007-06-07  8:27   ` Richard Purdie [this message]
2007-06-07  8:53     ` Nitin Gupta
2007-06-07  6:06 ` Paul Mundt

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=1181204856.6086.13.camel@localhost.localdomain \
    --to=rpurdie@openedhand.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nitingupta910@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox