[PINENTRY PATCH] tty button-related fixes and refinements
NIIBE Yutaka
gniibe at fsij.org
Mon Oct 3 09:06:29 CEST 2016
On 09/27/2016 01:02 PM, Ineiev wrote:
> On Fri, Apr 08, 2016 at 01:47:38PM -0400, Ineiev wrote:
>>
>> I attach some fixes I've come up with when playing with non-ASCII
>> texts in buttons.
>>
>> The first one simplifies key checking code a little bit; the second
>> one fixes a few bugs in underscore processing; the third one lets
>> the user see the localized message even if the pinentry doesn't
>> support its accelerators (by the way, I think that pinentry
>> should localize the messages that aren't customized from the agent,
>> like "Press any key to continue."); the last one adds a default
>> message to a call of button() so that pinentry won't skip a button
>> when its text is localized.
>
> I've fixed a few typos in comments and log messages.
Thank you for your fixes.
I applied three bug fixes, with commit message edit by me.
0001-tty-Refactor-usage-of-tolower.patch
0002-tty-Fix-underscore-processing-in-accelerators.patch
0004-tty-Avoid-using-button-with-NULL-default-text.patch
For 0001, I modified your change a bit to keep the output for input
key is as same as before.
The patch:
0003-tty-Show-supplied-message-when-using-default.patch
is introducing a new feature. Let me have (more) time to review this
closely.
--
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: OpenPGP digital signature
URL: </pipermail/attachments/20161003/1f0e5317/attachment.sig>
More information about the Gnupg-devel
mailing list