You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Arfrever Frehtes Taifersar Arahesis <ar...@gmail.com> on 2007/08/30 17:42:13 UTC
[PATCH] "." and "\n" not worth translating
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hello,
I was updating Polish translation and I found 2 funny strings marked for translation in subversion/svnsync/main.c:
"." in line 744
"\n" in line 848
I think that these strings aren't worth translating.
[[[
* subversion/svnsync/main.c: Don't translate "." and "\n".
Patch by: Arfrever Frehtes Taifersar Arahesis <ar...@gmail.com>
]]]
Index: subversion/svnsync/main.c
===================================================================
- --- subversion/svnsync/main.c (revision 26392)
+++ subversion/svnsync/main.c (working copy)
@@ -741,7 +741,7 @@
node_baton_t *fb = file_baton;
edit_baton_t *eb = fb->edit_baton;
- - SVN_ERR(svn_cmdline_printf(pool, _(".")));
+ SVN_ERR(svn_cmdline_printf(pool, "."));
SVN_ERR(svn_cmdline_fflush(stdout));
return eb->wrapped_editor->apply_textdelta(fb->wrapped_node_baton,
@@ -845,7 +845,7 @@
SVN_ERR(eb->wrapped_editor->close_directory(baton, pool));
}
- - SVN_ERR(svn_cmdline_printf(pool, _("\n")));
+ SVN_ERR(svn_cmdline_printf(pool, "\n"));
return eb->wrapped_editor->close_edit(eb->wrapped_edit_baton, pool);
}
- --
Arfrever Frehtes Taifersar Arahesis
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.6 (GNU/Linux)
iD8DBQFG1wHM/axNJ4Xo/ZERArvfAJ4tTgEiphKEnnR1+Ehd6oY4xLVfWwCdERQX
A5w/iBomstghYe8jBJPwaC0=
=sztR
-----END PGP SIGNATURE-----
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] "." and "\n" not worth translating
Posted by Daniel Rall <dl...@collab.net>.
On Thu, 30 Aug 2007, Daniel Rall wrote:
>
> On Aug 30, 2007, at 10:42 AM, Arfrever Frehtes Taifersar Arahesis wrote:
>
> >-----BEGIN PGP SIGNED MESSAGE-----
> >Hash: SHA1
> >
> >Hello,
> >I was updating Polish translation and I found 2 funny strings
> >marked for translation in subversion/svnsync/main.c:
> >"." in line 744
> >"\n" in line 848
> >I think that these strings aren't worth translating.
>
> Definitely, nice catch -- non-localization is consistent with the
> behavior we have in the 'svn' command-line client (in response to
> notifications).
Oh, I neglected to mention that I committed your patch in r26401.
Thanks again,
- Dan
Re: [PATCH] "." and "\n" not worth translating
Posted by Daniel Rall <dl...@finemaltcoding.com>.
On Aug 30, 2007, at 10:42 AM, Arfrever Frehtes Taifersar Arahesis wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hello,
> I was updating Polish translation and I found 2 funny strings
> marked for translation in subversion/svnsync/main.c:
> "." in line 744
> "\n" in line 848
> I think that these strings aren't worth translating.
Definitely, nice catch -- non-localization is consistent with the
behavior we have in the 'svn' command-line client (in response to
notifications).
> [[[
> * subversion/svnsync/main.c: Don't translate "." and "\n".
>
> Patch by: Arfrever Frehtes Taifersar Arahesis <ar...@gmail.com>
> ]]]
Thanks for including a log message! In the future, please include
the symbol names of the functions changed by your patch. :-)
For some reason, this patch was malformed in a couple spots:
> Index: subversion/svnsync/main.c
> ===================================================================
> - --- subversion/svnsync/main.c (revision 26392)
> +++ subversion/svnsync/main.c (working copy)
> @@ -741,7 +741,7 @@
> node_baton_t *fb = file_baton;
> edit_baton_t *eb = fb->edit_baton;
>
> - - SVN_ERR(svn_cmdline_printf(pool, _(".")));
...here...
> + SVN_ERR(svn_cmdline_printf(pool, "."));
> SVN_ERR(svn_cmdline_fflush(stdout));
>
> return eb->wrapped_editor->apply_textdelta(fb->wrapped_node_baton,
> @@ -845,7 +845,7 @@
> SVN_ERR(eb->wrapped_editor->close_directory(baton, pool));
> }
>
> - - SVN_ERR(svn_cmdline_printf(pool, _("\n")));
...and here.
> + SVN_ERR(svn_cmdline_printf(pool, "\n"));
>
> return eb->wrapped_editor->close_edit(eb->wrapped_edit_baton,
> pool);
> }
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org