Fwd: Moved runner "Amarok" to kdereview

Aaron J. Seigo aseigo at kde.org
Mon Oct 19 20:55:56 CEST 2009


On October 18, 2009, Jan Gerrit Marker wrote:
> Am Sonntag 18 Oktober 2009 16:30:59 schrieb Albert Astals Cid:
> > I see
> >   amarokrunner_config.cpp:77: warning: taking address of temporary
> > that should be fixed as those warnings are pretty serious
> 
> Novice question: why are they serious?

they can cause crashes. the object is "temporary" so the compiler is free to 
re-use that memory whenever it wants, meaning that it's possible for the 
accesses made to that object are accessing memory that has since been 
overwritten with something other than the code/data one might expect there ... 
leading to a crash.

> I fixed both.

great :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20091019/47874f96/attachment.sig 


More information about the Plasma-devel mailing list