dyng / ctrlsf.vim

A text searching plugin mimics Ctrl-Shift-F on Sublime Text 2
1.6k stars 86 forks source link

support specify new arguments for CtrlSFUpdate #334

Closed hiberabyss closed 2 years ago

hiberabyss commented 2 years ago

Could use CtrlSFUpdate -B 10 like command to update search arguments

dyng commented 2 years ago

CtrlSFUpdate的改进需求我觉得是合理的,不过有几个点建议改进:

  1. 目前的实现方式比较tricky,建议改一下。可以改成下面这样:
    1. ctrlsf#opt#ParseOptions(a:args)增加一个override参数,用于覆盖前面的参数。
    2. 去掉s:DoSearchSync(a:args)s:DoSearchAsync(a:args)a:args参数(我看了下函数内部其实没有用到)。
  2. 把每个功能拆分成几个PR,然后每个PR都squash到一个commit。