[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [patch] org-delete-indentation
From: |
Rasmus |
Subject: |
Re: [O] [patch] org-delete-indentation |
Date: |
Thu, 21 May 2015 00:48:29 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
Rasmus <address@hidden> writes:
> Nicolas Goaziou <address@hidden> writes:
>
>> Also, shouldn't the final (delete-indentation ARG) be in the "else" part
>> of the `if'?
>
> It was, at least on my disk. I did not check the patch.
>
>>> +(ert-deftest test-org-delete-indentation ()
>>> + "Test M-^ (`org-delete-indentation') specification."
>>
>> I suggest to omit binding in the description. That's one thing less we
>> have to keep up-to-date if it ever changes.
>
> OK.
>
> The attached patch is updated and seems pretty good. It also handles tags
> and entities according to org-auto-align-tags.
Pushed.
> The second patch (0003) adds support for tables in the sense that
Not pushed. I'm still uncertain if this makes sense unless a some key
enables breaking table cells.
Rasmus
--
9000!