----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128885/#review99093 -----------------------------------------------------------
Just checked that the engine.h is not installed, so no problems in changing the data types from a binary compatibility point of view. - Sune Vuorela On Sept. 11, 2016, 10:57 a.m., Christoph Cullmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128885/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2016, 10:57 a.m.) > > > Review request for KDE Frameworks, Boudhayan Gupta, David Faure, and Pinak > Ahuja. > > > Repository: baloo > > > Description > ------- > > Increase size limit of baloo index for 64-bit machines to avoid crashs after > > 5GB of index size. > (Better would be additional out-of-space handling, but ATM baloo has zero > checks for that) > > The size limit for 32-bit is still 1GB, like before (there was a silent > overflow from 5GB to 1GB in the computation), people with large homes will > still get random segfaults on 32-bit. > > Patch based on patch from Hao Zhang, Bug 364475 > https://bugs.kde.org/show_bug.cgi?id=364475 > > > Diffs > ----- > > src/engine/database.cpp 89e2e03 > src/engine/databasesize.h aa180fb > src/engine/transaction.cpp 0af20be > src/tools/balooctl/statuscommand.cpp 73289c4 > > Diff: https://git.reviewboard.kde.org/r/128885/diff/ > > > Testing > ------- > > > Thanks, > > Christoph Cullmann > >
