JazzCore / ctrlp-cmatcher

CtrlP C matching extension
BSD 2-Clause "Simplified" License
207 stars 22 forks source link

Cmatcher matches whole string in Tag mode #24

Open DXist opened 10 years ago

DXist commented 10 years ago

Cmatcher should match just tag name in Tag mode. Now it matches whole string.

JazzCore commented 10 years ago

Works good for me. Can you give a test case?

DXist commented 10 years ago

I've shared sample tags file. Results for 'Good' query:

Stock matcher Cmatcher

Stock matcher puts short match at the bottom. When I add tags from project virtualenv things get even worse.

JazzCore commented 10 years ago

This differs from the issue that you've mentioned in title, doesnt it? I see that cmatcher properly matches only in tag names (tag comments does not even get passed to matcher).

Regarding your test case, I'm seeing different results with stock matcher ( it does not show 'Good' on top of results ), when cmatcher results are the same. It can be a flaw in matcher algorithm, I'll see what I can do.

DXist commented 10 years ago

You are right. I've changed the title to express my problem better.