----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101362/#review3336 -----------------------------------------------------------
main-widget.cpp <http://git.reviewboard.kde.org/r/101362/#comment2800> You can actually create the whole dialog here, (ie. move the code from top here) and therefore you won't even need the "else {" part. In fact, you won't need even the member variable (the m_noAccountsDialog), you can create it on place. - Martin On May 14, 2011, 4:52 p.m., Tarun Mall wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101362/ > ----------------------------------------------------------- > > (Updated May 14, 2011, 4:52 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > This patch will check if no accounts will be present then an dialog box will > be opened telling no accounts found and giving an option to create on. > > > Diffs > ----- > > main-widget.h 7a5e417 > main-widget.cpp 20e8003 > > Diff: http://git.reviewboard.kde.org/r/101362/diff > > > Testing > ------- > > I tested this patch with > 1. No accounts > 2. Creating one account > 3. Deleting that account and then creating one again. > > > Thanks, > > Tarun > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
