<div class="gmail_quote">2010/9/14 Ingo Klöcker <span dir="ltr"><<a href="mailto:kloecker@kde.org">kloecker@kde.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">On Tuesday 14 September 2010, Mark wrote:<br>
> On Mon, Sep 13, 2010 at 11:13 PM, Lubos Lunak <<a href="mailto:l.lunak@suse.cz">l.lunak@suse.cz</a>> wrote:<br>
> > On Monday 13 of September 2010, Mark wrote:<br>
> > > On Mon, Sep 13, 2010 at 10:00 PM, Mark <<a href="mailto:markg85@gmail.com">markg85@gmail.com</a>> wrote:<br>
> > > > --- src/dolphincontextmenu.cpp (revision 1174907)<br>
> > > > +++ src/dolphincontextmenu.cpp (working copy)<br>
> > > > @@ -69,6 +69,9 @@<br>
> > > ><br>
> > > >      const DolphinView* view =<br>
> > > ><br>
> > > > m_mainWindow->activeViewContainer()->view(); m_selectedUrls =<br>
> > > > view->selectedUrls();<br>
> > > ><br>
> > > >      m_selectedItems = view->selectedItems();<br>
> > > ><br>
> > > > +    m_keyInfo = new KModifierKeyInfo();<br>
> ><br>
> >  And who deletes it?<br>
><br>
> Not in the above patch, but the destructor now does that:<br>
> DolphinContextMenu::~DolphinContextMenu()<br>
<br>
</div>Please use QScopedPointer<KModifierKeyInfo> for m_keyInfo instead of a<br>
raw pointer that you have to delete manually.<br>
<br>
<br>
Regards,<br>
<font color="#888888">Ingo<br>
</font></blockquote></div><br><div>Oh wow, neat!</div><div>I didn't even know about that Qt class. I will use it from now on when i need to make pointers.</div>