Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SimplyFold has issues recalculating the fold when adding lines to the end of a fold #116

Open
ghost opened this issue Jan 22, 2020 · 6 comments

Comments

@ghost
Copy link

ghost commented Jan 22, 2020

Steps to reproduce:

  • Create python file as follows
class A:
    """Class Docstring
    """
    def __init__(self):
        """init docstring
        """
        pass

    def myprint(self):
        """Testing function docstring
        """
        print('Printed myprint')
        print('Printed myprint a second time')
  • Then, save it as test.py
  • Open the file again
  • Go to the last line
  • Add another line at the end of myprint
class A:
    """Class Docstring
    """
    def __init__(self):
        """init docstring
        """
        pass

    def myprint(self):
        """Testing function docstring
        """
        print('Printed myprint')
        print('Printed myprint a second time')
        print('another line')
  • Now, the last line is excluded from the fold
  • As an example, if you press zM, you get:

+-- 13 lines: class A: Class Docstring--------------------------------------------
print('another line')

@tmhedberg
Copy link
Owner

tmhedberg commented Jan 22, 2020 via email

@obrusvit
Copy link

I can confirm this is occurring in my Vim setup exactly as nabeel-ms describes it.
However, If I close the file and then I open it again, the folds are correct again.

@tmhedberg
Copy link
Owner

Please paste the output of :set foldmethod? foldexpr? foldlevel? foldminlines? foldnestmax?.

What is the state of the existing folds in the file at the time that you add the additional line at the end? Are they open or closed?

I'm still not seeing this behavior, so I'm trying to figure out what's different for me.

It would still be helpful if you could try this with all plugins except for this one disabled. A lot of weird problems can be caused by the interaction between multiple plugins that weren't designed to work together, since Vim doesn't really isolate them.

@ghost
Copy link
Author

ghost commented Jan 24, 2020

The output is as follows:

foldmethod=expr
foldexpr=SimpylFold#FoldExpr(v:lnum)
foldlevel=3
foldminlines=1
foldnestmax=20

I have uninstalled all my plugins excluding YouCompleteMe.
In this case, to avoid issues, I used zR to reopen all folds before appending. I still encountered the same issues. However, it should be noted that when I use zx to recompute the folds in the file, the file folding is fixed and has no issues.

@vecin2
Copy link

vecin2 commented May 26, 2020

I've encountered the same issue. Thanks for the zx workaround!
This is my output of running the above:

  foldexpr=SimpylFold#FoldExpr(v:lnum)
  foldlevel=0
  foldminlines=1
  foldnestmax=20

I removed all my config and left only this plugin then I was able to replicate with a file like the following:

def some_fuction(var):
    var = 1 + 2

Then I added and an extra line:

def some_fuction(var):
    var = 1 + 2
    var2 =2 

Then press <Esc> to go back to normal mode and then zm and this is what I got:

+--  2 lines: def some_fuction(var):------------------------
    var2 = 2

@yongrenjie
Copy link

yongrenjie commented Jul 26, 2020

Hi all -
I suspect this has something to do with the caching of the folds. I have a similar issue whereby if I add lines before a fold, all subsequent folds get messed up badly upon saving. This technically sounds closer to #112 but since the video there is no longer accessible I'm not sure if the behaviour is the same.

With git bisect I find that the behaviour was introduced in commit 2dfeb35.

zx alone does not work for me, unfortunately. I find that I need to regenerate the cache, so the following autocmd (which I placed in ~/.vim/after/ftplugin/python.vim) solves my issue:

function UpdateFolds()
    call SimpylFold#Recache()
    FastFoldUpdate!
endfunction
autocmd BufWritePre <buffer> call UpdateFolds()

There is probably a tiny bit of lag especially with large files, but since it's only triggered upon saving (which takes a while anyway), it's quite difficult to notice. On a 1200-line file it takes me 100–200 ms to recalculate the folds and write the buffer. One could probably use normal zx in place of FastFoldUpdate! if FastFold isn't installed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants