Review Request 120085: Use only file extension to determine file type.

Sergey Kalinichev kalinichev.so.0 at gmail.com
Sun Sep 7 15:00:12 UTC 2014


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

(Updated Sept. 7, 2014, 3 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDevelop.


Repository: kdevplatform


Description
-------

 Real project example:
someFile.txt begins with `/*`
it considered as c++ file by KMimeType::findByUrl(url).
Then we do: d->fileExtensionCache.insert(extension, languages); So now we think that c++ support can parse txt files.
 Then there is a CMakeLists.txt file that is already parsed by cmake language support, but on project opening (or when opening it in the editor) we try to parse it again and see that it extension is `*.txt` and we already registered txt files as supported by c++ language support, so this file is being parsed by c++ language support, but as it already has context created by cmake support and we assume that it's created by us -> crash


Diffs
-----

  shell/languagecontroller.cpp 817c1ce 

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


Testing
-------


Thanks,

Sergey Kalinichev

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


More information about the KDevelop-devel mailing list