[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] sequence type corrections and enhancements
From: |
Felix |
Subject: |
Re: [Chicken-hackers] [PATCH] sequence type corrections and enhancements |
Date: |
Wed, 14 Sep 2011 03:52:58 -0400 (EDT) |
From: Peter Bex <address@hidden>
Subject: Re: [Chicken-hackers] [PATCH] sequence type corrections and
enhancements
Date: Wed, 14 Sep 2011 09:27:10 +0200
> On Wed, Sep 14, 2011 at 12:51:55AM +0200, Felix wrote:
>> >
>> > I think you overlooked these entries:
>> >
>> > - reverse: argument and result
>> > - member, assv, assoc: specialization types
>> >
>> > Also, member has a (forall a) in its declaration but doesn't use it,
>> > which looks wrong to me.
>> >
>> > Further, "set-groups!" from POSIX is declared to accept a generic list.
>> > This could probably be improved to (list-of (or fixnum number)), I think.
>> >
>>
>> Thanks for pointing this out. If you are willing to sign off and
>> apply this patch, I'd like to fix that in a different one.
>
> I have no idea how this works. Just saying "I'm okay with it" and
> merging it in from your branch?
For example. You can also use "git commit --amend" to modify the commit
message of the current branch tip after you merged or cherry-picked.
Well, that is what I think it does.
cheers,
felix