[Orgmode] Re: [PATCH] org: rework property set

2010-12-13 Thread Bernt Hansen
Thanks Julien, This patch format is better. Could you put the comments like 'I may have done this badly, ...' and 'It works but there maybe some corners case ...' after the --- and before the diffstat? This isn't really useful to keep permanently in the org-mode history if this patch is accepted

[Orgmode] Re: [PATCH] org: rework property set

2010-12-14 Thread Julien Danjou
On Mon, Dec 13 2010, Bernt Hansen wrote: > This patch format is better. Could you put the comments like 'I may > have done this badly, ...' and 'It works but there maybe some corners > case ...' after the --- and before the diffstat? Sure, this is how I did usually, but last time I did that IIRC

[Orgmode] Re: [PATCH] org: rework property set

2010-12-14 Thread Bernt Hansen
Julien Danjou writes: > On Tue, Dec 14 2010, Giovanni Ridolfi wrote: > >> I think that you can put some extra, useful, information, that can go >> in the Changelog, as lines not beginning with "*". >> >> But if you write very unformal/personal comments like: >> 'I may have done this badly, ...

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-14 Thread Giovanni Ridolfi
Julien Danjou writes: > On Mon, Dec 13 2010, Bernt Hansen wrote: > >> This patch format is better. Could you put the comments like 'I may >> have done this badly, ...' and 'It works but there maybe some corners >> case ...' after the --- and before the diffstat? > > Sure, this is how I did usual

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-14 Thread Julien Danjou
On Tue, Dec 14 2010, Giovanni Ridolfi wrote: > I think that you can put some extra, useful, information, that can go > in the Changelog, as lines not beginning with "*". > > But if you write very unformal/personal comments like: > 'I may have done this badly, ...' you should write after the ---

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-14 Thread Nick Dokos
Julien Danjou wrote: > On Tue, Dec 14 2010, Giovanni Ridolfi wrote: > > > I think that you can put some extra, useful, information, that can go > > in the Changelog, as lines not beginning with "*". > > > > But if you write very unformal/personal comments like: > > 'I may have done this badly

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-16 Thread Carsten Dominik
Hi Julien, are you going to resend the patch? I do like the functionality! Or did you already resend it? Maybe I missed it. Unfortunately I am loosing my overview here. Cheers - Carsten On Dec 14, 2010, at 11:30 AM, Julien Danjou wrote: On Tue, Dec 14 2010, Giovanni Ridolfi wrote: I thi

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-16 Thread Julien Danjou
On Thu, Dec 16 2010, Carsten Dominik wrote: > are you going to resend the patch? I do like the functionality! > Or did you already resend it? Maybe I missed it. Unfortunately > I am loosing my overview here. If the code seems fine to you, I can resend the patch with a clean commit message, for

Re: [Orgmode] Re: [PATCH] org: rework property set

2010-12-16 Thread Carsten Dominik
Please do, thank you. - Carsten On Dec 16, 2010, at 2:45 PM, Julien Danjou wrote: On Thu, Dec 16 2010, Carsten Dominik wrote: are you going to resend the patch? I do like the functionality! Or did you already resend it? Maybe I missed it. Unfortunately I am loosing my overview here. If