From e3fad2d08b1cdd238f04bc41f7cea76194d052ed Mon Sep 17 00:00:00 2001
From: cylon <cylon@openmailbox.org>
Date: Sun, 25 Sep 2016 12:16:58 +0200
Subject: [PATCH] Applied pylint on converter.py

---
 converter.py | 75 ++++++++++++++++++++++++++--------------------------
 1 file changed, 38 insertions(+), 37 deletions(-)

diff --git a/converter.py b/converter.py
index 34314a0..cf81ed6 100644
--- a/converter.py
+++ b/converter.py
@@ -1,61 +1,62 @@
 # -*- coding: utf-8 -*-
+
+""" Entry point for project conversion
+    @file
+"""
+
 import os
 import argparse
 import cmake
 import ewpproject
 import uvprojxproject
 
-def findFile (path,fileext):
-    
-    f = ''
+def find_file(path, fileext):
+    """ Find file with extension in path
+        @param path Root path of the project
+        @param fileext File extension to find
+        @return File name
+    """
+    filename = ''
     for root, dirs, files in os.walk(path):
         for file in files:
             if file.endswith(fileext):
-                f = os.path.join(root, file)
-    
-    return f
-             
+                file = os.path.join(root, filename)
+    return filename
+
 if __name__ == '__main__':
-    
+    """ Parses params and calls the right conversion"""
+
     parser = argparse.ArgumentParser()
-    parser.add_argument("path", type=str, help="Root directory of project")    
-    parser.add_argument("--ewp",help="Search for *.EWP file in project structure",action='store_true')    
-    parser.add_argument("--uvprojx",help="Search for *.UPROJX file in project structure",action='store_true')
-    
+    parser.add_argument("path", type=str, help="Root directory of project")
+    parser.add_argument("--ewp", help="Search for *.EWP file in project structure", action='store_true')
+    parser.add_argument("--uvprojx", help="Search for *.UPROJX file in project structure", action='store_true')
     args = parser.parse_args()
-        
-    if (os.path.isdir (args.path)):
-        
+
+    if os.path.isdir(args.path):
         if args.ewp:
-            print ('Looking for *.ewp file in ' + args.path)
-            file = findFile(args.path,'.ewp')
-            if (len(file)):
-                print ('Found project file: ' + file)
-                project = ewpproject.EWPProject(args.path,file)
+            print('Looking for *.ewp file in ' + args.path)
+            filename = find_file(args.path, '.ewp')
+            if len(filename):
+                print('Found project file: ' + filename)
+                project = ewpproject.EWPProject(args.path, filename)
                 project.parseProject()
                 project.displaySummary()
-                
-                cmakefile = cmake.CMake(project.getProject(),args.path)
+                cmakefile = cmake.CMake(project.getProject(), args.path)
                 cmakefile.populateCMake()
-                
             else:
-                print ('No project *.ewp file found')
-                
+                print('No project *.ewp file found')
         elif args.uvprojx:
-            print ('Looking for *.uvprojx file in ' + args.path)
-            file = findFile(args.path,'.uvprojx')
-            if (len(file)):
-                print ('Found project file: ' + file)
-                project = uvprojxproject.UVPROJXProject(args.path,file)
+            print('Looking for *.uvprojx file in ' + args.path)
+            filename = find_file(args.path, '.uvprojx')
+            if len(filename):
+                print('Found project file: ' + filename)
+                project = uvprojxproject.UVPROJXProject(args.path, filename)
                 project.parseProject()
                 project.displaySummary()
-                
-                cmakefile = cmake.CMake(project.getProject(),args.path)
+
+                cmakefile = cmake.CMake(project.getProject(), args.path)
                 cmakefile.populateCMake()
-                
             else:
-                print ('No project *.uvprojx file found')
+                print('No project *.uvprojx file found')
     else:
-
-        print ('Not a valid file path')               
-
+        print('Not a valid file path')