I agree with all of Simon's comments.<br><br>cjl<br><br><div class="gmail_quote">On Thu, Feb 9, 2012 at 4:18 AM, Simon Schampijer <span dir="ltr"><<a href="mailto:simon@schampijer.de">simon@schampijer.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 01/23/2012 02:48 PM, Ajay Garg wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
<br>
Removed unncessary "gettext" notations; plus added "TRANS" labels for not<br>
translating format-specifier tokens.<br>
<br>
Thanks Chris Leonard.<br>
<br>
  extensions/cpsection/<u></u>aboutcomputer/model.py |    2 +-<br>
  src/jarabe/desktop/<u></u>favoritesview.py         |    2 +-<br>
  src/jarabe/frame/<u></u>activitiestray.py          |    7 +++++--<br>
  3 files changed, 7 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/extensions/cpsection/<u></u>aboutcomputer/model.py b/extensions/cpsection/<u></u>aboutcomputer/model.py<br>
index 431c9c0..86d2e3f 100644<br>
--- a/extensions/cpsection/<u></u>aboutcomputer/model.py<br>
+++ b/extensions/cpsection/<u></u>aboutcomputer/model.py<br>
@@ -180,7 +180,7 @@ def get_wireless_firmware():<br>
      if len(firmware_info) == 1:<br>
          return firmware_info.values()[0]<br>
<br>
-    return ', '.join([_('%(interface)s: %(version)s') %<br>
+    return ', '.join(['%(interface)s: %(version)s' %<br>
                        {'interface': interface, 'version': version}<br>
                        for interface, version in firmware_info.items()])<br>
</blockquote>
<br></div>
This one looks good.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/src/jarabe/desktop/<u></u>favoritesview.py b/src/jarabe/desktop/<u></u>favoritesview.py<br>
index 132d14f..5c76cdc 100644<br>
--- a/src/jarabe/desktop/<u></u>favoritesview.py<br>
+++ b/src/jarabe/desktop/<u></u>favoritesview.py<br>
@@ -336,7 +336,7 @@ class FavoritesView(hippo.Canvas):<br>
              schoolserver.register_laptop()<br>
          except RegisterError, e:<br>
              alert.props.title = _('Registration Failed')<br>
-            alert.props.msg = _('%s') % e<br>
+            alert.props.msg = '%s' % e<br>
          else:<br>
              alert.props.title = _('Registration Successful')<br>
              alert.props.msg = _('You are now registered ' \<br>
</blockquote>
<br></div>
This one looks good as well.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/src/jarabe/frame/<u></u>activitiestray.py b/src/jarabe/frame/<u></u>activitiestray.py<br>
index 941b174..2b8b7c6 100644<br>
--- a/src/jarabe/frame/<u></u>activitiestray.py<br>
+++ b/src/jarabe/frame/<u></u>activitiestray.py<br>
@@ -541,6 +541,7 @@ class BaseTransferPalette(Palette):<br>
          transferred = self._format_size(<br>
                  self.file_transfer.props.<u></u>transferred_bytes)<br>
          total = self._format_size(self.file_<u></u>transfer.file_size)<br>
+        # TRANS: do not translate the two "%s".<br>
          self.progress_label.props.<u></u>label = _('%s of %s') % (transferred, total)<br>
</blockquote>
<br></div>
If I would add a comment here I would be verbose about the context. For example:<br>
<br>
# TRANS: file transfer, bytes transferred, e.g. 128 of 1024<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -555,6 +556,7 @@ class IncomingTransferPalette(<u></u>BaseTransferPalette):<br>
          self.file_transfer.connect('<u></u>notify::state', self.__notify_state_cb)<br>
<br>
          nick = str(self.file_transfer.buddy.<u></u>props.nick)<br>
+        # TRANS: do not translate the token "%s".<br>
          label = glib.markup_escape_text(_('<u></u>Transfer from %s') % (nick,))<br>
          self.props.secondary_text = label<br>
</blockquote>
<br></div>
No need to add an extra trans information here. I think translators know how to deal with this one.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -589,7 +591,7 @@ class IncomingTransferPalette(<u></u>BaseTransferPalette):<br>
              type_description = mime.get_mime_description(<u></u>mime_type)<br>
<br>
              size = self._format_size(self.file_<u></u>transfer.file_size)<br>
-            label = gtk.Label(_('%s (%s)') % (size, type_description))<br>
+            label = gtk.Label('%s (%s)' % (size, type_description))<br>
              vbox.add(label)<br>
              label.show()<br>
</blockquote>
<br></div>
This one looks good.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -688,6 +690,7 @@ class OutgoingTransferPalette(<u></u>BaseTransferPalette):<br>
          self.file_transfer.connect('<u></u>notify::state', self.__notify_state_cb)<br>
<br>
          nick = str(file_transfer.buddy.props.<u></u>nick)<br>
+        # TRANS: do not translate "%s".<br>
          label = glib.markup_escape_text(_('<u></u>Transfer to %s') % (nick,))<br>
          self.props.secondary_text = label<br>
</blockquote>
<br></div>
Same as above: No need to add an extra trans information here. I think translators know how to deal with this one.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -719,7 +722,7 @@ class OutgoingTransferPalette(<u></u>BaseTransferPalette):<br>
              type_description = mime.get_mime_description(<u></u>mime_type)<br>
<br>
              size = self._format_size(self.file_<u></u>transfer.file_size)<br>
-            label = gtk.Label(_('%s (%s)') % (size, type_description))<br>
+            label = gtk.Label('%s (%s)' % (size, type_description))<br>
              vbox.add(label)<br>
              label.show()<br>
</blockquote>
<br></div>
This one looks good.<br>
<br>
Regards,<br>
   Simon<div class="HOEnZb"><div class="h5"><br>
______________________________<u></u>_________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.<u></u>org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/<u></u>listinfo/sugar-devel</a><br>
</div></div></blockquote></div><br>