[kdev-clang] /: Always update opened in editor documents
Milian Wolff
mail at milianw.de
Wed Aug 27 11:52:23 UTC 2014
On Wednesday 27 August 2014 11:29:20 Sergey Kalinichev wrote:
> Git commit 83b50cbf3707a2e8cd8f1893c262156a3d925493 by Sergey Kalinichev.
> Committed on 26/08/2014 at 08:12.
> Pushed by skalinichev into branch 'master'.
>
> Always update opened in editor documents
>
> This is important as those files won't be reparsed properly after
> e.g. crash or if they were modified in previous session, but not closed.
Hello Sergey,
this patch is wrong I think. It work-arounds an underlying issue. When you
open KDevelop without a crash or without modifications, then using the cache
is just fine. Always parsing is _wrong_.
I'll revert this patch. Please let us discuss how to come up with a proper
solution to this problem.
The other patch also could be simplified (which I did, without test-compiling,
so I'm also to blame).
Generally, please put the patches on reviewboard first.
--
Milian Wolff
mail at milianw.de
http://milianw.de
More information about the KDevelop-devel
mailing list