]> source.dussan.org Git - aspectj.git/commitdiff
148190#32
authoraclement <aclement>
Mon, 15 Jan 2007 08:44:25 +0000 (08:44 +0000)
committeraclement <aclement>
Mon, 15 Jan 2007 08:44:25 +0000 (08:44 +0000)
ajde/src/org/aspectj/ajde/ui/internal/AjcBuildOptions.java [deleted file]
ajde/src/org/aspectj/ajde/ui/internal/BrowserViewProperties.java [deleted file]
ajde/src/org/aspectj/ajde/ui/internal/UserPreferencesStore.java

diff --git a/ajde/src/org/aspectj/ajde/ui/internal/AjcBuildOptions.java b/ajde/src/org/aspectj/ajde/ui/internal/AjcBuildOptions.java
deleted file mode 100644 (file)
index 95df7d2..0000000
+++ /dev/null
@@ -1,317 +0,0 @@
-/* *******************************************************************
- * Copyright (c) 1999-2001 Xerox Corporation, 
- *               2002 Palo Alto Research Center, Incorporated (PARC).
- * 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 
- * http://www.eclipse.org/legal/epl-v10.html 
- *  
- * Contributors: 
- *     Xerox/PARC     initial implementation 
- *     AMC 01.20.2003 extended to support AspectJ 1.1 options,
- *                                       bugzilla #29769
- * ******************************************************************/
-
-
-package org.aspectj.ajde.ui.internal;
-
-import java.util.HashSet;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.Set;
-import java.util.StringTokenizer;
-
-import org.aspectj.ajde.BuildOptionsAdapter;
-import org.aspectj.ajde.ui.UserPreferencesAdapter;
-
-public class AjcBuildOptions implements BuildOptionsAdapter {
-       
-       private UserPreferencesAdapter preferencesAdapter = null;
-       
-       private static final String AJC = "ajc";
-       private static final String USE_JAVAC_MODE = AJC + ".useJavacMode";
-       private static final String WORKING_DIR = AJC + ".workingDir";
-       private static final String PREPROCESS_MODE = AJC + ".preprocessMode";
-       private static final String CHARACTER_ENCODING = AJC + ".characterEncoding";
-       private static final String SOURCE_ONE_POINT_FOUR_MODE = AJC + ".sourceOnePointFourMode";
-       private static final String LENIENT_MODE = AJC + ".lenientSpecMode";
-       private static final String STRICT_MODE = AJC + ".strictSpecMode";
-       private static final String PORTING_MODE = AJC + ".portingMode";
-       private static final String VERBOSE_MODE = AJC + ".verboseMode";
-       private static final String NONSTANDARD_OPTIONS = AJC + ".nonStandardOptions";
-       // 1.1 constants added by AMC
-    private static final String INCREMENTAL_MODE = AJC + ".incrementalMode";
-    private static final String COMPLIANCE_LEVEL = AJC + ".complianceLevel";
-       private static final String SOURCE_LEVEL = AJC + ".sourceLevel";
-       private static final String WARNINGS = AJC + ".warnings";
-       private static final String DEBUG_OPTIONS = AJC + ".debugOptions";
-       private static final String NO_IMPORT_ERROR = AJC + ".noImportError";
-       private static final String PRESERVE_LOCALS = AJC + ".preserveLocals";
-       private static final String DEFAULT = "default";
-       
-       public AjcBuildOptions(UserPreferencesAdapter userPreferencesAdapter) {
-               this.preferencesAdapter = userPreferencesAdapter;
-               
-               
-       }
-
-       /** @deprecated */      
-       public boolean getUseJavacMode() {
-               return getBooleanOptionVal(USE_JAVAC_MODE);
-       }
-       
-       /** @deprecated */
-       public void setUseJavacMode(boolean value) {
-               setBooleanOptionVal(USE_JAVAC_MODE, value);
-       }
-       
-       /** @deprecated */
-       public String getWorkingOutputPath() {
-               return preferencesAdapter.getProjectPreference(WORKING_DIR);
-       }
-       
-       /** @deprecated */
-       public void setWorkingDir(String path) {
-               preferencesAdapter.setProjectPreference(WORKING_DIR, path);     
-       }
-       
-       /** @deprecated */
-       public boolean getPreprocessMode() {
-               return getBooleanOptionVal(PREPROCESS_MODE);
-       }
-       
-       /** @deprecated */
-       public void setPreprocessMode(boolean value) {
-               setBooleanOptionVal(PREPROCESS_MODE, value);    
-       }
-       
-       public String getCharacterEncoding() {
-               return preferencesAdapter.getProjectPreference(CHARACTER_ENCODING);     
-       }
-       
-       public void setCharacterEncoding(String value) {
-               preferencesAdapter.setProjectPreference(CHARACTER_ENCODING, value);     
-       }
-       
-       /** @deprecated */
-       public boolean getSourceOnePointFourMode() {
-               return getBooleanOptionVal(SOURCE_ONE_POINT_FOUR_MODE);
-       }
-       
-       /** @deprecated */
-       public void setSourceOnePointFourMode(boolean value) {
-               setBooleanOptionVal(SOURCE_ONE_POINT_FOUR_MODE, value); 
-       }
-       
-    public boolean getIncrementalMode() {
-        return getBooleanOptionVal(INCREMENTAL_MODE);
-    }
-
-    public void setIncrementalMode(boolean value) {
-        setBooleanOptionVal(INCREMENTAL_MODE, value);
-    }
-
-       /** @deprecated */
-       public boolean getLenientSpecMode() {
-               return getBooleanOptionVal(LENIENT_MODE);
-       }
-       
-       /** @deprecated */
-       public void setLenientSpecMode(boolean value) {
-               setBooleanOptionVal(LENIENT_MODE, value);       
-       }
-       
-       /** @deprecated */
-       public boolean getStrictSpecMode() {
-               return getBooleanOptionVal(STRICT_MODE);        
-       }
-       
-       /** @deprecated */
-       public void setStrictSpecMode(boolean value) {
-               setBooleanOptionVal(STRICT_MODE, value);
-       }
-       
-       /** @deprecated */
-       public boolean getPortingMode() {
-               return getBooleanOptionVal(PORTING_MODE);
-       }
-       
-       /** @deprecated */
-       public void setPortingMode(boolean value) {
-               setBooleanOptionVal(PORTING_MODE, value);
-       }
-       
-       public boolean getVerboseMode() {
-               return getBooleanOptionVal(VERBOSE_MODE);
-       }
-       
-       public void setVerboseMode(boolean value) {
-               setBooleanOptionVal(VERBOSE_MODE, value);
-       }
-       
-       public String getNonStandardOptions() {
-               return preferencesAdapter.getProjectPreference(NONSTANDARD_OPTIONS);            
-       }
-       
-       public void setNonStandardOptions(String value) {
-               preferencesAdapter.setProjectPreference(NONSTANDARD_OPTIONS, value);    
-       }
-
-       // -------------
-       // new 1.1 compiler options start here...
-       
-       /**
-        * JDK Compliance level to be used by the compiler, either 
-        * VERSION_13, VERSION_14 or VERSION_15.
-        * From -1.3 / -1.4 / -1.5
-        */
-       public String getComplianceLevel() {
-               return preferencesAdapter.getProjectPreference(COMPLIANCE_LEVEL);               
-       }
-       
-       public void setComplianceLevel( String value ) {
-               preferencesAdapter.setProjectPreference(COMPLIANCE_LEVEL,value);        
-       }
-       
-       /**
-        * Source compatibility level, either VERSION_13, VERSION_14 or VERSION_15.
-        * From -source (eclipse option)
-        */
-       public String getSourceCompatibilityLevel() {
-               return preferencesAdapter.getProjectPreference(SOURCE_LEVEL);           
-       }
-
-       public void setSourceCompatibilityLevel(String value) {
-               preferencesAdapter.setProjectPreference(SOURCE_LEVEL, value);           
-       }
-       
-       /**
-        * Optional warnings, empty List is equivalent to -warn:none,
-        * returning null uses eclipse compiler default settings
-        * From -warn:xxx,yyy
-        */
-       public Set getWarnings() {
-               String warnings = preferencesAdapter.getProjectPreference(WARNINGS);
-               return toWarningSet( warnings );
-       }
-       
-       public void setWarnings( Set warningSet ) {
-               String warnings = fromWarningSet( warningSet );
-               preferencesAdapter.setProjectPreference(WARNINGS, warnings); 
-       }       
-       
-       /**
-        * Debug level. DEBUG_ALL == {SOURCE, LINES, VARS}.
-        * Empty list is equivalent to -g:none, returning
-        * non uses eclipse compiler default settings
-        * From -g:xxx
-        */
-       public Set getDebugLevel() {
-               String debug = preferencesAdapter.getProjectPreference(DEBUG_OPTIONS);
-               return toDebugSet( debug );
-       }
-
-       public void setDebugLevel( Set debugSet ) {
-               String debug = fromDebugSet( debugSet );
-               preferencesAdapter.setProjectPreference(DEBUG_OPTIONS, debug); 
-       }       
-       
-       /**
-        * No errors generated for unresolved imports
-        * From -noImportError
-        */
-       public boolean getNoImportError() {
-               return getBooleanOptionVal(NO_IMPORT_ERROR);
-       }
-       
-       public void setNoImportError(boolean value) {
-               setBooleanOptionVal(NO_IMPORT_ERROR, value);    
-       }
-       
-       /**
-        * Preserve all unused local variables (for debug)
-        * From -preserveAllLocals
-        */
-       public boolean getPreserveAllLocals() {
-               return getBooleanOptionVal(PRESERVE_LOCALS);
-       }
-       
-       public void setPreserveAllLocals(boolean value) {
-               setBooleanOptionVal(PRESERVE_LOCALS, value);    
-       }
-       
-       private boolean getBooleanOptionVal(String name) {
-               if (preferencesAdapter.getProjectPreference(name) != null) {
-                       return preferencesAdapter.getProjectPreference(name).equals("true");
-               } else {
-                       return false;
-               }
-       }
-       
-       private void setBooleanOptionVal(String name, boolean value) {
-               if (value) {
-                       preferencesAdapter.setProjectPreference(name, "true");
-               } else {
-                       preferencesAdapter.setProjectPreference(name, "false");
-               }
-       }
-       
-       private Set toWarningSet( String warnings ) {
-               if ( null == warnings ) return null;
-               if ( warnings.equals(DEFAULT) ) return null;
-               
-               Set warningSet = new HashSet(); 
-               StringTokenizer tok = new StringTokenizer( warnings, "," );
-               while ( tok.hasMoreTokens() ) {
-                       String warning = tok.nextToken();
-                       warningSet.add( warning );      
-               }
-               return warningSet;
-       }
-       
-       private String fromWarningSet( Set warningSet ) {
-               if ( warningSet == null ) return DEFAULT;
-               
-               StringBuffer warnings = new StringBuffer();
-               Iterator it = warningSet.iterator();
-               while ( it.hasNext() ) {
-                       String w = (String) it.next();  
-                       if (warnings.length() > 0 ) warnings.append(',');
-                       warnings.append( w );
-               }
-               return warnings.toString();
-       }
-
-       private Set toDebugSet( String debugOptions ) {
-               if ( null == debugOptions ) return null;
-               if ( debugOptions.equals(DEFAULT) ) return null;
-               
-               Set debugSet = new HashSet();   
-               StringTokenizer tok = new StringTokenizer( debugOptions, "," );
-               while ( tok.hasMoreTokens() ) {
-                       String debug = tok.nextToken();
-                       debugSet.add( debug );  
-               }
-               return debugSet;        }
-       
-       private String fromDebugSet( Set debugSet ) {
-               if ( debugSet == null ) return DEFAULT;
-               
-               StringBuffer debugOptions = new StringBuffer();
-               Iterator it = debugSet.iterator();
-               while ( it.hasNext() ) {
-                       String d = (String) it.next();  
-                       if (debugOptions.length() > 0 ) debugOptions.append(',');
-                       debugOptions.append( d );
-               }
-               return debugOptions.toString();
-       }
-
-
-       public Map getJavaOptionsMap() {
-               return null;
-       }
-
-}
diff --git a/ajde/src/org/aspectj/ajde/ui/internal/BrowserViewProperties.java b/ajde/src/org/aspectj/ajde/ui/internal/BrowserViewProperties.java
deleted file mode 100644 (file)
index 885279a..0000000
+++ /dev/null
@@ -1,169 +0,0 @@
-/* *******************************************************************
- * Copyright (c) 1999-2001 Xerox Corporation, 
- *               2002 Palo Alto Research Center, Incorporated (PARC).
- * 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 
- * http://www.eclipse.org/legal/epl-v10.html 
- *  
- * Contributors: 
- *     Xerox/PARC     initial implementation 
- * ******************************************************************/
-
-package org.aspectj.ajde.ui.internal;
-
-import java.io.File;
-import java.io.FileInputStream;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Properties;
-import java.util.StringTokenizer;
-
-import org.aspectj.ajde.Ajde;
-
-/**
- * @deprecated this class should be removed from the source tree
- */
-public class BrowserViewProperties {
-    public static final String FILE_NAME = "/.ajbrowser";
-    private final String VALUE_SEP = ";";
-    private Properties properties = new Properties();
-
-    public BrowserViewProperties() {
-        try {
-            if (new File(getPropertiesFilePath()).exists()) {
-                properties.load(new FileInputStream(getPropertiesFilePath()));
-            }
-        } catch (IOException ioe) {
-            Ajde.getDefault().getErrorHandler().handleError("Could not read properties", ioe);
-        }
-    }
-       
-    public List getStructureViewOrdering() {
-        return getPropertyValues("structureView.ordering");
-    }
-
-    public void setStructureViewOrdering(List ordering) {
-        storeProperty("structureView.ordering", ordering);
-    }
-
-//    /**
-//     * Returns default values if no associations are saved.
-//     */
-//    public List getActiveAssociations() {
-//        List associations = getPropertyValues("structureView.associations");
-//        if (associations.size() == 0) {
-//            associations.add(Advice.METHOD_CALL_SITE_RELATION.toString());
-//            associations.add(Advice.METHOD_RELATION.toString());
-//            associations.add(Advice.CONSTRUCTOR_CALL_SITE_RELATION.toString());
-//            associations.add(Advice.CONSTRUCTOR_RELATION.toString());
-//            associations.add(Advice.FIELD_ACCESS_RELATION.toString());
-//            associations.add(Advice.INITIALIZER_RELATION.toString());
-//            associations.add(Advice.HANDLER_RELATION.toString());
-//            associations.add(Advice.INTRODUCTION_RELATION.toString());
-//            associations.add(Introduction.INTRODUCES_RELATION.toString());
-//            associations.add(Inheritance.IMPLEMENTS_RELATION.toString());
-//            associations.add(Inheritance.INHERITS_RELATION.toString());
-//            associations.add(Inheritance.INHERITS_MEMBERS_RELATION.toString());
-//            associations.add(Reference.USES_POINTCUT_RELATION.toString());
-//            associations.add(Reference.IMPORTS_RELATION.toString());
-//        }
-//        return associations;
-//    }
-
-    public void setActiveAssociations(List associations) {
-        storeProperty("structureView.associations", associations);
-    }
-
-    public void setActiveFilteredMemberKinds(List associations) {
-        storeProperty("structureView.filtering.memberKinds", associations);
-    }
-
-    public String getActiveHierarchy() {
-       return getProperty("structureView.hierarchy");
-    }
-
-    public void setActiveHierarchy(String hierarchy) {
-        storeProperty("structureView.hierarchy", hierarchy);
-    }
-
-    public List getActiveVisibility() {
-        return getPropertyValues("structureView.filtering.accessibility");
-    }
-
-    public void setActiveVisiblity(List visibility) {
-        storeProperty("structureView.filtering.accessibility", visibility);
-    }
-
-    public List getActiveModifiers() {
-        return getPropertyValues("structureView.filtering.modifiers");
-    }
-
-    public List getActiveFilteredMemberKinds() {
-        return getPropertyValues("structureView.filtering.memberKinds");
-    }
-
-    public String getActiveGranularity() {
-        return getProperty("structureView.granularity");
-    }
-
-    public void setActiveGranularity(String granularity) {
-    storeProperty("structureView.granularity", granularity);
-    }
-
-    public void setActiveModifiers(List modifiers) {
-        storeProperty("structureView.filtering.modifiers", modifiers);
-    }
-
-    protected String getProperty(String name) {
-        return properties.getProperty(name);
-    }
-
-    protected List getPropertyValues(String name) {
-        List values = new ArrayList();
-        String valuesString = properties.getProperty(name);
-        if (valuesString != null && !valuesString.trim().equals("")) {
-            StringTokenizer st = new StringTokenizer(valuesString, VALUE_SEP);
-            while (st.hasMoreTokens()) {
-                values.add(st.nextToken());
-            }
-        }
-        return values;
-    }
-
-    private void storeProperty(String name, String value) {
-        properties.setProperty(name, value);
-        saveProperties();
-    }
-
-    private void storeProperty(String name, List values) {
-        String valuesString = "";
-        for (Iterator it = values.iterator(); it.hasNext(); ) {
-            valuesString += (String)it.next() + ';';
-        }
-        properties.setProperty(name, valuesString);
-        saveProperties();
-    }
-
-    private void saveProperties() {
-        try {
-            properties.store(new FileOutputStream(getPropertiesFilePath()), "AJDE Settings");
-        } catch (IOException ioe) {
-            Ajde.getDefault().getErrorHandler().handleError("Could not write properties", ioe);
-        }
-    }
-
-    protected String getPropertiesFilePath() {
-        String path = System.getProperty("user.home");
-        if (path == null) {
-            path = ".";
-        }
-        return path + FILE_NAME;
-    }
-}
index c80a06ae9f92a358c4782f5aaf5bab891c72df4f..7e791cc5c24ee659c94b05577e37c3b05f60b7bd 100644 (file)
@@ -8,7 +8,8 @@
  * http://www.eclipse.org/legal/epl-v10.html 
  *  
  * Contributors: 
- *     Xerox/PARC     initial implementation 
+ *     Xerox/PARC     initial implementation
+ *     Helen Hawkins  Converted to new interface (bug 148190) 
  * ******************************************************************/
 
  
@@ -27,6 +28,8 @@ import java.util.StringTokenizer;
 
 import org.aspectj.ajde.Ajde;
 import org.aspectj.ajde.ui.UserPreferencesAdapter;
+import org.aspectj.bridge.IMessage;
+import org.aspectj.bridge.Message;
 import org.aspectj.util.LangUtil;
 
 public class UserPreferencesStore implements UserPreferencesAdapter {
@@ -113,7 +116,8 @@ public class UserPreferencesStore implements UserPreferencesAdapter {
             in = new FileInputStream(file);
             properties.load(in);
         } catch (IOException ioe) {
-            Ajde.getDefault().getErrorHandler().handleError("Error reading properties from " + path, ioe);
+               Message msg = new Message("Error reading properties from " + path,IMessage.ERROR,ioe,null);
+               Ajde.getDefault().getMessageHandler().handleMessage(msg);
         } finally {
             if (null != in) {
                 try {
@@ -134,7 +138,8 @@ public class UserPreferencesStore implements UserPreferencesAdapter {
             out = new FileOutputStream(path);
             properties.store(out, "AJDE Settings");
         } catch (IOException ioe) {
-            Ajde.getDefault().getErrorHandler().handleError("Error writing properties to " + path, ioe);
+               Message msg = new Message("Error writing properties to " + path,IMessage.ERROR,ioe,null);
+               Ajde.getDefault().getMessageHandler().handleMessage(msg);
         } finally {
             if (null != out) {
                 try {