> On apr 28, 2016, 9:12 a.m., Milian Wolff wrote: > > src/sycoca/ksycocautils.cpp, line 29 > > <https://git.reviewboard.kde.org/r/127770/diff/2/?file=461459#file461459line29> > > > > as a follow-up cleanup I suggest to introduce an enum to hold this > > magic value
I can do that if the code stays. I made another rr to remove it: https://git.reviewboard.kde.org/r/127786/ - Jos ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127770/#review94946 ----------------------------------------------------------- On apr 28, 2016, 7:01 a.m., Jos van den Oever wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127770/ > ----------------------------------------------------------- > > (Updated apr 28, 2016, 7:01 a.m.) > > > Review request for KDE Frameworks and David Faure. > > > Repository: kservice > > > Description > ------- > > The code for reading and writing of strings in KSycoca is not symmetrical. > Strings of any length can be written, but only strings of less than 8192 > bytes may be read. This limit is set in KSycocaUtilsPrivate::read. The limit > is probably there to avoid out-of-memory situations. > > On my system I have a lot of XDG data dirs. The length of the environment > variable is currently 4092 bytes. KSycocaBuild saves that as UTF-16 which > needs 8184 bytes. KBuildSycoca save that without complaint but complains when > reading it. > > > The simplest solution here is to simply increase the magic number 8192 to > e.g. 65528. This is just a temporary buffer. > > Or we just check the size of the whole cache file (e.g. < 100M) and remove > all other limits. That would simplify > > KSycocaUtilsPrivate::read(*str, header.prefixes); > > to > > *str >> header.prefixes; > > This patch chooses the first option. > > > Diffs > ----- > > src/sycoca/ksycocautils.cpp 1ba75e8 > > Diff: https://git.reviewboard.kde.org/r/127770/diff/ > > > Testing > ------- > > Ran unit tests on KService. All but one of the previously failing tests on > NixOS is now fixed. > > > Thanks, > > Jos van den Oever > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel