----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129114/#review99831 -----------------------------------------------------------
Fix it, then Ship it! Looks good. kdialog/widgets.cpp (line 115) <https://git.reviewboard.kde.org/r/129114/#comment67093> QLatin1String is better for a comparison - David Faure On Oct. 6, 2016, 5:57 p.m., Anthony Fieroni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129114/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2016, 5:57 p.m.) > > > Review request for KDE Base Apps and David Faure. > > > Repository: kde-baseapps > > > Description > ------- > > ^^ > > > Diffs > ----- > > kdialog/widgets.cpp 70f4258 > > Diff: https://git.reviewboard.kde.org/r/129114/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Fieroni > >