Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-25 Thread Ihor Radchenko
Uwe Brauer writes: "KLG" == Kévin Le Gouguec writes: > >> Ihor Radchenko writes: >>> The tentative fix is attached, but please >>> double check because I am not very familiar with the indentation code. > >> I've applied your fix on top of 281a0e26b; AFAICT it works as expected. > > I

Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-25 Thread Uwe Brauer
>>> "KLG" == Kévin Le Gouguec writes: > Uwe Brauer writes: > "KLG" == Kévin Le Gouguec writes: >> >>> I've applied your fix on top of 281a0e26b; AFAICT it works as expected. >> >> I applied the patch on top of e2fa3c4c4046b6, > (Me too actually; 281a0e26b is the commit ID I got by

Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-24 Thread Ihor Radchenko
Kévin Le Gouguec writes: > Mmm, starting from emacs -Q and running M-x org-indent-mode, I see Org > indent the :PROPERTIES: drawers in both files, both with the tip of > Org's 'main' branch (e2fa3c4) and with Ihor's patch applied. Same on my side. org-indent-mode works fine for me. Best, Ihor

Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-24 Thread Kévin Le Gouguec
Uwe Brauer writes: "KLG" == Kévin Le Gouguec writes: > >> I've applied your fix on top of 281a0e26b; AFAICT it works as expected. > > I applied the patch on top of e2fa3c4c4046b6, (Me too actually; 281a0e26b is the commit ID I got by applying Ihor's patch; apologies for the confusion) >

Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-24 Thread Uwe Brauer
>>> "KLG" == Kévin Le Gouguec writes: > Ihor Radchenko writes: >> The tentative fix is attached, but please >> double check because I am not very familiar with the indentation code. > I've applied your fix on top of 281a0e26b; AFAICT it works as expected. I applied the patch on top of

Re: [PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-24 Thread Kévin Le Gouguec
Ihor Radchenko writes: >The tentative fix is attached, but please > double check because I am not very familiar with the indentation code. I've applied your fix on top of 281a0e26b; AFAICT it works as expected. Thanks for cooking up a test case! 'make check'

[PATCH] Re: [BUG] indention of drawer does not work [9.5 (release_9.5-145-gd18beb @ /home/oub/emacs/site-lisp/packages/org/)]

2021-10-24 Thread Ihor Radchenko
Kévin Le Gouguec writes: > fc80d052d was a rather chunky commit so I did not dig into the patch; > Ihor, does it make sense to you that it might have introduced > unfortunate side-effects wrt :PROPERTIES: indentation? You are right. org--get-expected-indentation have some expectations about