From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bastien <bzg@gnu.org>
Subject: Re: [patch] a small change
Date: Tue, 01 May 2012 11:55:37 +0200
Message-ID: <87ehr4w7hi.fsf@gnu.org>
References: <CAJpRBmdME5kxc+_DhbhWDfHEWJAJ2HPdMwT2AAYGofBiDRz9eA@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([208.118.235.92]:45775)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bastienguerry@googlemail.com>) id 1SP9n6-0007Dv-IW
	for emacs-orgmode@gnu.org; Tue, 01 May 2012 05:54:41 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <bastienguerry@googlemail.com>) id 1SP9n4-0004Bb-V5
	for emacs-orgmode@gnu.org; Tue, 01 May 2012 05:54:40 -0400
Received: from mail-wg0-f49.google.com ([74.125.82.49]:43843)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bastienguerry@googlemail.com>) id 1SP9n4-0004AV-M8
	for emacs-orgmode@gnu.org; Tue, 01 May 2012 05:54:38 -0400
Received: by wgbds1 with SMTP id ds1so2699493wgb.30
	for <emacs-orgmode@gnu.org>; Tue, 01 May 2012 02:54:35 -0700 (PDT)
In-Reply-To: <CAJpRBmdME5kxc+_DhbhWDfHEWJAJ2HPdMwT2AAYGofBiDRz9eA@mail.gmail.com>
	(feng shu's message of "Tue, 1 May 2012 17:44:24 +0800")
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
To: feng shu <tumashu@gmail.com>
Cc: emacs-orgmode@gnu.org

Hi Feng,

thanks for the patch.  Can you give it a better name?

Also split it into two patches, one for the typo one for 
the other change.

As for the code:

- don't use 'nil, use nil
- (append (list 'x 'y) 'y) => (list 'x 'y 'z)

Thanks!

-- 
 Bastien