Review Request 115226: quickopen abbreviation matching

Sven Brauch svenbrauch at googlemail.com
Thu Jan 23 02:07:27 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115226/
-----------------------------------------------------------

(Updated Jan. 23, 2014, 2:07 a.m.)


Review request for KDevelop and Vishesh Handa.


Changes
-------

fix issues raised by milian


Repository: kdevplatform


Description
-------

Adds abbreviation matching for quickopen, similar to what the completion does now.

For duchain items, you can do component-wise abbrev matching with the same algorithm like in kate.
For example, py::pyccwor matches "Python::PythonCodeCompletionWorker", kdev::cccont matches "KDevelop::CodeCompletionContext", etc.

For project items, you can type any set of characters from the path of the item. So, btbrowser/cml matches addons/kate/backtracebrowser/CMakeLists.txt. The idea is that often you can abbreviate path names but they seldomly contain capital letters, so it doesn't make much sense to restrict it to capital letters.

I'm not sure this is the perfect way to do it, feedback is welcome.


Diffs (updated)
-----

  language/CMakeLists.txt d5f29c1 
  language/interfaces/abbreviations.h PRE-CREATION 
  language/interfaces/quickopenfilter.h cd3afa9 
  plugins/projectfilter/tests/test_projectfilter.cpp 21c6675 
  plugins/quickopen/projectitemquickopen.cpp 8be9dd8 
  plugins/quickopen/tests/quickopentest.h eff5b93 
  plugins/quickopen/tests/quickopentest.cpp 3827237 
  plugins/quickopen/tests/quickopentestbase.h 1205379 

Diff: https://git.reviewboard.kde.org/r/115226/diff/


Testing
-------


Thanks,

Sven Brauch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140123/b5e0c7de/attachment.html>


More information about the KDevelop-devel mailing list