----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129732/#review101686 -----------------------------------------------------------
Fix it, then Ship it! autotests/kfiledialog_unittest.cpp (line 109) <https://git.reviewboard.kde.org/r/129732/#comment68103> Will be uninitialized if the lambda isn't called. Better initialize it. - David Faure On Dec. 31, 2016, 9:42 a.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129732/ > ----------------------------------------------------------- > > (Updated Dec. 31, 2016, 9:42 a.m.) > > > Review request for Plasma, David Faure and Friedrich W. H. Kossebau. > > > Repository: plasma-integration > > > Description > ------- > > 7bbbd93cd3fc0abdffd3fa7f144cb50a33fafad9 makes the save dialog appear as Open > dialog. > > Simplify the code in that commit so it does not regress anymore. > > Comes with a unit test (argably a bit meh since it could loop forever if the > dialog does not show, but i guess it is good if that happens too :D) > > > Diffs > ----- > > autotests/kfiledialog_unittest.cpp 47a5543 > src/platformtheme/kdeplatformfiledialoghelper.cpp 990b983 > > Diff: https://git.reviewboard.kde.org/r/129732/diff/ > > > Testing > ------- > > New test fails without the patch, works with it. Kate Save As dialog no > longer shows as Open dialog. > > > Thanks, > > Albert Astals Cid > >