From c2443c74d464b1824804f1dca112da6976864142 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=B4me=20Rapinat?= <romjerome@yahoo.fr>
Date: Thu, 13 Dec 2012 20:43:32 +0000
Subject: [PATCH] update and add more comments

svn: r20782
---
 po/check_po     |  4 ++--
 po/update_po.py | 39 +++++++++++++++++++++++++--------------
 2 files changed, 27 insertions(+), 16 deletions(-)

diff --git a/po/check_po b/po/check_po
index 47a3e4d27..aca83b53c 100755
--- a/po/check_po
+++ b/po/check_po
@@ -548,6 +548,7 @@ def analyze_msgs( args, fname, msgs, nr_templates = None, nth = 0 ):
 
 	for c in checks:
 		c.summary()
+		c.diag()
 
 	po_coverage = (1.0 - (float(nr_untranslated) / float(nr_msgs))) * 100
 	print "%-20s%5.2f%%" % ( "PO Coverage:",       po_coverage )
@@ -564,14 +565,13 @@ def analyze_msgs( args, fname, msgs, nr_templates = None, nth = 0 ):
 	else:
 		print text + ' (previous gramps.pot)'
 
-
 def main():
     
     parser = ArgumentParser( description='This program validates a PO file for GRAMPS.')
                  
     parser.add_argument("-s", dest="summary",
               choices=[file for file in os.listdir('.') if file.endswith('.po')], 
-              default=False, help="only give the summary")
+              default=False, help="the summary of check, and if need, it gives details")
               
     args = parser.parse_args()
     
diff --git a/po/update_po.py b/po/update_po.py
index ae0503e66..cda442aa3 100644
--- a/po/update_po.py
+++ b/po/update_po.py
@@ -52,6 +52,7 @@ import os
 import sys
 from argparse import ArgumentParser
 
+# Windows OS
 
 if sys.platform == 'win32':          
     # GetText Win 32 obtained from http://gnuwin32.sourceforge.net/packages/gettext.htm
@@ -61,6 +62,9 @@ if sys.platform == 'win32':
     msgattribCmd = os.path.join('C:', 'Program Files(x86)', 'gettext', 'bin', 'msgattrib.exe')
     xgettextCmd = os.path.join('C:', 'Program Files(x86)', 'gettext', 'bin', 'xgettext.exe')
     pythonCmd = os.path.join(sys.prefix, 'bin', 'python.exe')
+
+# Others OS
+
 elif sys.platform == 'linux2' or os.name == 'darwin':
     msgmergeCmd = 'msgmerge'
     msgfmtCmd = 'msgfmt'
@@ -70,9 +74,14 @@ elif sys.platform == 'linux2' or os.name == 'darwin':
 else:
     print ("ERROR: unknown system, don't know msgmerge, ... commands")
     sys.exit(0)
-    
+
+# List of available languages, useful for grouped actions
+
+# need files with po extension
 LANG = [file for file in os.listdir('.') if file.endswith('.po')]
+# add a special 'all' argument (for 'check' and 'merge' arguments)
 LANG.append("all")
+# visual polish on the languages list
 LANG.sort()
 
 def tests():
@@ -116,7 +125,7 @@ def tests():
     except:
         print ('Please, install python')
         
-# See also 'get_string' from Gramps 2.0 (sample with SAX)
+
 def TipsParse(filename, mark):
     """
     Experimental alternative to 'intltool-extract' for 'tips.xml'.
@@ -205,6 +214,7 @@ def HolidaysParse(filename, mark):
         if key.attrib.get(mark):
             line = key.attrib
             string = line.items
+            # mapping via the line dict (_name is the key)
             name = 'char *s = N_("%(_name)s");\n' % line
             holidays.write(name)
     holidays.close()
@@ -298,19 +308,19 @@ def KeyParse(filename, mark):
     
     '''
     application/x-gramps-xml:
-		_description=Gramps XML database
-		default_action_type=application
-		short_list_application_ids=gramps
-		short_list_application_ids_for_novice_user_level=gramps
-		short_list_application_ids_for_intermediate_user_level=gramps
-		short_list_application_ids_for_advanced_user_level=gramps
-		category=Documents/Genealogy
-		icon-filename=/usr/share/gramps/gramps.png
-		open=gramps %f
+    _description=Gramps XML database
+    default_action_type=application
+    short_list_application_ids=gramps
+    short_list_application_ids_for_novice_user_level=gramps
+    short_list_application_ids_for_intermediate_user_level=gramps
+    short_list_application_ids_for_advanced_user_level=gramps
+    category=Documents/Genealogy
+    icon-filename=/usr/share/gramps/gramps.png
+    open=gramps %f
 
     application/x-gedcom:
-		_description=GEDCOM
-		default_action_type=application
+    _description=GEDCOM
+    default_action_type=application
     
     msgid "Gramps XML database"
     msgid "GEDCOM"
@@ -394,7 +404,6 @@ def main():
     
     args = parser.parse_args()
     namespace, extra = parser.parse_known_args()
-    #print(args, '\n\t\t###\n', vars(args), '\n\t\t###\n', sys.argv[2:])
 
     if args.test:
         tests()
@@ -415,11 +424,13 @@ def main():
         clean()
         
     if args.merge:
+        #retrieve() windows os?
         if sys.argv[2:] == ['all']:
             sys.argv[2:] = LANG
         merge(sys.argv[2:])
         
     if args.check:
+        #retrieve() windows os?
         if sys.argv[2:] == ['all']:
             sys.argv[2:] = LANG
         check(sys.argv[2:])