Re: [PATCH] Fix translation of strings in foreach lists.

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

 



Quoting "Shawn O. Pearce" <spearce@xxxxxxxxxxx>:
>I not sure if it's worth changing, but you don't necessarily need
>those [append]'s in there.

Thanks for the info. It is basically up to Shawn to decide which style
he likes more.

"[mc Foo]\n\n$err"

is much shorter.  So I'd prefer that over [append [mc Foo] "\n\n$err"].

Err, but the usual case is that the argument to [mc] is more than one word, and (additionally) that argument has to be quoted by double-quotes (as opposed to braces) because otherwise xgettext doesn't extract the message correctly. Hence, the only possibility here were

  "[mc "Some Text"]\n\n$err"

Is this valid Tcl? Being a newcomer to this language I have to says the quotes here look rather strange :-)

Its not like you've saved a dq-eval either, as you still have to do one
for the \n\n.  But either way is valid Tcl, and works, and the append
version is not so bad that I'd reject a patch that used it.

Ok, thanks.

Christian
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux