----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120904/#review97358 -----------------------------------------------------------
src/kcharselect.h (line 210) <https://git.reviewboard.kde.org/r/120904/#comment65722> QVector? - Christoph Feck On July 13, 2016, 9:23 p.m., Christoph Feck wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120904/ > ----------------------------------------------------------- > > (Updated July 13, 2016, 9:23 p.m.) > > > Review request for KDE Frameworks and Daniel Laidig. > > > Repository: kwidgetsaddons > > > Description > ------- > > To be able to fix bug 142625 we need new API in KCharSelect. > > I am not completely sure if we should do it in KWidgetsAddons at all, because > the required data updates might unreasonable blow up the size of the > repository, but I just came up with this proposal for the API. Note that only > the header diff is shown for the API review, the .cpp changes can be done > after we settle for something. > > Some things to discuss: > * uint is used by static QChar functions as the datatype, I adopted it > * it might make sense to have a function that returns the currently selected > character(s) as a simple QString, with all UTF-16 handling already done. > Naming suggestions? > > > Diffs > ----- > > src/kcharselect.h 05ec77c > > Diff: https://git.reviewboard.kde.org/r/120904/diff/ > > > Testing > ------- > > No. > > > Thanks, > > Christoph Feck > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
