> On Sept. 17, 2015, 9:25 p.m., Albert Astals Cid wrote: > > Would these also need i18n? > > > > if (running) { > > out << "Baloo File Indexer is running\n"; > > out << "Indexer state: " << > > stateString(schedulerinterface.state()) << endl; > > } > > else { > > out << "Baloo File Indexer is NOT running\n"; > > }
Now that you mention it I realize almost all string in balooctl are not i18nized. I'm gonna fix that in a seperate patch, since this patch mainly deals with i18nizing IndexerState, where ever we use it. - Pinak ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125257/#review85591 ----------------------------------------------------------- On Sept. 16, 2015, 11:36 a.m., Pinak Ahuja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125257/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2015, 11:36 a.m.) > > > Review request for Baloo, Localization and Translation (l10n) and Vishesh > Handa. > > > Repository: baloo > > > Description > ------- > > Delibrately left of baloo-monitor as it will not be the final public facing > interface and will be removed soon. (once I fix this > https://git.reviewboard.kde.org/r/124720/) > > > Diffs > ----- > > src/file/indexerstate.h b2bc8f4 > src/qml/experimental/CMakeLists.txt 335cc9e > src/qml/experimental/Messages.sh PRE-CREATION > src/tools/baloo-monitor/CMakeLists.txt cf9ac51 > src/tools/balooctl/Messages.sh 87bd496 > > Diff: https://git.reviewboard.kde.org/r/125257/diff/ > > > Testing > ------- > > Not too sure how to test it? Also I hope C++ qml plugins will work properly > with i18n? > > > Thanks, > > Pinak Ahuja > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<