Review Request 127173: Avoid unset values in comicdata

David Edmundson david at davidedmundson.co.uk
Thu Feb 25 13:33:21 UTC 2016



> On Feb. 25, 2016, 9:54 a.m., Marco Martin wrote:
> > Ship It!
> 
> Marco Martin wrote:
>     perhaps related to https://bugs.kde.org/show_bug.cgi?id=359614 ?

that's the reason I had a comic strip on my desktop :)
but I don't see from the traces (sorry, I didn't save my valgrind trace) how they could be the same.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127173/#review92758
-----------------------------------------------------------


On Feb. 25, 2016, 1:03 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127173/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2016, 1:03 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> -------
> 
> The default constructor is sometimes called.
> At which point we end up using uninitiased values.
> 
> 
> Diffs
> -----
> 
>   applets/comic/comicdata.h 85ccbd5dbfd14a74025ebc38cd9729f6e9c16514 
>   applets/comic/comicdata.cpp 85be78105d435863d0df8d3b2b9c246b7e6232d5 
> 
> Diff: https://git.reviewboard.kde.org/r/127173/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160225/b13b7f27/attachment.html>


More information about the Plasma-devel mailing list