Review Request 110288: Checks if an optional field was instantiated before accessing it

Albert Astals Cid aacid at kde.org
Thu Aug 15 22:11:48 UTC 2013



> On May 4, 2013, 9:14 a.m., Giorgos Tsiapaliokas wrote:
> > The patch is ok, it can go in.
> > 
> > Are you familiar with out git workflow policy?
> > 
> > http://community.kde.org/Plasma/IntegrationBranches/GeneralPolicy
> > 
> > If you have any questions don't hesitate to ask :)

Janitorial dude question: Has this been commited and you forgot to mark it as submitted or it does still need submitting?


- Albert


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


On May 3, 2013, 8:38 p.m., Denis Falqueto wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110288/
> -----------------------------------------------------------
> 
> (Updated May 3, 2013, 8:38 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> The private field m_konsolewidget is initialized only on some specific cases, so it should be accessed with caution. But there are some places in witch it's being used without checking if it was instantiated or not. This patch fixes those accesses, by checking if it is instantiated or not.
> 
> 
> This addresses bug 319291.
>     http://bugs.kde.org/show_bug.cgi?id=319291
> 
> 
> Diffs
> -----
> 
>   plasmate/mainwindow.cpp 1ef5a81 
> 
> Diff: http://git.reviewboard.kde.org/r/110288/diff/
> 
> 
> Testing
> -------
> 
> I've created a data engine in python successfully.
> 
> 
> Thanks,
> 
> Denis Falqueto
> 
>

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


More information about the Plasma-devel mailing list