indenting long lists fails because of maxoff #63

Activity

1. blueyed commented on Feb 22, 2017

   

   blueyed

   on Feb 22, 2017

   Member

   More actions

   #64 has a fix/config option.

   @temaput
   What would you set it to? Please provide your feedback over there.
2. temaput commented on Feb 23, 2017

   

   temaput

   on Feb 23, 2017

   Last edited by temaput

   Author

   More actions

   Personaly I removed it completely on my machine (the stopline arg from searchpairs to be precise). Maybe I am not used to dealing with really long files, but I am not experiencing any lags without maxoff at all. So I think maybe to make it optional, or make it turning on only in long files, like YouCompleteMe does...
3. mgedmin commented on Aug 30, 2017

   

   mgedmin

   on Aug 30, 2017

   More actions

   How about double the default value every time someone files a bug complaining about this? Undo the last doubling if somebody complains about indentation being noticeably slow. Should converge to a reasonable default value after a few iterations.

   (I'm reading this issue because I also have list literals in my unit tests that are longer than 50 lines. In this particular project they're shorter than 100 lines. I've now set g:python_pep8_indent_max_back_search to 1000 in my .vimrc and will see if I can perceive any slowdown.)
4. blueyed commented on Dec 22, 2018

   

   blueyed

   on Dec 22, 2018

   Member

   More actions

   Defaults have been adjusted by now.
   How does it work for you?
5. Kache commented on Oct 29, 2024

   

   Kache

   on Oct 29, 2024

   More actions

   In case anyone else runs into this (since it's not yet mentioned here), the current workaround from #91 is configured at:

   vim-python-pep8-indent/indent/python.vim

   Lines 58 to 60 in 60ba5e1

   	" Pairs to look for when searching for opening parenthesis.
   	" The value is the maximum offset in lines.
   	let s:paren_pairs = {'()': 50, '[]': 100, '{}': 1000}

   Looks like maintainers want to keep it private and keep the freedom to change it in the future

   In the meantime, I've set my own to:

   let s:paren_pairs = {'()': 9999, '[]': 9999, '{}': 9999}