[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [bug-gettext] [PATCH] format-kde: Recognize KUIT markup

From: Daiki Ueno
Subject: Re: [bug-gettext] [PATCH] format-kde: Recognize KUIT markup
Date: Mon, 09 Mar 2015 18:37:11 +0900
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux)

Chusslove Illich <address@hidden> writes:

> I've tested the patched xgettext on a number of KDE software modules,
> using the full ensemble of --flag options, and everything seems to
> work as expected. So, at the moment, I have nothing to add or change.

Thanks for testing.  Pushed as:

For reference, changes from v1 are:
- split KUIT handling from format-kde.c to format-kde-kuit.c
- add documentation
- remove parser->is_unlikely_intentional callback, which turned out not

> While testing, some messages did still end up without flags, but that was
> due to a separate aspect I wrote about earlier:
> https://lists.gnu.org/archive/html/bug-gnu-utils/2009-04/msg00010.html

Yes, the --kde option is only effective for C++:

>> [...] and maintain it in gettext, since it is you who invented the stuff
>> in the KDE side :-)
> Sure, I'd be happy to.
> But this doesn't include adding and keeping up-to-date a bunch of default
> keywords, does it? :)

I'm not strongly for or against that.  I think this could be done by
adding a bunch of xgettext_record_flag calls to x-c.c:init_flag_table_c.
If they are useful, could you propose a patch?

>> Also it might look a bit surprising that the 3rd and 4th string is marked
>> as kde-format, even if it doesn't contain a format directive. However,
>> this is the original behavior, and should be addressed separately.
> Well, it was the point that all strings get one of two kde-* format flags,
> so I'm not sure I understand the remark. I mean, the behavior is surprising
> from the point of view of common Gettext usage, but for this kind of usage
> (format directives always processed), it is in accordance with that passage
> from the Gettext manual I cited.

Ah, right.  I now see that by re-reading your mail.  Perhaps the
documentation could be a bit clearer, by telling that the --flag option
and the "xgettext:" comment have the same effect.

Daiki Ueno

reply via email to

[Prev in Thread] Current Thread [Next in Thread]