Re: [ki18n] src: small qstring optimizations

2015-01-25 Thread Milian Wolff
On Sunday 25 January 2015 12:36:08 David Faure wrote: > On Wednesday 21 January 2015 21:32:07 Nick Shaforostoff wrote: > > +static const QString SUBSTITUTE_ME=QStringLiteral("%1"); > > I'm not sure that adding global static (non-POD) objects in libraries is a > good idea. > We've had many problems

Re: [ki18n] src: small qstring optimizations

2015-01-25 Thread David Faure
On Wednesday 21 January 2015 21:32:07 Nick Shaforostoff wrote: > +static const QString SUBSTITUTE_ME=QStringLiteral("%1"); I'm not sure that adding global static (non-POD) objects in libraries is a good idea. We've had many problems with that in the past (see below for details), and they slow do

Re: ki18n and KUIT

2013-12-02 Thread Chusslove Illich
> [: Aurélien Gâteau :] > Diving in, I found out the i18n*() functions no longer handle KUIT markup. > There are new functions called xi18n*() which do so. [...] That is when I > realized it is quite a lot of work to go through all translations and > check whether they use KUIT or not. Should we ac

Re: ki18n and KUIT

2013-12-02 Thread Stephen Kelly
Aurélien Gâteau wrote: > I am also worried about translatable texts defined in .ui files: right > now the tr2i18n() function which is used to translate strings coming > from .ui files is calling ki18nc() or ki18n(), meaning there is no way > to use KUIT within .ui files. Should we change tr2i18n()

Re: ki18n

2013-09-18 Thread Aleix Pol
On Tue, Sep 17, 2013 at 10:55 PM, Treeve Jelbert wrote: > 1. ki18n exists in both tier2 and staging > > 2. kunitconversion (tier2) depends on ki18n(tier2) > ___ > Kde-frameworks-devel mailing list > Kde-frameworks-devel@kde.org > https://mail.kde.org/ma

Re: Ki18n mostly ready

2013-08-06 Thread Chusslove Illich
> [: Sebastian Kügler :] > Putting KLocalizedString::insertCatalog in is good enough, mark it > DEPRECATED so we get warned that porting work is coming up. The way I see it, there are no more KLocalizedString::insertCatalog() calls anywhere :) All newly incoming code for porting will have the olde

Re: Ki18n mostly ready

2013-08-06 Thread Chusslove Illich
> [: Albert Astals Cid :] > to make it more obvious maybe it should be KI18N_TRANSLATION_DOMAIN ? Well possibly, but to me it looks redundant: "translation domain" is already Gettext-specific terminology, and it would be bizarre to use more than one Gettext-based system in the same piece of code s

Re: Ki18n mostly ready

2013-08-05 Thread Martin Graesslin
On Monday 05 August 2013 20:57:33 Chusslove Illich wrote: > My notion of source compatibility was that it builds and works, not only > that it builds :) Is there really a point of leaving in dummy methods just > so that the build can proceed, even if they do nothing? As I mentioned, my > intention

Re: Ki18n mostly ready

2013-08-05 Thread Kevin Ottens
On Monday 05 August 2013 20:57:33 Chusslove Illich wrote: > My notion of source compatibility was that it builds and works, not only > that it builds :) Is there really a point of leaving in dummy methods just > so that the build can proceed, even if they do nothing? As I mentioned, my > intention

Re: Ki18n mostly ready

2013-08-05 Thread Kevin Ottens
On Monday 05 August 2013 22:35:08 Albert Astals Cid wrote: > El Dilluns, 5 d'agost de 2013, a les 22:14:07, Chusslove Illich va escriure: > > > [: Albert Astals Cid :] > > > How would that > > > > > > set(TRANSLATION_DOMAIN "foobar") > > > > > > help you knowing if you want a i18n() or a tr()

Re: Ki18n mostly ready

2013-08-05 Thread Albert Astals Cid
El Dilluns, 5 d'agost de 2013, a les 22:14:07, Chusslove Illich va escriure: > > [: Albert Astals Cid :] > > How would that > > > > set(TRANSLATION_DOMAIN "foobar") > > > > help you knowing if you want a i18n() or a tr() based system? > > It would automatically use i18n() if TRANSLATION_DOMA

Re: Ki18n mostly ready

2013-08-05 Thread Chusslove Illich
> [: Albert Astals Cid :] > How would that > set(TRANSLATION_DOMAIN "foobar") > help you knowing if you want a i18n() or a tr() based system? It would automatically use i18n() if TRANSLATION_DOMAIN is defined :) Because it would have no purpose with tr()-based code. >> [: Chusslove Illich :]

Re: Ki18n mostly ready

2013-08-05 Thread Albert Astals Cid
El Dilluns, 5 d'agost de 2013, a les 21:10:15, Chusslove Illich va escriure: > >>> [: Albert Astals Cid :] > >>> If I can't convice you so that we write software that primarily works > >>> for ourselves can I at least convince you to make so that > >>> kconfig_compiler forces you to pass a command

Re: Ki18n mostly ready

2013-08-05 Thread Sebastian Kügler
Hi Chusslove, On Monday, August 05, 2013 20:57:33 Chusslove Illich wrote: > > [: Sebastian Kügler :] > > I'm not really happy about this patch, since it introduces build breakage. > > [...] Also, a reminder: Please build plasma-framework[master] and kde- > > workspace[frameworks-scratch] if you do

Re: Ki18n mostly ready

2013-08-05 Thread Chusslove Illich
>>> [: Albert Astals Cid :] >>> If I can't convice you so that we write software that primarily works >>> for ourselves can I at least convince you to make so that >>> kconfig_compiler forces you to pass a command line parameter saying >>> which i18n model you follow and fails otherwise so people h

Re: Ki18n mostly ready

2013-08-05 Thread Chusslove Illich
> [: Sebastian Kügler :] > I'm not really happy about this patch, since it introduces build breakage. > [...] Also, a reminder: Please build plasma-framework[master] and kde- > workspace[frameworks-scratch] if you do changes, to make sure that no one > else has to sift through mailinglist threads [

Re: Ki18n mostly ready

2013-08-05 Thread Martin Graesslin
On Monday 05 August 2013 16:19:07 Sebastian Kügler wrote: > Hi Chusslove, > > On Friday, August 02, 2013 12:00:50 Chusslove Illich wrote: > > > [: David Faure :] > > > There's no way to keep the [insertCatalog] method for source > > > compatibility > > > and implement it somehow on top of the new

Re: Ki18n mostly ready

2013-08-05 Thread Albert Astals Cid
El Dilluns, 5 d'agost de 2013, a les 07:18:29, Kevin Ottens va escriure: > On Saturday 03 August 2013 16:29:27 Albert Astals Cid wrote: > > El Divendres, 2 d'agost de 2013, a les 20:49:02, Kevin Ottens va escriure: > > > On Friday 02 August 2013 19:15:50 Albert Astals Cid wrote: > > > > El Divendre

Re: Ki18n mostly ready

2013-08-05 Thread Sebastian Kügler
Hi Chusslove, On Friday, August 02, 2013 12:00:50 Chusslove Illich wrote: > > [: David Faure :] > > There's no way to keep the [insertCatalog] method for source compatibility > > and implement it somehow on top of the new solution? > > The problem is that insertCatalog no longer has a meaning. It

Re: Ki18n mostly ready

2013-08-04 Thread Kevin Ottens
On Saturday 03 August 2013 16:29:27 Albert Astals Cid wrote: > El Divendres, 2 d'agost de 2013, a les 20:49:02, Kevin Ottens va escriure: > > On Friday 02 August 2013 19:15:50 Albert Astals Cid wrote: > > > El Divendres, 2 d'agost de 2013, a les 07:55:52, Kevin Ottens va escriure: > > > > Well, th

Re: Ki18n mostly ready

2013-08-03 Thread Albert Astals Cid
El Divendres, 2 d'agost de 2013, a les 20:49:02, Kevin Ottens va escriure: > On Friday 02 August 2013 19:15:50 Albert Astals Cid wrote: > > El Divendres, 2 d'agost de 2013, a les 07:55:52, Kevin Ottens va escriure: > > > Well, the default has to make sense to someone who just makes a Qt > > > appli

Re: Ki18n mostly ready

2013-08-02 Thread Kevin Ottens
On Friday 02 August 2013 19:15:50 Albert Astals Cid wrote: > El Divendres, 2 d'agost de 2013, a les 07:55:52, Kevin Ottens va escriure: > > Well, the default has to make sense to someone who just makes a Qt > > application and use KConfig as an extra. If kconfig_compiler generates by > > default so

Re: Ki18n mostly ready

2013-08-02 Thread Albert Astals Cid
El Divendres, 2 d'agost de 2013, a les 07:55:52, Kevin Ottens va escriure: > On Thursday 01 August 2013 22:03:36 Albert Astals Cid wrote: > > El Dijous, 1 d'agost de 2013, a les 12:54:07, Kevin Ottens va escriure: > > > Hello, > > > > > > On Thursday 01 August 2013 11:58:44 Chusslove Illich wrote:

Re: Ki18n mostly ready

2013-08-02 Thread Chusslove Illich
> [: David Faure :] > There's no way to keep the [insertCatalog] method for source compatibility > and implement it somehow on top of the new solution? The problem is that insertCatalog no longer has a meaning. Its purpose was to add more translation catalogs into the current process, and all of t

Re: Ki18n mostly ready

2013-08-02 Thread David Faure
On Wednesday 31 July 2013 19:58:52 Chusslove Illich wrote: > /** > - * Inserts the catalog in the main locale object if it exists. > - * Otherwise the catalog name is stored and added once the main locale > gets created - * @since 4.6 > - * @deprecated since 5.0, use KLocalized

Re: Ki18n mostly ready

2013-08-01 Thread Kevin Ottens
On Thursday 01 August 2013 22:03:36 Albert Astals Cid wrote: > El Dijous, 1 d'agost de 2013, a les 12:54:07, Kevin Ottens va escriure: > > Hello, > > > > On Thursday 01 August 2013 11:58:44 Chusslove Illich wrote: > > > > [: Kevin Ottens :] > > > > What's needed for kconfig_compiler? Because curre

Re: Ki18n mostly ready

2013-08-01 Thread Albert Astals Cid
El Dijous, 1 d'agost de 2013, a les 12:54:07, Kevin Ottens va escriure: > Hello, > > On Thursday 01 August 2013 11:58:44 Chusslove Illich wrote: > > > [: Kevin Ottens :] > > > What's needed for kconfig_compiler? Because currently kconfig isn't > > > supposed to depend on ki18n at all. > > > > It

Re: Ki18n mostly ready

2013-08-01 Thread Kevin Ottens
Hello, On Thursday 01 August 2013 11:58:44 Chusslove Illich wrote: > > [: Kevin Ottens :] > > What's needed for kconfig_compiler? Because currently kconfig isn't > > supposed to depend on ki18n at all. > > It does generate translation calls as necessary, and currently it accepts an > option (from

Re: Ki18n mostly ready

2013-08-01 Thread Chusslove Illich
> [: Kevin Ottens :] > What's needed for kconfig_compiler? Because currently kconfig isn't > supposed to depend on ki18n at all. It does generate translation calls as necessary, and currently it accepts an option (from .kcfgc file) whether to generate tr or i18n calls. So another option to specify

Re: Ki18n mostly ready

2013-07-31 Thread Kevin Ottens
Hello, On Wednesday 31 July 2013 19:58:52 Chusslove Illich wrote: > Ki18n modifications for KF5 are now roughly completed, in the sense that it > behaves according to the new spec, the frameworks branch compiles, and the > existing unit tests pass (those still applicable). Things left to do > incl