> 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 :) > > Albert Astals Cid wrote: > Janitorial dude question: Has this been commited and you forgot to mark > it as submitted or it does still need submitting? >
No this hasn't been submitted yet. I am waiting for the contributor. I am aware of the existence of this review. If the contributor doesn't show up I will submit it. - Giorgos ----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
