----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109555/#review30128 -----------------------------------------------------------
This review has been submitted with commit ffc8a6a316cacc8df6a91584653fe2c867a5d489 by Bernd Buschinski to branch master. - Commit Hook On March 18, 2013, 12:45 a.m., Bernd Buschinski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109555/ > ----------------------------------------------------------- > > (Updated March 18, 2013, 12:45 a.m.) > > > Review request for kdelibs. > > > Description > ------- > > the Patch differs from the patch in Bug 316923 > as it also makes > ulimit -s 2147483649 > work. > > On Systems that don't have rlim_t, availableStackSize is basically just a > static constant with 8*1024*1024, which fits in int. > > > This addresses bug 316923. > http://bugs.kde.org/show_bug.cgi?id=316923 > > > Diffs > ----- > > kjs/regexp.h f6132e6 > kjs/regexp.cpp 5294d62 > > Diff: http://git.reviewboard.kde.org/r/109555/diff/ > > > Testing > ------- > > ulimit -s unlimited > ./kjs_bin.shell -e '/a/.test("a")' > > > ulimit -s 2147483649 (over int limit) > ./kjs_bin.shell -e '/a/.test("a")' > > > ulimit -s 2047483649 (under int limit) > ./kjs_bin.shell -e '/a/.test("a")' > > > and without setting ulimit (default) > ./kjs_bin.shell -e '/a/.test("a")' > > They all work. > > > Thanks, > > Bernd Buschinski > >
