----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129657/#review101463 -----------------------------------------------------------
Ship it! Ship It! - Stephen Kelly On Dec. 15, 2016, 11:15 p.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129657/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2016, 11:15 p.m.) > > > Review request for KDE Frameworks and Stephen Kelly. > > > Repository: kitemmodels > > > Description > ------- > > After > int proxyEndRemove = proxyStartRemove; > proxyEndRemove += rc; was adding 0 (empty root) > and then --proxyEndRemove; was making us end up with proxyEndRemove < > proxyStartRemove. > > > Diffs > ----- > > autotests/kselectionproxymodeltest.cpp > 483e7c42dabab6aa560622ff0418ee7f90363e15 > src/kselectionproxymodel.cpp 0f57c70f05d4cbde9b14f4257bff1365ef5443f6 > > Diff: https://git.reviewboard.kde.org/r/129657/diff/ > > > Testing > ------- > > New unittest + unchecking calendar in korganizer no longer asserts. > > > Thanks, > > David Faure > >