From c0db4cdc640920a99133fc4b6c111e9bd9f9630b Mon Sep 17 00:00:00 2001 From: jlin16 Date: Wed, 14 Feb 2007 03:08:45 +0000 Subject: [PATCH] add check for empty value of definitions. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2391 6f19259b-4bc3-4df7-8a09-765794883524 --- .../toolchain/ToolChainId.java | 57 ++++++++++++------- 1 file changed, 38 insertions(+), 19 deletions(-) diff --git a/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/toolchain/ToolChainId.java b/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/toolchain/ToolChainId.java index b331af363e..65e20bd665 100644 --- a/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/toolchain/ToolChainId.java +++ b/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/toolchain/ToolChainId.java @@ -158,53 +158,70 @@ public class ToolChainId { String rLine = null; String inLine[] = new String[2]; while ((rLine = reader.readLine()) != null) { + rLine = rLine.trim(); if ((rLine.startsWith("ACTIVE_PLATFORM")) && (activePlatform == null)) { // Only one active platform is permitted! - inLine = rLine.trim().split("="); - activePlatform = inLine[1].trim(); + inLine = rLine.split("="); + if (inLine.length > 1) { + activePlatform = inLine[1].trim(); + } } if ((rLine.startsWith("TARGET" + " ")) || (rLine.startsWith("TARGET" + "\t")) || (rLine.startsWith("TARGET="))) { // Handle multiple Target Names if (rLine.contains(",")) { - inLine = rLine.trim().split("="); - buildTargets = inLine[1].trim().replaceAll(",", " "); + inLine = rLine.split("="); + if (inLine.length > 1) { + buildTargets = inLine[1].trim().replaceAll(",", " "); + } } else { inLine = rLine.trim().split("="); - buildTargets = inLine[1].trim(); + if (inLine.length > 1) { + buildTargets = inLine[1].trim(); + } } } if (rLine.startsWith("TARGET_ARCH")) { // Handle multiple Target Architectures if (rLine.contains(",")) { - inLine = rLine.trim().split("="); - targetArchs = inLine[1].trim().replaceAll(",", " "); + inLine = rLine.split("="); + if (inLine.length > 1) { + targetArchs = inLine[1].trim().replaceAll(",", " "); + } } else { - inLine = rLine.trim().split("="); - targetArchs = inLine[1].trim(); + inLine = rLine.split("="); + if (inLine.length > 1) { + targetArchs = inLine[1].trim(); + } } } if (rLine.startsWith("TOOL_CHAIN_CONF")) { // Only one file is permitted - inLine = rLine.trim().split("="); - toolDefinitionFile = inLine[1].trim(); + inLine = rLine.split("="); + if (inLine.length > 1) { + toolDefinitionFile = inLine[1].trim(); + } } if (rLine.startsWith("TOOL_CHAIN_TAG")) { // Handle multiple Tool TagNames if (rLine.contains(",")) { - inLine = rLine.trim().split("="); - tagNames = inLine[1].trim().replaceAll(",", " "); + inLine = rLine.split("="); + if (inLine.length > 1) { + tagNames = inLine[1].trim().replaceAll(",", " "); + } } else { - inLine = rLine.trim().split("="); - tagNames = inLine[1].trim(); + inLine = rLine.split("="); + if (inLine.length > 1) { + tagNames = inLine[1].trim(); + } } } if (rLine.startsWith("MULTIPLE_THREAD")) { // Handle Thread Enable flag - if ((rLine.trim().toLowerCase().contains("enabled")) - || (rLine.trim().toLowerCase().contains("true"))) { + if ((rLine.toLowerCase().contains("enabled")) + || (rLine.toLowerCase().contains("true"))) { threadEnabled = true; } else { threadEnabled = false; @@ -213,8 +230,10 @@ public class ToolChainId { if (rLine.startsWith("MAX_CONCURRENT_THREAD_NUMBER")) { // Handle Thread Enable flag - inLine = rLine.trim().split("="); - maxThreadCount = Integer.valueOf(inLine[1].trim()); + inLine = rLine.split("="); + if (inLine.length > 1) { + maxThreadCount = Integer.valueOf(inLine[1].trim()); + } } } reader.close(); -- 2.39.2