Optimise dtection by limiting number of lines that is analysed when fail is loaded.

[suokko]

attaching patch for fix.

[suokko]

From 64f24f37c0b9530a8eb5ec54bb92436519dff20d Mon Sep 17 00:00:00 2001 From: Pauli Nieminen suokkos@gmail.com Date: Mon, 10 Aug 2009 21:28:57 +0300 Subject: [PATCH] Limit number of lines that are analysed to improve oepning speed if file is very large.

Default number of lines to analyse is 1024. Default can be changed with g:detectindent_max_lines_to_analyse variable.

Signed-off-by: Pauli Nieminen suokkos@gmail.com

doc/detectindent.txt | 4 ++++ plugin/detectindent.vim | 11 +++++++++++ 2 files changed, 15 insertions(+), 0 deletions(-)

diff --git a/doc/detectindent.txt b/doc/detectindent.txt index 0035cb1..ed7e50b 100644 --- a/doc/detectindent.txt +++ b/doc/detectindent.txt @@ -30,6 +30,10 @@ Author: Ciaran McCreesh :let g:detectindent_preferred_indent = 4 < in your |vimrc| file.

• To set limit for number of lines that will be analysed set: >

• :let g:detectindent_max_lines_to_analyse = 1024 +< in your |vimrc| file. +

  1. DetectIndent ChangeLog detectindent-changelog

diff --git a/plugin/detectindent.vim b/plugin/detectindent.vim index 0e90f0b..b6bfdd1 100644 --- a/plugin/detectindent.vim +++ b/plugin/detectindent.vim @@ -46,6 +46,10 @@ fun! DetectIndent() let l:has_leading_spaces = 0 let l:shortest_leading_spaces_run = 0 let l:longest_leading_spaces_run = 0

• let l:max_lines = 1024
• if exists("g:detectindent_max_lines_to_analyse")
• let l:max_lines = g:detectindent_max_lines_to_analyse

• endif

  let l:idx_end = line("$") let l:idx = 1 @@ -93,6 +97,13 @@ fun! DetectIndent() endif

  let l:idx = l:idx + 1

  +

• let l:max_lines = l:max_lines - 1 +
• if l:max_lines == 0
• let l:idx = l:idx_end + 1

• endif

  + endwhile

  if l:has_leading_tabs && ! l:has_leading_spaces

  1.6.3.3

[ciaranm]

Github doesn't seem to like inline patches like that. Could you please fork the repo, apply your change and then send a pull request instead?

[suokko]

yes. I noticed. Seems like git hub is missing major feature that can't make attachments. Maybe there is some tags to make plain text possible in messages but not easy to know it.

Is git send-email ok? (I did send it already to your email contact in github) I can fork and send pull request if you still want it.

[ciaranm]

Yep, that's fine, thanks. I've applied your patch.

Cheers.