LanguageController extension cache fails for system includes without extension

[Milian Wolff]

David Nolden, 20.03.2011:
> I see no reason to rush on this. The "looking into files" behavior is a
> showstopper for really large projects (makes KDevelop unusable), while the
> other thing is just an annoyance.

No, KDevelop has not tons of files without extension. Only that part is what I 
want to remove. Personally I find it much more a showstopper that standard 
headers are not parsed correctly.

> There are a few things which we can do:
> - Allow looking into the file if it is open in the editor

most of the time you won't open a system header

> - Use the duchain to determine the language-support. If the file was ever
> included somewhere, there will be a TopDUContext for it.

no there wont. You create a parsejob for the include file but the language 
controller will not find a language for it (as it has no extension).

bye

> Am 15.03.2011 23:16 schrieb "Milian Wolff" <mail at milianw.de>:
> > Milian Wolff, 07.03.2011:
> >> Hey all, esp. David.
> >> 
> >> It seems like the extension cache in the LanguageController needs some
> 
> love
> 
> >> or get removed. It prevents proper parsing for system includes without
> >> extensions (e.g. cstddef, ...). I wonder why we didn't spot this
> 
> sooner...
> 
> >> https://bugs.kde.org/show_bug.cgi?id=267877
> >> 
> >> Imo the only solution to this is: get rid of the code preventing
> 
> KMimeType
> 
> >> to look into the file... If there is no extension, the contents is all
> >> we have...
> >> 
> >> I understand you had issues with kdevelop trying to parse data folders
> >> or something like that in your project. Imo the correct solution would
> >> be to somehow ignore these, maybe even just by adding a .kdev_ignore
> >> file to
> 
> the
> 
> >> data folder. Bonus points for proper UI integration of course.
> > 
> > *push*
> > 
> > David - without any response I see no other possibility but to remove
> > that
> > 
> > code responsible for this bug. Please answer, otherwise I'll push the
> 
> required
> 
> > patch on the weekend.
> > 
> > bye
> > --
> > Milian Wolff
> > mail at milianw.de
> > http://milianw.de


-- 
Milian Wolff
mail at milianw.de
http://milianw.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110321/36e80e33/attachment-0001.sig>