All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Fixed translation error (say(_())) functions
@ 2015-05-03 22:05 Alangi Derick
  2015-05-04 15:19 ` Matthieu Moy
  0 siblings, 1 reply; 2+ messages in thread
From: Alangi Derick @ 2015-05-03 22:05 UTC (permalink / raw)


Signed-off-by: Alangi Derick <alangiderick@gmail.com>
---
 base85.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/base85.c b/base85.c
index 5ca601e..4160f44 100644
--- a/base85.c
+++ b/base85.c
@@ -78,7 +78,7 @@ int decode_85(char *dst, const char *buffer, int len)
 
 void encode_85(char *buf, const unsigned char *data, int bytes)
 {
-	say("encode 85");
+	say(_("encode 85"));
 	while (bytes) {
 		unsigned acc = 0;
 		int cnt;
@@ -96,7 +96,7 @@ void encode_85(char *buf, const unsigned char *data, int bytes)
 		}
 		buf += 5;
 	}
-	say("\n");
+	say(_("\n"));
 
 	*buf = 0;
 }
-- 
2.4.0.1.gd092831.dirty

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Fixed translation error (say(_())) functions
  2015-05-03 22:05 [PATCH] Fixed translation error (say(_())) functions Alangi Derick
@ 2015-05-04 15:19 ` Matthieu Moy
  0 siblings, 0 replies; 2+ messages in thread
From: Matthieu Moy @ 2015-05-04 15:19 UTC (permalink / raw)
  To: Alangi Derick; +Cc: git

It seems you are posting putting the list in Bcc. Don't do that: it
makes it painful to reply to your message (I had to re-add git@vger
manually).

Alangi Derick <alangiderick@gmail.com> writes:

> Subject: [PATCH] Fixed translation error (say(_())) functions

Please, read the "(2) Describe your changes well." section of
SubmittingPatches (actually, read the whole document if not done
already, but particularly this section). Your subject does not adhere
the usual format.

> Signed-off-by: Alangi Derick <alangiderick@gmail.com>
> ---
>  base85.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/base85.c b/base85.c
> index 5ca601e..4160f44 100644
> --- a/base85.c
> +++ b/base85.c
> @@ -78,7 +78,7 @@ int decode_85(char *dst, const char *buffer, int len)
>  
>  void encode_85(char *buf, const unsigned char *data, int bytes)
>  {
> -	say("encode 85");
> +	say(_("encode 85"));

These are debug statements (see the top of base85.c). I don't think you
want to translate them.

> @@ -96,7 +96,7 @@ void encode_85(char *buf, const unsigned char *data, int bytes)
>  		}
>  		buf += 5;
>  	}
> -	say("\n");
> +	say(_("\n"));

Same, plus the fact that a \n would not need to be translated anyway I
guess.

Regards,

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-05-04 15:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-03 22:05 [PATCH] Fixed translation error (say(_())) functions Alangi Derick
2015-05-04 15:19 ` Matthieu Moy

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.