Review Request 108943: List project names in matches

[Milian Wolff]

> On Feb. 13, 2013, 4:25 p.m., Milian Wolff wrote:
> > app/plasma/runner/kdevelopsessions.h, line 27
> > <http://git.reviewboard.kde.org/r/108943/diff/1/?file=113936#file113936line27>
> >
> >     mark this type as movable please
> 
> Aurélien Gâteau wrote:
>     I am afraid I don't know what "mark as movable" means.
> 
> Milian Wolff wrote:
>     Q_DECLARE_METATYPE(Session, Q_MOVABLE_TYPE);
> 
> Aurélien Gâteau wrote:
>     I just read the doc about this. It says "Q_MOVABLE_TYPE specifies that Type has a constructor and/or a destructor but can be moved in memory using memcpy().". Are you sure one can copy QStrings with memcpy()?

yes


- Milian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108943/#review27382
-----------------------------------------------------------


On Feb. 13, 2013, 5:08 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108943/
> -----------------------------------------------------------
> 
> (Updated Feb. 13, 2013, 5:08 p.m.)
> 
> 
> Review request for KDevelop and Sebastian Kügler.
> 
> 
> Description
> -------
> 
> List project names in matches
> 
> This is useful when one does not take the time to give name to KDevelop
> sessions.
> 
> 
> Diffs
> -----
> 
>   app/plasma/runner/kdevelopsessions.h eb85da9825d5c8bf1db02b082738ae82a7acc108 
>   app/plasma/runner/kdevelopsessions.cpp a1755379ca9fea8144c211b63c3b1a912263ecc6 
> 
> Diff: http://git.reviewboard.kde.org/r/108943/diff/
> 
> 
> Testing
> -------
> 
> Tested with krunner and homerun
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

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