1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-23 14:42:11 +02:00

compilation warnings

This commit is contained in:
Andrew Gvozdev 2010-04-28 20:03:14 +00:00
parent 44f53ef53d
commit 97285f3b56
2 changed files with 14 additions and 15 deletions

View file

@ -30,7 +30,7 @@ import org.eclipse.cdt.make.internal.core.scannerconfig2.ScannerConfigProfileMan
import org.eclipse.core.resources.IProject;
import org.eclipse.core.runtime.IProgressMonitor;
import org.eclipse.core.runtime.ISafeRunnable;
import org.eclipse.core.runtime.Platform;
import org.eclipse.core.runtime.SafeRunner;
/**
* Utility class for build and job related functionality
@ -55,7 +55,8 @@ public class SCJobsUtil {
this.rc = rc;
}
public String toString() {
@Override
public String toString() {
return rc ? "true" : "false"; //$NON-NLS-1$ //$NON-NLS-2$
}
private boolean rc;
@ -84,9 +85,9 @@ public class SCJobsUtil {
getSCProfileInstance(project, context, buildInfo.getSelectedProfileId());
final IScannerInfoCollector collector = profileInstance.getScannerInfoCollector();
List providerIds = buildInfo.getProviderIdList();
List<String> providerIds = buildInfo.getProviderIdList();
for (int i = 0; i < providerIds.size(); ++i) {
final String providerId = (String) providerIds.get(i);
final String providerId = providerIds.get(i);
if (buildInfo.isProviderOutputParserEnabled(providerId)) {
final IExternalScannerInfoProvider esiProvider = profileInstance.
createExternalScannerInfoProvider(providerId);
@ -114,7 +115,7 @@ public class SCJobsUtil {
}
};
Platform.run(runnable);
SafeRunner.run(runnable);
}
}
}
@ -165,7 +166,7 @@ public class SCJobsUtil {
}
};
Platform.run(runnable);
SafeRunner.run(runnable);
}
return rc.get();
@ -215,7 +216,7 @@ public class SCJobsUtil {
}
};
Platform.run(runnable);
SafeRunner.run(runnable);
}
return rc.get();

View file

@ -1,5 +1,5 @@
/*******************************************************************************
* Copyright (c) 2004, 2009 IBM Corporation and others.
* Copyright (c) 2004, 2010 IBM Corporation and others.
* All rights reserved. This program and the accompanying materials
* are made available under the terms of the Eclipse Public License v1.0
* which accompanies this distribution, and is available at
@ -15,7 +15,6 @@ import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.PrintStream;
import java.util.Iterator;
import java.util.List;
import org.eclipse.cdt.make.core.MakeCorePlugin;
@ -41,7 +40,8 @@ public class SCDMakefileGenerator extends DefaultRunSIProvider {
/* (non-Javadoc)
* @see org.eclipse.cdt.make.internal.core.scannerconfig2.DefaultRunSIProvider#initialize()
*/
protected boolean initialize() {
@Override
protected boolean initialize() {
String args = buildInfo.getProviderRunArguments(providerId);
if (null == args)
args = " -E -P -v -dD "; //$NON-NLS-1$
@ -69,7 +69,7 @@ public class SCDMakefileGenerator extends DefaultRunSIProvider {
boolean rc = false;
if (collector instanceof IScannerInfoCollector2) {
IScannerInfoCollector2 collector2 = (IScannerInfoCollector2) collector;
List commands = collector2.getCollectedScannerInfo(
List<CCommandDSC> commands = collector2.getCollectedScannerInfo(
resource.getProject(), ScannerInfoTypes.UNDISCOVERED_COMPILER_COMMAND);
if (commands != null && commands.size() > 0) {
@ -79,16 +79,14 @@ public class SCDMakefileGenerator extends DefaultRunSIProvider {
buffer.append(".PHONY: all"); //$NON-NLS-1$
buffer.append(DENDL);
buffer.append("COMMANDS := "); //$NON-NLS-1$
for (Iterator i = commands.iterator(); i.hasNext(); ) {
CCommandDSC cmd = (CCommandDSC) i.next();
for (CCommandDSC cmd : commands) {
buffer.append("\t\\"+ENDL+"\t scd_cmd_"); //$NON-NLS-1$ //$NON-NLS-2$
buffer.append(cmd.getCommandId());
}
buffer.append(DENDL);
buffer.append("all: $(COMMANDS)"); //$NON-NLS-1$
buffer.append(DENDL);
for (Iterator i = commands.iterator(); i.hasNext(); ) {
CCommandDSC cmd = (CCommandDSC) i.next();
for (CCommandDSC cmd : commands) {
buffer.append("scd_cmd_"); //$NON-NLS-1$
buffer.append(cmd.getCommandId());
buffer.append(':');