ADT: Remove StreamHelper and integrate it in AdtPlugin.
StreamHelper was used before by AdtPlugin, DdmsPlugin and EditorsPlugin. It's now only used by AdtPlugin, so the only 2 methods in it can go in AdtPlugin. I merged the externalized string in the messages.properties file used by AdtPlugin.
This commit is contained in:
@@ -54,7 +54,6 @@ Export-Package: com.android.ide.eclipse.adt;x-friends:="com.android.ide.eclipse.
|
|||||||
com.android.ide.eclipse.adt.sdk;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.adt.sdk;x-friends:="com.android.ide.eclipse.tests",
|
||||||
com.android.ide.eclipse.adt.wizards.newproject;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.adt.wizards.newproject;x-friends:="com.android.ide.eclipse.tests",
|
||||||
com.android.ide.eclipse.adt.ui;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.adt.ui;x-friends:="com.android.ide.eclipse.tests",
|
||||||
com.android.ide.eclipse.common;x-friends:="com.android.ide.eclipse.tests",
|
|
||||||
com.android.ide.eclipse.common.project;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.common.project;x-friends:="com.android.ide.eclipse.tests",
|
||||||
com.android.ide.eclipse.common.resources;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.common.resources;x-friends:="com.android.ide.eclipse.tests",
|
||||||
com.android.ide.eclipse.editors;x-friends:="com.android.ide.eclipse.tests",
|
com.android.ide.eclipse.editors;x-friends:="com.android.ide.eclipse.tests",
|
||||||
|
|||||||
@@ -30,7 +30,6 @@ import com.android.ide.eclipse.adt.sdk.LoadStatus;
|
|||||||
import com.android.ide.eclipse.adt.sdk.Sdk;
|
import com.android.ide.eclipse.adt.sdk.Sdk;
|
||||||
import com.android.ide.eclipse.adt.sdk.Sdk.ITargetChangeListener;
|
import com.android.ide.eclipse.adt.sdk.Sdk.ITargetChangeListener;
|
||||||
import com.android.ide.eclipse.adt.ui.EclipseUiHelper;
|
import com.android.ide.eclipse.adt.ui.EclipseUiHelper;
|
||||||
import com.android.ide.eclipse.common.StreamHelper;
|
|
||||||
import com.android.ide.eclipse.common.project.BaseProjectHelper;
|
import com.android.ide.eclipse.common.project.BaseProjectHelper;
|
||||||
import com.android.ide.eclipse.common.project.ExportHelper;
|
import com.android.ide.eclipse.common.project.ExportHelper;
|
||||||
import com.android.ide.eclipse.common.project.ExportHelper.IExportCallback;
|
import com.android.ide.eclipse.common.project.ExportHelper.IExportCallback;
|
||||||
@@ -109,6 +108,7 @@ import java.net.MalformedURLException;
|
|||||||
import java.net.URL;
|
import java.net.URL;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
import java.util.Arrays;
|
import java.util.Arrays;
|
||||||
|
import java.util.Calendar;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -203,7 +203,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
@Override
|
@Override
|
||||||
public void println(String message) {
|
public void println(String message) {
|
||||||
// write the date/project tag first.
|
// write the date/project tag first.
|
||||||
String tag = StreamHelper.getMessageTag(mProject != null ? mProject.getName() : null);
|
String tag = getMessageTag(mProject != null ? mProject.getName() : null);
|
||||||
|
|
||||||
print(tag);
|
print(tag);
|
||||||
if (mPrefix != null) {
|
if (mPrefix != null) {
|
||||||
@@ -536,7 +536,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
* @return the image descriptor
|
* @return the image descriptor
|
||||||
*/
|
*/
|
||||||
public static ImageDescriptor getImageDescriptor(String path) {
|
public static ImageDescriptor getImageDescriptor(String path) {
|
||||||
return imageDescriptorFromPlugin(PLUGIN_ID, path);
|
return imageDescriptorFromPlugin(PLUGIN_ID, path);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -732,7 +732,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
String message = String.format(format, args);
|
String message = String.format(format, args);
|
||||||
Status status = new Status(IStatus.ERROR, PLUGIN_ID, message, exception);
|
Status status = new Status(IStatus.ERROR, PLUGIN_ID, message, exception);
|
||||||
getDefault().getLog().log(status);
|
getDefault().getLog().log(status);
|
||||||
StreamHelper.printToStream(sPlugin.mAndroidConsoleErrorStream, tag, message);
|
printToStream(sPlugin.mAndroidConsoleErrorStream, tag, message);
|
||||||
showAndroidConsole();
|
showAndroidConsole();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -744,7 +744,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
*/
|
*/
|
||||||
public static synchronized void printErrorToConsole(String tag, Object... objects) {
|
public static synchronized void printErrorToConsole(String tag, Object... objects) {
|
||||||
if (sPlugin != null) {
|
if (sPlugin != null) {
|
||||||
StreamHelper.printToStream(sPlugin.mAndroidConsoleErrorStream, tag, objects);
|
printToStream(sPlugin.mAndroidConsoleErrorStream, tag, objects);
|
||||||
|
|
||||||
showAndroidConsole();
|
showAndroidConsole();
|
||||||
}
|
}
|
||||||
@@ -782,7 +782,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
if (sPlugin != null) {
|
if (sPlugin != null) {
|
||||||
if (level <= sPlugin.mBuildVerbosity) {
|
if (level <= sPlugin.mBuildVerbosity) {
|
||||||
String tag = project != null ? project.getName() : null;
|
String tag = project != null ? project.getName() : null;
|
||||||
StreamHelper.printToStream(sPlugin.mAndroidConsoleStream, tag, objects);
|
printToStream(sPlugin.mAndroidConsoleStream, tag, objects);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -794,7 +794,7 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
*/
|
*/
|
||||||
public static synchronized void printToConsole(String tag, Object... objects) {
|
public static synchronized void printToConsole(String tag, Object... objects) {
|
||||||
if (sPlugin != null) {
|
if (sPlugin != null) {
|
||||||
StreamHelper.printToStream(sPlugin.mAndroidConsoleStream, tag, objects);
|
printToStream(sPlugin.mAndroidConsoleStream, tag, objects);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1398,4 +1398,40 @@ public class AdtPlugin extends AbstractUIPlugin {
|
|||||||
SdkStatsService.ping("adt", versionString, getDisplay()); //$NON-NLS-1$
|
SdkStatsService.ping("adt", versionString, getDisplay()); //$NON-NLS-1$
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Prints messages, associated with a project to the specified stream
|
||||||
|
* @param stream The stream to write to
|
||||||
|
* @param tag The tag associated to the message. Can be null
|
||||||
|
* @param objects The objects to print through their toString() method (or directly for
|
||||||
|
* {@link String} objects.
|
||||||
|
*/
|
||||||
|
public static synchronized void printToStream(MessageConsoleStream stream, String tag,
|
||||||
|
Object... objects) {
|
||||||
|
String dateTag = getMessageTag(tag);
|
||||||
|
|
||||||
|
for (Object obj : objects) {
|
||||||
|
stream.print(dateTag);
|
||||||
|
if (obj instanceof String) {
|
||||||
|
stream.println((String)obj);
|
||||||
|
} else {
|
||||||
|
stream.println(obj.toString());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Creates a string containing the current date/time, and the tag
|
||||||
|
* @param tag The tag associated to the message. Can be null
|
||||||
|
* @return The dateTag
|
||||||
|
*/
|
||||||
|
public static String getMessageTag(String tag) {
|
||||||
|
Calendar c = Calendar.getInstance();
|
||||||
|
|
||||||
|
if (tag == null) {
|
||||||
|
return String.format(Messages.Console_Date_Tag, c);
|
||||||
|
}
|
||||||
|
|
||||||
|
return String.format(Messages.Console_Data_Project_Tag, c, tag);
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -16,6 +16,10 @@ public class Messages extends NLS {
|
|||||||
|
|
||||||
public static String AdtPlugin_Parsing_Resources;
|
public static String AdtPlugin_Parsing_Resources;
|
||||||
|
|
||||||
|
public static String Console_Data_Project_Tag;
|
||||||
|
|
||||||
|
public static String Console_Date_Tag;
|
||||||
|
|
||||||
public static String Could_Not_Find;
|
public static String Could_Not_Find;
|
||||||
|
|
||||||
public static String Could_Not_Find_Folder;
|
public static String Could_Not_Find_Folder;
|
||||||
|
|||||||
@@ -14,3 +14,5 @@ AdtPlugin_Android_SDK_Content_Loader=Android SDK Content Loader
|
|||||||
AdtPlugin_Parsing_Resources=Parsing Resources
|
AdtPlugin_Parsing_Resources=Parsing Resources
|
||||||
AdtPlugin_Android_SDK_Resource_Parser=Android SDK Resource Parser
|
AdtPlugin_Android_SDK_Resource_Parser=Android SDK Resource Parser
|
||||||
AdtPlugin_Failed_To_Parse_s=Failed to parse:
|
AdtPlugin_Failed_To_Parse_s=Failed to parse:
|
||||||
|
Console_Date_Tag=[%1$tF %1$tT]
|
||||||
|
Console_Data_Project_Tag=[%1$tF %1$tT - %2$s]
|
||||||
|
|||||||
@@ -1,21 +0,0 @@
|
|||||||
|
|
||||||
package com.android.ide.eclipse.common;
|
|
||||||
|
|
||||||
import org.eclipse.osgi.util.NLS;
|
|
||||||
|
|
||||||
public class Messages extends NLS {
|
|
||||||
private static final String BUNDLE_NAME = "com.android.ide.eclipse.common.messages"; //$NON-NLS-1$
|
|
||||||
|
|
||||||
public static String Console_Data_Project_Tag;
|
|
||||||
|
|
||||||
public static String Console_Date_Tag;
|
|
||||||
|
|
||||||
|
|
||||||
static {
|
|
||||||
// initialize resource bundle
|
|
||||||
NLS.initializeMessages(BUNDLE_NAME, Messages.class);
|
|
||||||
}
|
|
||||||
|
|
||||||
private Messages() {
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -1,63 +0,0 @@
|
|||||||
/*
|
|
||||||
* Copyright (C) 2007 The Android Open Source Project
|
|
||||||
*
|
|
||||||
* Licensed under the Eclipse Public License, Version 1.0 (the "License");
|
|
||||||
* you may not use this file except in compliance with the License.
|
|
||||||
* You may obtain a copy of the License at
|
|
||||||
*
|
|
||||||
* http://www.eclipse.org/org/documents/epl-v10.php
|
|
||||||
*
|
|
||||||
* Unless required by applicable law or agreed to in writing, software
|
|
||||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
||||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
||||||
* See the License for the specific language governing permissions and
|
|
||||||
* limitations under the License.
|
|
||||||
*/
|
|
||||||
|
|
||||||
package com.android.ide.eclipse.common;
|
|
||||||
|
|
||||||
import org.eclipse.ui.console.MessageConsoleStream;
|
|
||||||
|
|
||||||
import java.util.Calendar;
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Stream helper class.
|
|
||||||
*/
|
|
||||||
public class StreamHelper {
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Prints messages, associated with a project to the specified stream
|
|
||||||
* @param stream The stream to write to
|
|
||||||
* @param tag The tag associated to the message. Can be null
|
|
||||||
* @param objects The objects to print through their toString() method (or directly for
|
|
||||||
* {@link String} objects.
|
|
||||||
*/
|
|
||||||
public static synchronized void printToStream(MessageConsoleStream stream, String tag,
|
|
||||||
Object... objects) {
|
|
||||||
String dateTag = getMessageTag(tag);
|
|
||||||
|
|
||||||
for (Object obj : objects) {
|
|
||||||
stream.print(dateTag);
|
|
||||||
if (obj instanceof String) {
|
|
||||||
stream.println((String)obj);
|
|
||||||
} else {
|
|
||||||
stream.println(obj.toString());
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Creates a string containing the current date/time, and the tag
|
|
||||||
* @param tag The tag associated to the message. Can be null
|
|
||||||
* @return The dateTag
|
|
||||||
*/
|
|
||||||
public static String getMessageTag(String tag) {
|
|
||||||
Calendar c = Calendar.getInstance();
|
|
||||||
|
|
||||||
if (tag == null) {
|
|
||||||
return String.format(Messages.Console_Date_Tag, c);
|
|
||||||
}
|
|
||||||
|
|
||||||
return String.format(Messages.Console_Data_Project_Tag, c, tag);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -1,2 +0,0 @@
|
|||||||
Console_Date_Tag=[%1$tF %1$tT]
|
|
||||||
Console_Data_Project_Tag=[%1$tF %1$tT - %2$s]
|
|
||||||
Reference in New Issue
Block a user