Re: [PATCH 3/5] emacs: Allow the indentation of content to be toggled.
Hi I have reviewed the patch series within the limits of my lisp knowledge. Patch 1: is rather beyond my lisp so I won't comment on that (and I have never tried crypto): Patches 2-5 look fine with one bikeshed and one query for my own understanding. The bikeshed is that I agree with Jani in id:CAB+hUn8KhXHTRCdrLe0cT=8mdtz9ZntPFf13mq0iCo4CX=b...@mail.gmail.com that I would prefer to reserve '' for saving a file. The query: in + (props (notmuch-show-get-message-properties)) + (indenting notmuch-show-indent-content)) (with-temp-buffer (insert all) - (indent-rigidly (point-min) (point-max) (- depth)) + (if indenting + (indent-rigidly (point-min) (point-max) (- depth))) Is the local variable `indenting' needed because notmuch-show-indent-content is buffer local and this is in the with-temp-buffer? Thanks Mark ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 3/5] emacs: Allow the indentation of content to be toggled.
On Tue, 31 Jan 2012 16:15:11 +, Mark Walters markwalters1...@gmail.com wrote: Hi I have reviewed the patch series within the limits of my lisp knowledge. Patch 1: is rather beyond my lisp so I won't comment on that (and I have never tried crypto): Patches 2-5 look fine with one bikeshed and one query for my own understanding. Thanks. The bikeshed is that I agree with Jani in id:CAB+hUn8KhXHTRCdrLe0cT=8mdtz9ZntPFf13mq0iCo4CX=b...@mail.gmail.com that I would prefer to reserve '' for saving a file. '' is 'remove indent', or 'look to the left'. '' is 'toggle truncation', or 'look to the right'. (I'm not overly worried about this - would 't' be more acceptable?) The query: in +(props (notmuch-show-get-message-properties)) +(indenting notmuch-show-indent-content)) (with-temp-buffer (insert all) - (indent-rigidly (point-min) (point-max) (- depth)) + (if indenting + (indent-rigidly (point-min) (point-max) (- depth))) Is the local variable `indenting' needed because notmuch-show-indent-content is buffer local and this is in the with-temp-buffer? Yes. pgp5qaBUtv6Ol.pgp Description: PGP signature ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 3/5] emacs: Allow the indentation of content to be toggled.
On Tue, 31 Jan 2012 16:23:32 +, David Edmondson d...@dme.org wrote: On Tue, 31 Jan 2012 16:15:11 +, Mark Walters markwalters1...@gmail.com wrote: Hi I have reviewed the patch series within the limits of my lisp knowledge. Patch 1: is rather beyond my lisp so I won't comment on that (and I have never tried crypto): Patches 2-5 look fine with one bikeshed and one query for my own understanding. Thanks. The bikeshed is that I agree with Jani in id:CAB+hUn8KhXHTRCdrLe0cT=8mdtz9ZntPFf13mq0iCo4CX=b...@mail.gmail.com that I would prefer to reserve '' for saving a file. '' is 'remove indent', or 'look to the left'. '' is 'toggle truncation', or 'look to the right'. (I'm not overly worried about this - would 't' be more acceptable?) I personally would prefer 't' but it is easy to remap so it really isn't important. Best wishes Mark ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch