branches/kdevelop/3.4/src

[Jens Dagerbo]

On 9/14/06, Matt Rogers <mattr at kde.org> wrote:
> On Thursday 14 September 2006 01:03, Andras Mantia wrote:
> > On Thursday 14 September 2006 03:47, David Nolden wrote:
> > > Am Freitag, 8. September 2006 00:45 schrieb Andras Mantia:
> > > > SVN commit 581925 by amantia:
> > > >
> > > > Fix the famous XIM crash when during loading of a project the user
> > > > switches to another application (the backtrace with the
> > > > KateViewInternal::updateMicroFocusHint).
> > > >
> > > > CCMAIL: kdevelop-devel at kdevelop.org
> > > >
> > > >  M  +4 -2      editorproxy.cpp
> > >
> > > Hi! Thanks for fixing this annoying crash, but theres a problem with
> > > the patch. It makes all the quickopen-menus not work(they only open
> > > the file, but because the focus is not set to the editor, the line is
> > > not changed.).
> > >
> > > Has anyone an Idea how to fix this? I've already fiddled around with
> > > closing the quickopen-windows earlier and using setFocus() on the
> > > editor, but that didn't help.
> >
> > You might remove the check here as I committed the workaround to the
> > katepart itself.
> > Of course it will mean that the crash will happen with KDe <= 3.5.4!
> >
> > Andras
>
> And since we support more than that, we still need the check. :)
>

Hang on.. If the editor is not in focus, we do not set the line number !?
I haven't tested it yet and I might be reading it wrong, but won't
this break pretty much all navigation?

Including:
Quick Open (as David noticed)
Grep
Bookmarks
Replace
ClassView
Make output
Breakpoints
...

It doesn't look like a very nice solution, does it? In fact, it seems
pretty close to making KDevelop useless...

// jens