[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Move ambitus print callback to scheme
|
From: |
Carl Sorensen |
|
Subject: |
Re: [PATCH] Move ambitus print callback to scheme |
|
Date: |
Wed, 26 Aug 2009 14:43:34 -0600 |
On 8/25/09 2:28 PM, "Neil Puttock" <address@hidden> wrote:
> Hi,
>
> I've just posted a revised patchset which deals with Han-Wen's comments.
>
> http://codereview.appspot.com/110047/show
The patch looks good to me.
I have a question about style, though.
You code the empty list as (list). I typically code the empty list as '().
It there a preference? I suspect that we ought to be consistent, although
it's not highly important. It could be part of the code janitor work,
though.
Thanks,
Carl
- [PATCH] Move ambitus print callback to scheme, Neil Puttock, 2009/08/19
- Re: [PATCH] Move ambitus print callback to scheme, Neil Puttock, 2009/08/25
- Re: [PATCH] Move ambitus print callback to scheme,
Carl Sorensen <=
- Re: [PATCH] Move ambitus print callback to scheme, Han-Wen Nienhuys, 2009/08/27
- Re: [PATCH] Move ambitus print callback to scheme, Neil Puttock, 2009/08/27
- Re: [PATCH] Move ambitus print callback to scheme, Nicolas Sceaux, 2009/08/28
- Re: [PATCH] Move ambitus print callback to scheme, Carl Sorensen, 2009/08/28
- Re: [PATCH] Move ambitus print callback to scheme, David Kastrup, 2009/08/29
- Re: [PATCH] Move ambitus print callback to scheme, Nicolas Sceaux, 2009/08/29
- Re: [PATCH] Move ambitus print callback to scheme, Carl Sorensen, 2009/08/29
- Re: [PATCH] Move ambitus print callback to scheme, David Kastrup, 2009/08/30