zk-phi / phi-search

Another incremental search command, compatible with “multiple-cursors”
127 stars 12 forks source link

Latest commit (14e2cc0) breaks multiple cursors #15

Closed nispio closed 11 years ago

nispio commented 11 years ago

The latest commit breaks phi-search in multpile-cursors mode. To reproduce the problem, try the following:

Add the following dummy text lines to a buffer:

// Test document
code // comment
code // comment
code // comment

Now using multiple-cursors set a cursor at the beginning of each line of "code", do a phi-search for // and then press enter:

C-> C-> C-s / / <return>

The expected behavior here is that each of the fake cursors jumps ahead to the next occurrence of //. However, what I am seeing after commit 14e2cc0 is that the primary cursor jumps ahead as expected, but the fake cursors all jump to the first occurrence of // in the file.

zk-phi commented 11 years ago

fixed. thanks for reports and sorry for inconvenience.