Improve commandline handling control flow

This uses a switch on the action value, which makes it more clear what
code runs when. No actual behaviour is changed, most of the changes in
this commit are indentation changes.
This commit is contained in:
Matthijs Kooijman 2014-04-07 10:06:48 +02:00 committed by Cristian Maglie
parent f3565a1bda
commit 7548591d51
1 changed files with 37 additions and 33 deletions

View File

@ -430,46 +430,50 @@ public class Base {
} }
} }
if (action == ACTION.UPLOAD || action == ACTION.VERIFY) { switch (action) {
// Set verbosity for command line build case VERIFY:
Preferences.set("build.verbose", "" + doVerboseBuild); case UPLOAD:
Preferences.set("upload.verbose", "" + doVerboseUpload); // Set verbosity for command line build
Preferences.set("build.verbose", "" + doVerboseBuild);
Preferences.set("upload.verbose", "" + doVerboseUpload);
Editor editor = editors.get(0); Editor editor = editors.get(0);
// Do board selection if requested // Do board selection if requested
processBoardArgument(selectBoard); processBoardArgument(selectBoard);
if (action == ACTION.UPLOAD) { if (action == ACTION.UPLOAD) {
// Build and upload // Build and upload
if (selectPort != null) if (selectPort != null)
editor.selectSerialPort(selectPort); editor.selectSerialPort(selectPort);
editor.exportHandler.run(); editor.exportHandler.run();
} else { } else {
// Build only // Build only
editor.runHandler.run(); editor.runHandler.run();
} }
// Error during build or upload // Error during build or upload
int res = editor.status.mode; int res = editor.status.mode;
if (res == EditorStatus.ERR) if (res == EditorStatus.ERR)
System.exit(1); System.exit(1);
// No errors exit gracefully // No errors exit gracefully
System.exit(0); System.exit(0);
} break;
case GUI:
// Check if there were previously opened sketches to be restored
restoreSketches();
// Check if there were previously opened sketches to be restored // Create a new empty window (will be replaced with any files to be opened)
restoreSketches(); if (editors.isEmpty()) {
handleNew();
}
// Create a new empty window (will be replaced with any files to be opened) // Check for updates
if (editors.isEmpty()) { if (Preferences.getBoolean("update.check")) {
handleNew(); new UpdateCheck(this);
} }
break;
// Check for updates
if (Preferences.getBoolean("update.check")) {
new UpdateCheck(this);
} }
} }