----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104855/#review13774 -----------------------------------------------------------
kjs/commonunicode.h <http://git.reviewboard.kde.org/r/104855/#comment10934> This needs a bit of a comment tweak --- it's not in Zs but rather Cf, but ES 5th edition treats it specially. (And differently from the rule in 3rd edition, which no-one implemented). - Maks Orlovich On May 4, 2012, 5:40 p.m., Bernd Buschinski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104855/ > ----------------------------------------------------------- > > (Updated May 4, 2012, 5:40 p.m.) > > > Review request for kdelibs. > > > Description > ------- > > kjs: Unify/de-duplicate Space checking > > function.cpp isStrWhiteSpace was missing "0xFEFF: // ZERO WIDTH NO-BREAK > SPACE" > > > Diffs > ----- > > kjs/commonunicode.h PRE-CREATION > kjs/function.cpp 5f39ae6 > kjs/lexer.cpp e89de5f > > Diff: http://git.reviewboard.kde.org/r/104855/diff/ > > > Testing > ------- > > > Thanks, > > Bernd Buschinski > >
