> On Nov. 22, 2015, 7:51 p.m., Milian Wolff wrote: > > src/klocalizedcontext.h, line 59 > > <https://git.reviewboard.kde.org/r/126087/diff/5/?file=417518#file417518line59> > > > > virtual
It already is, I'll mark it as override. > On Nov. 22, 2015, 7:51 p.m., Milian Wolff wrote: > > src/klocalizedcontext.h, line 132 > > <https://git.reviewboard.kde.org/r/126087/diff/5/?file=417518#file417518line132> > > > > are we sure we'll never add anything else? i.e. is pimpling a safer > > choice? Good point. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126087/#review88697 ----------------------------------------------------------- On Nov. 20, 2015, 3:03 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126087/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2015, 3:03 p.m.) > > > Review request for KDE Frameworks, Chusslove Illich and Marco Martin. > > > Repository: ki18n > > > Description > ------- > > The only way to use `i18n*()` so far in QML is by depending on all > KDeclarative. This patch moves the necessary bits there so it can be adopted > by an application or framework just by offering the needed bits within KI18n. > This is done by offering an object with methods that map to the `i18n*()` C++ > counter-parts. > > > Diffs > ----- > > autotests/CMakeLists.txt 1cf0f7a > autotests/ki18ndeclarativetest.cpp PRE-CREATION > autotests/test.qml PRE-CREATION > docs/programmers-guide.md 13a5f9d > src/CMakeLists.txt 818595e > src/klocalizedcontext.h PRE-CREATION > src/klocalizedcontext.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/126087/diff/ > > > Testing > ------- > > Ported KDeclarative, everything still seems to work. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
