]> git.proxmox.com Git - mirror_edk2.git/commitdiff
enhance ModuleReader
authoralfred <alfred@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 22 Aug 2006 08:47:40 +0000 (08:47 +0000)
committeralfred <alfred@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 22 Aug 2006 08:47:40 +0000 (08:47 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@1350 6f19259b-4bc3-4df7-8a09-765794883524

Tools/Source/MigrationTools/org/tianocore/migration/FirstPanel.java
Tools/Source/MigrationTools/org/tianocore/migration/MigrationTool.java
Tools/Source/MigrationTools/org/tianocore/migration/ModuleInfo.java
Tools/Source/MigrationTools/org/tianocore/migration/ModuleReader.java

index 8c428087d669283ac0aa5870697c62ede84ef008..de8e75f14290d504f99118e24f904fda4fa9938f 100644 (file)
@@ -59,12 +59,18 @@ public final class FirstPanel extends JPanel implements ActionListener, ItemList
                \r
                filebox = new JCheckBox("Output to logfile", true);\r
                filebox.addItemListener(this);\r
                \r
                filebox = new JCheckBox("Output to logfile", true);\r
                filebox.addItemListener(this);\r
+               \r
                screenbox = new JCheckBox("Specify logfile", false);\r
                screenbox.addItemListener(this);\r
                screenbox = new JCheckBox("Specify logfile", false);\r
                screenbox.addItemListener(this);\r
-               mibox = new JCheckBox("Show ModuleInfo", false);\r
+               \r
+               mibox = new JCheckBox("Print ModuleInfo", false);\r
                mibox.addItemListener(this);\r
                mibox.addItemListener(this);\r
+               MigrationTool.printModuleInfo = false;\r
+               \r
                criticbox = new JCheckBox("Run Critic", true);\r
                criticbox.addItemListener(this);\r
                criticbox = new JCheckBox("Run Critic", true);\r
                criticbox.addItemListener(this);\r
+               MigrationTool.doCritic = true;\r
+               \r
                defaultpathbox = new JCheckBox("Use Default Output Path", true);\r
                defaultpathbox.addItemListener(this);\r
                \r
                defaultpathbox = new JCheckBox("Use Default Output Path", true);\r
                defaultpathbox.addItemListener(this);\r
                \r
@@ -205,14 +211,16 @@ public final class FirstPanel extends JPanel implements ActionListener, ItemList
                }\r
        } else if (e.getSource() == mibox) {\r
                if (e.getStateChange() == ItemEvent.DESELECTED) {\r
                }\r
        } else if (e.getSource() == mibox) {\r
                if (e.getStateChange() == ItemEvent.DESELECTED) {\r
-                       System.out.println("mibox DESELECTED");\r
+                       MigrationTool.printModuleInfo = false;\r
                } else if (e.getStateChange() == ItemEvent.SELECTED) {\r
                } else if (e.getStateChange() == ItemEvent.SELECTED) {\r
-                       System.out.println("mibox SELECTED");\r
+                       MigrationTool.printModuleInfo = true;\r
                }\r
        } else if (e.getSource() == criticbox) {\r
                if (e.getStateChange() == ItemEvent.DESELECTED) {\r
                }\r
        } else if (e.getSource() == criticbox) {\r
                if (e.getStateChange() == ItemEvent.DESELECTED) {\r
+                       MigrationTool.doCritic = false;\r
                        System.out.println("criticbox DESELECTED");\r
                } else if (e.getStateChange() == ItemEvent.SELECTED) {\r
                        System.out.println("criticbox DESELECTED");\r
                } else if (e.getStateChange() == ItemEvent.SELECTED) {\r
+                       MigrationTool.doCritic = true;\r
                        System.out.println("criticbox SELECTED");\r
                }\r
        } else if (e.getSource() == defaultpathbox) {\r
                        System.out.println("criticbox SELECTED");\r
                }\r
        } else if (e.getSource() == defaultpathbox) {\r
index d6119e6e45eee3634e03600dd77ee5614bb171b4..9448acf76617d93cd6b90035deec08a287596ef6 100644 (file)
@@ -16,6 +16,11 @@ public class MigrationTool {
        public static UI ui = null;\r
        public static Database db = null;\r
        \r
        public static UI ui = null;\r
        public static Database db = null;\r
        \r
+       public static final String migrationcomment = "//%$//";\r
+       \r
+       public static boolean printModuleInfo = false;\r
+       public static boolean doCritic = false;\r
+       \r
        public static void main(String[] args) throws Exception {\r
                ui = FirstPanel.init();\r
                db = Database.init();\r
        public static void main(String[] args) throws Exception {\r
                ui = FirstPanel.init();\r
                db = Database.init();\r
index 4d59a9e953bc264d88d813795326398395fd8b60..aa6e7a2ba0bac74ca5ade28c5768e760e49c5199 100644 (file)
@@ -14,7 +14,6 @@ package org.tianocore.migration;
 \r
 import java.io.*;\r
 import java.util.*;\r
 \r
 import java.io.*;\r
 import java.util.*;\r
-import java.util.regex.*;\r
 \r
 /*\r
        Class ModuleInfo is built for scanning the source files, it contains all the needed\r
 \r
 /*\r
        Class ModuleInfo is built for scanning the source files, it contains all the needed\r
@@ -30,12 +29,9 @@ public class ModuleInfo {
                }\r
                MigrationTool.ui.println(outputpath);\r
                \r
                }\r
                MigrationTool.ui.println(outputpath);\r
                \r
-               moduleScan();\r
+               mainFlow();\r
        }\r
 \r
        }\r
 \r
-       //public static UI ui = null;                           //if MIM is still usefull, this can be given to it\r
-       public static Database db = null;                       //if MIM is still usefull, this can be given to it\r
-       \r
        public String modulepath = null;\r
        \r
        public String outputpath = null;\r
        public String modulepath = null;\r
        \r
        public String outputpath = null;\r
@@ -61,32 +57,14 @@ public class ModuleInfo {
        public Set<String> protocol = new HashSet<String>();\r
        public Set<String> ppi = new HashSet<String>();\r
        \r
        public Set<String> protocol = new HashSet<String>();\r
        public Set<String> ppi = new HashSet<String>();\r
        \r
-       private static String migrationcomment = "//%$//";\r
-       \r
-       private void moduleScan() throws Exception {\r
-               Common.toDoAll(modulepath, ModuleInfo.class.getMethod("enroll", String.class), this, null, Common.FILE);\r
+       private void mainFlow() throws Exception {\r
                \r
                \r
-               String filename = null;\r
-               if (msaorinf.isEmpty()) {\r
-                       MigrationTool.ui.println("No INF nor MSA file found!");\r
-                       System.exit(0);\r
-               } else {\r
-                       filename = MigrationTool.ui.choose("Found .inf or .msa file for module\n" + modulepath + "\nChoose one Please", msaorinf.toArray());\r
-               }\r
-               //ModuleReader mr = new ModuleReader(modulepath, this, db, ui);\r
-               if (filename.contains(".inf")) {\r
-                       ModuleReader.readInf(filename, this);\r
-               } else if (filename.contains(".msa")) {\r
-                       ModuleReader.readMsa(filename, this);\r
-               }\r
+               ModuleReader.ModuleScan(this);\r
                \r
                \r
-               CommentOutNonLocalHFile();\r
-               parsePreProcessedSourceCode();\r
-\r
                SourceFileReplacer.flush(this); // some adding library actions are taken here,so it must be put before "MsaWriter"\r
                \r
                // show result\r
                SourceFileReplacer.flush(this); // some adding library actions are taken here,so it must be put before "MsaWriter"\r
                \r
                // show result\r
-               if (MigrationTool.ui.yesOrNo("Parse of the Module Information has completed. View details?")) {\r
+               if (MigrationTool.printModuleInfo) {\r
                        MigrationTool.ui.println("\nModule Information : ");\r
                        MigrationTool.ui.println("Entrypoint : " + entrypoint);\r
                        show(protocol, "Protocol : ");\r
                        MigrationTool.ui.println("\nModule Information : ");\r
                        MigrationTool.ui.println("Entrypoint : " + entrypoint);\r
                        show(protocol, "Protocol : ");\r
@@ -102,8 +80,11 @@ public class ModuleInfo {
                \r
                new MsaWriter(this).flush();\r
 \r
                \r
                new MsaWriter(this).flush();\r
 \r
+               if (MigrationTool.doCritic) {\r
+               Critic.fireAt(outputpath + File.separator + "Migration_" + modulename);\r
+               }\r
+               \r
                Common.deleteDir(modulepath + File.separator + "temp");\r
                Common.deleteDir(modulepath + File.separator + "temp");\r
-               //Common.toDoAll(modulepath + File.separator + "temp", Common.class.getMethod("deleteDir", String.class), null, null, Common.DIR);\r
                \r
                MigrationTool.ui.println("Errors Left : " + MigrationTool.db.error);\r
                MigrationTool.ui.println("Complete!");\r
                \r
                MigrationTool.ui.println("Errors Left : " + MigrationTool.db.error);\r
                MigrationTool.ui.println("Complete!");\r
@@ -116,147 +97,6 @@ public class ModuleInfo {
                MigrationTool.ui.println(hash);\r
        }\r
        \r
                MigrationTool.ui.println(hash);\r
        }\r
        \r
-       // add '//' to all non-local include lines\r
-       private void CommentOutNonLocalHFile() throws IOException {\r
-               BufferedReader rd;\r
-               String line;\r
-               String curFile;\r
-               PrintWriter outfile;\r
-\r
-               Pattern ptninclude = Pattern.compile("[\"<](.*[.]h)[\">]");\r
-               Matcher mtrinclude;\r
-\r
-               Iterator<String> ii = localmodulesources.iterator();\r
-               while ( ii.hasNext() ) {\r
-                       curFile = ii.next();\r
-                       rd = new BufferedReader(new FileReader(modulepath + File.separator + curFile));\r
-                       Common.ensureDir(modulepath + File.separator + "temp" + File.separator + curFile);\r
-                       outfile = new PrintWriter(new BufferedWriter(new FileWriter(modulepath + File.separator + "temp" + File.separator + curFile)));\r
-                       while ((line = rd.readLine()) != null) {\r
-                               if (line.contains("#include")) {\r
-                                       mtrinclude = ptninclude.matcher(line);\r
-                                       if (mtrinclude.find() && localmodulesources.contains(mtrinclude.group(1))) {\r
-                                       } else {\r
-                                               line = migrationcomment + line;\r
-                                       }\r
-                               }\r
-                               outfile.append(line + '\n');\r
-                       }\r
-                       outfile.flush();\r
-                       outfile.close();\r
-               }\r
-       }\r
-\r
-       private void parsePreProcessedSourceCode() throws Exception {\r
-               //Cl cl = new Cl(modulepath);\r
-               //cl.execute("Fat.c");\r
-               //cl.generateAll(preprocessedccodes);\r
-               //\r
-               //System.out.println("Note!!!! The CL is not implemented now , pls do it manually!!! RUN :");\r
-               //System.out.println("cl " + modulepath + "\\temp\\*.c" + " -P");\r
-               //String[] list = new File(modulepath + File.separator + "temp").list();        // without CL , add\r
-               BufferedReader rd = null;\r
-               String ifile = null;\r
-               String line = null;\r
-               String temp = null;\r
-               \r
-               Iterator<String> ii = localmodulesources.iterator();\r
-               while (ii.hasNext()) {\r
-                       temp = ii.next();\r
-                       if (temp.contains(".c")) {\r
-                               preprocessedccodes.add(temp);\r
-                       }\r
-               }\r
-               \r
-               ii = preprocessedccodes.iterator();\r
-               \r
-               Pattern patefifuncc = Pattern.compile("g?(BS|RT)\\s*->\\s*([a-zA-Z_]\\w*)",Pattern.MULTILINE);\r
-               Pattern patentrypoint = Pattern.compile("EFI_([A-Z]*)_ENTRY_POINT\\s*\\(([^\\(\\)]*)\\)",Pattern.MULTILINE);\r
-               Matcher matguid;\r
-               Matcher matfuncc;\r
-               Matcher matfuncd;\r
-               Matcher matenclosereplace;\r
-               Matcher matefifuncc;\r
-               Matcher matentrypoint;\r
-               Matcher matmacro;\r
-               \r
-               while (ii.hasNext()) {\r
-                       StringBuffer wholefile = new StringBuffer();\r
-                       ifile = ii.next();\r
-                       rd = new BufferedReader(new FileReader(modulepath + File.separator + "temp" + File.separator + ifile));\r
-                       while ((line = rd.readLine()) != null) {\r
-                               wholefile.append(line + '\n');\r
-                       }\r
-                       line = wholefile.toString();\r
-                       \r
-                       // if this is a Pei phase module , add these library class to .msa\r
-                       matentrypoint = patentrypoint.matcher(line);\r
-                       if (matentrypoint.find()) {\r
-                               entrypoint = matentrypoint.group(2);\r
-                               if (matentrypoint.group(1).matches("PEIM")) {\r
-                                       hashrequiredr9libs.add("PeimEntryPoint");\r
-                               } else {\r
-                                       hashrequiredr9libs.add("UefiDriverEntryPoint");\r
-                               }\r
-                       }\r
-                       \r
-                       // find guid\r
-                       matguid = Guid.ptnguid.matcher(line);                                                                           // several ways to implement this , which one is faster ? :\r
-                       while (matguid.find()) {                                                                                                        // 1.currently , find once , then call to identify which is it\r
-                               if ((temp = Guid.register(matguid, this, db)) != null) {                                // 2.use 3 different matchers , search 3 times to find each\r
-                                       //matguid.appendReplacement(result, MigrationTool.db.getR9Guidname(temp));              // search the database for all 3 kinds of guids , high cost\r
-                               }\r
-                       }\r
-                       //matguid.appendTail(result);\r
-                       //line = result.toString();\r
-\r
-                       // find EFI call in form of '->' , many 'gUnicodeCollationInterface->' like things are not changed\r
-                       // This item is not simply replaced , special operation is required.\r
-                       matefifuncc = patefifuncc.matcher(line);\r
-                       while (matefifuncc.find()) {\r
-                               hashEFIcall.add(matefifuncc.group(2));\r
-                       }\r
-\r
-                       // find function call\r
-                       matfuncc = Func.ptnfuncc.matcher(line);\r
-                       while (matfuncc.find()) {\r
-                               if ((temp = Func.register(matfuncc, this, db)) != null) {\r
-                                       //MigrationTool.ui.println(ifile + "  dofunc  " + temp);\r
-                                       //matfuncc.appendReplacement(result, MigrationTool.db.getR9Func(temp));\r
-                               }\r
-                       }\r
-                       //matfuncc.appendTail(result);\r
-                       //line = result.toString();\r
-\r
-                       // find macro\r
-                       matmacro = Macro.ptntmacro.matcher(line);\r
-                       while (matmacro.find()) {\r
-                               if ((temp = Macro.register(matmacro, this, db)) != null) {\r
-                               }\r
-                       }\r
-                       \r
-                       // find function definition\r
-                       // replace all {} to @\r
-                       while ((matenclosereplace = Func.ptnbrace.matcher(line)).find()) {\r
-                               line = matenclosereplace.replaceAll("@");\r
-                       }\r
-\r
-                       matfuncd = Func.ptnfuncd.matcher(line);\r
-                       while (matfuncd.find()) {\r
-                               if ((temp = Func.register(matfuncd, this, db)) != null) {\r
-                               }\r
-                       }\r
-               }\r
-               \r
-               // op on hash\r
-               Iterator<String> funcci = hashfuncc.iterator();\r
-               while (funcci.hasNext()) {\r
-                       if (!hashfuncd.contains(temp = funcci.next()) && !hashEFIcall.contains(temp)) {\r
-                               hashnonlocalfunc.add(temp);                                     // this set contains both changed and not changed items\r
-                       }\r
-               }\r
-       }\r
-       \r
        public final void enroll(String filepath) throws Exception {\r
                String[] temp;\r
                if (filepath.contains(".c") || filepath.contains(".C") || filepath.contains(".h") || \r
        public final void enroll(String filepath) throws Exception {\r
                String[] temp;\r
                if (filepath.contains(".c") || filepath.contains(".C") || filepath.contains(".h") || \r
@@ -286,7 +126,7 @@ public class ModuleInfo {
                }\r
                return false;\r
        }\r
                }\r
                return false;\r
        }\r
-       \r
+\r
        public static final void triger(String path) throws Exception {\r
                MigrationTool.ui.println("Project Migration");\r
                MigrationTool.ui.println("Copyright (c) 2006, Intel Corporation");\r
        public static final void triger(String path) throws Exception {\r
                MigrationTool.ui.println("Project Migration");\r
                MigrationTool.ui.println("Copyright (c) 2006, Intel Corporation");\r
index dd8df97b5ebdb251caaaa3fca53a06bd66bf1c03..faa3efefb1399fbedd57aa32e5fe49f8ac831359 100644 (file)
@@ -15,14 +15,40 @@ package org.tianocore.migration;
 import java.io.*;\r
 import java.util.*;\r
 import java.util.regex.*;\r
 import java.io.*;\r
 import java.util.*;\r
 import java.util.regex.*;\r
+\r
 import org.tianocore.*;\r
 \r
 public final class ModuleReader {\r
 import org.tianocore.*;\r
 \r
 public final class ModuleReader {\r
+       private static ModuleInfo mi;\r
+       \r
        private static final Pattern ptninfequation = Pattern.compile("([^\\s]*)\\s*=\\s*([^\\s]*)");\r
        private static final Pattern ptnsection = Pattern.compile("\\[([^\\[\\]]*)\\]([^\\[\\]]*)\\n", Pattern.MULTILINE);\r
        private static final Pattern ptnfilename = Pattern.compile("[^\\s]+");\r
        \r
        private static final Pattern ptninfequation = Pattern.compile("([^\\s]*)\\s*=\\s*([^\\s]*)");\r
        private static final Pattern ptnsection = Pattern.compile("\\[([^\\[\\]]*)\\]([^\\[\\]]*)\\n", Pattern.MULTILINE);\r
        private static final Pattern ptnfilename = Pattern.compile("[^\\s]+");\r
        \r
-       public static final void readMsa(String name, ModuleInfo mi) throws Exception {\r
+       public static final void ModuleScan(ModuleInfo m) throws Exception {\r
+               mi = m;\r
+               \r
+               Common.toDoAll(mi.modulepath, ModuleInfo.class.getMethod("enroll", String.class), mi, null, Common.FILE);\r
+               \r
+               String filename = null;\r
+               if (mi.msaorinf.isEmpty()) {\r
+                       MigrationTool.ui.println("No INF nor MSA file found!");\r
+                       System.exit(0);\r
+               } else {\r
+                       filename = MigrationTool.ui.choose("Found .inf or .msa file for module\n" + mi.modulepath + "\nChoose one Please", mi.msaorinf.toArray());\r
+               }\r
+               if (filename.contains(".inf")) {\r
+                       readInf(filename);\r
+               } else if (filename.contains(".msa")) {\r
+                       readMsa(filename);\r
+               }\r
+               \r
+               CommentOutNonLocalHFile();\r
+               parsePreProcessedSourceCode();\r
+\r
+       }\r
+       \r
+       private static final void readMsa(String name) throws Exception {\r
                ModuleSurfaceAreaDocument msadoc = ModuleSurfaceAreaDocument.Factory.parse(new File(mi.modulepath + File.separator + name));\r
                ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = msadoc.getModuleSurfaceArea();\r
                MsaHeaderDocument.MsaHeader msaheader = msa.getMsaHeader();\r
                ModuleSurfaceAreaDocument msadoc = ModuleSurfaceAreaDocument.Factory.parse(new File(mi.modulepath + File.separator + name));\r
                ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = msadoc.getModuleSurfaceArea();\r
                MsaHeaderDocument.MsaHeader msaheader = msa.getMsaHeader();\r
@@ -42,7 +68,7 @@ public final class ModuleReader {
                }\r
        }\r
        \r
                }\r
        }\r
        \r
-       public static final void readInf(String name, ModuleInfo mi) throws Exception {\r
+       private static final void readInf(String name) throws Exception {\r
                System.out.println("\nParsing INF file: " + name);\r
                String wholeline;\r
                Matcher mtrinfequation;\r
                System.out.println("\nParsing INF file: " + name);\r
                String wholeline;\r
                Matcher mtrinfequation;\r
@@ -89,4 +115,145 @@ public final class ModuleReader {
                        }\r
                }\r
        }\r
                        }\r
                }\r
        }\r
+       \r
+       // add '//' to all non-local include lines\r
+       private static final void CommentOutNonLocalHFile() throws IOException {\r
+               BufferedReader rd;\r
+               String line;\r
+               String curFile;\r
+               PrintWriter outfile;\r
+\r
+               Pattern ptninclude = Pattern.compile("[\"<](.*[.]h)[\">]");\r
+               Matcher mtrinclude;\r
+\r
+               Iterator<String> ii = mi.localmodulesources.iterator();\r
+               while ( ii.hasNext() ) {\r
+                       curFile = ii.next();\r
+                       rd = new BufferedReader(new FileReader(mi.modulepath + File.separator + curFile));\r
+                       Common.ensureDir(mi.modulepath + File.separator + "temp" + File.separator + curFile);\r
+                       outfile = new PrintWriter(new BufferedWriter(new FileWriter(mi.modulepath + File.separator + "temp" + File.separator + curFile)));\r
+                       while ((line = rd.readLine()) != null) {\r
+                               if (line.contains("#include")) {\r
+                                       mtrinclude = ptninclude.matcher(line);\r
+                                       if (mtrinclude.find() && mi.localmodulesources.contains(mtrinclude.group(1))) {\r
+                                       } else {\r
+                                               line = MigrationTool.migrationcomment + line;\r
+                                       }\r
+                               }\r
+                               outfile.append(line + '\n');\r
+                       }\r
+                       outfile.flush();\r
+                       outfile.close();\r
+               }\r
+       }\r
+\r
+       private static final void parsePreProcessedSourceCode() throws Exception {\r
+               //Cl cl = new Cl(modulepath);\r
+               //cl.execute("Fat.c");\r
+               //cl.generateAll(preprocessedccodes);\r
+               //\r
+               //System.out.println("Note!!!! The CL is not implemented now , pls do it manually!!! RUN :");\r
+               //System.out.println("cl " + modulepath + "\\temp\\*.c" + " -P");\r
+               //String[] list = new File(modulepath + File.separator + "temp").list();        // without CL , add\r
+               BufferedReader rd = null;\r
+               String ifile = null;\r
+               String line = null;\r
+               String temp = null;\r
+               \r
+               Iterator<String> ii = mi.localmodulesources.iterator();\r
+               while (ii.hasNext()) {\r
+                       temp = ii.next();\r
+                       if (temp.contains(".c")) {\r
+                               mi.preprocessedccodes.add(temp);\r
+                       }\r
+               }\r
+               \r
+               ii = mi.preprocessedccodes.iterator();\r
+               \r
+               Pattern patefifuncc = Pattern.compile("g?(BS|RT)\\s*->\\s*([a-zA-Z_]\\w*)",Pattern.MULTILINE);\r
+               Pattern patentrypoint = Pattern.compile("EFI_([A-Z]*)_ENTRY_POINT\\s*\\(([^\\(\\)]*)\\)",Pattern.MULTILINE);\r
+               Matcher matguid;\r
+               Matcher matfuncc;\r
+               Matcher matfuncd;\r
+               Matcher matenclosereplace;\r
+               Matcher matefifuncc;\r
+               Matcher matentrypoint;\r
+               Matcher matmacro;\r
+               \r
+               while (ii.hasNext()) {\r
+                       StringBuffer wholefile = new StringBuffer();\r
+                       ifile = ii.next();\r
+                       rd = new BufferedReader(new FileReader(mi.modulepath + File.separator + "temp" + File.separator + ifile));\r
+                       while ((line = rd.readLine()) != null) {\r
+                               wholefile.append(line + '\n');\r
+                       }\r
+                       line = wholefile.toString();\r
+                       \r
+                       // if this is a Pei phase module , add these library class to .msa\r
+                       matentrypoint = patentrypoint.matcher(line);\r
+                       if (matentrypoint.find()) {\r
+                               mi.entrypoint = matentrypoint.group(2);\r
+                               if (matentrypoint.group(1).matches("PEIM")) {\r
+                                       mi.hashrequiredr9libs.add("PeimEntryPoint");\r
+                               } else {\r
+                                       mi.hashrequiredr9libs.add("UefiDriverEntryPoint");\r
+                               }\r
+                       }\r
+                       \r
+                       // find guid\r
+                       matguid = Guid.ptnguid.matcher(line);                                                                           // several ways to implement this , which one is faster ? :\r
+                       while (matguid.find()) {                                                                                                        // 1.currently , find once , then call to identify which is it\r
+                               if ((temp = Guid.register(matguid, mi, MigrationTool.db)) != null) {                            // 2.use 3 different matchers , search 3 times to find each\r
+                                       //matguid.appendReplacement(result, MigrationTool.db.getR9Guidname(temp));              // search the database for all 3 kinds of guids , high cost\r
+                               }\r
+                       }\r
+                       //matguid.appendTail(result);\r
+                       //line = result.toString();\r
+\r
+                       // find EFI call in form of '->' , many 'gUnicodeCollationInterface->' like things are not changed\r
+                       // This item is not simply replaced , special operation is required.\r
+                       matefifuncc = patefifuncc.matcher(line);\r
+                       while (matefifuncc.find()) {\r
+                               mi.hashEFIcall.add(matefifuncc.group(2));\r
+                       }\r
+\r
+                       // find function call\r
+                       matfuncc = Func.ptnfuncc.matcher(line);\r
+                       while (matfuncc.find()) {\r
+                               if ((temp = Func.register(matfuncc, mi, MigrationTool.db)) != null) {\r
+                                       //MigrationTool.ui.println(ifile + "  dofunc  " + temp);\r
+                                       //matfuncc.appendReplacement(result, MigrationTool.db.getR9Func(temp));\r
+                               }\r
+                       }\r
+                       //matfuncc.appendTail(result);\r
+                       //line = result.toString();\r
+\r
+                       // find macro\r
+                       matmacro = Macro.ptntmacro.matcher(line);\r
+                       while (matmacro.find()) {\r
+                               if ((temp = Macro.register(matmacro, mi, MigrationTool.db)) != null) {\r
+                               }\r
+                       }\r
+                       \r
+                       // find function definition\r
+                       // replace all {} to @\r
+                       while ((matenclosereplace = Func.ptnbrace.matcher(line)).find()) {\r
+                               line = matenclosereplace.replaceAll("@");\r
+                       }\r
+\r
+                       matfuncd = Func.ptnfuncd.matcher(line);\r
+                       while (matfuncd.find()) {\r
+                               if ((temp = Func.register(matfuncd, mi, MigrationTool.db)) != null) {\r
+                               }\r
+                       }\r
+               }\r
+               \r
+               // op on hash\r
+               Iterator<String> funcci = mi.hashfuncc.iterator();\r
+               while (funcci.hasNext()) {\r
+                       if (!mi.hashfuncd.contains(temp = funcci.next()) && !mi.hashEFIcall.contains(temp)) {\r
+                               mi.hashnonlocalfunc.add(temp);                                  // this set contains both changed and not changed items\r
+                       }\r
+               }\r
+       }\r
 }\r
 }\r