Changeset 2504 for branches/PluginInfrastructure Refactoring/HeuristicLab.PluginInfrastructure/Attributes/ApplicationAttribute.cs

Timestamp:

11/18/09 18:33:30 (14 years ago)

Author:

gkronber

Message:

Worked on core of plugin infrastructure.

• Collected all classes into a single assembly (HL.PluginInfrastructure)
• Moved SplashScreen and MainForm from HeuristicLab.exe project into the plugin infrastructure.
• Introduced namespaces
• Added strict access modifiers (internal)
• Fixed most FxCop warnings in plugin infrastructure core.
• Fixed issues with plugin load/unload events
• Deleted empty interface IControl

#799

File:

• branches/PluginInfrastructure Refactoring/HeuristicLab.PluginInfrastructure/Attributes/ApplicationAttribute.cs (modified) (4 diffs)

branches/PluginInfrastructure Refactoring/HeuristicLab.PluginInfrastructure/Attributes/ApplicationAttribute.cs

@@ -27 +27 @@
   /// <summary>
   /// This attribute can be used to specify meta data for applications. 
-  /// For example to specify name, version and description of applications.
+  /// For example to specify name and description of applications.
   /// </summary>
   [AttributeUsage(AttributeTargets.Class)]
-  public class ApplicationAttribute : System.Attribute {
+  public sealed class ApplicationAttribute : System.Attribute {
     private string name;
     /// <summary>
-    /// Gets or sets the name of the application.
+    /// Gets the name of the application.
     /// </summary>
     public string Name {
       get { return name; }
-      set { name = value; }
     }
-
-    //private string version;
-    ///// <summary>
-    ///// Gets or sets the version of the application.
-    ///// </summary>
-    //public string Version {
-    //  get { return version; }
-    //  set { version = value; }
-    //}
 
     private string description;
     /// <summary>
-    /// Gets or sets the description of the application.
+    /// Gets the description of the application.
     /// </summary>
     public string Description {
       get { return description; }
-      set { description = value; }
     }
 
     private bool restartOnErrors;
     /// <summary>
-    /// Gets or sets the boolean flag whether the plugin should be automatically restarted when it is closed because of an exception (for services).
+    /// Gets whether the plugin should be automatically restarted when it is closed because of an exception (for services).
     /// </summary>
     public bool RestartOnErrors {
       get { return restartOnErrors; }
-      set { restartOnErrors = value; }
     }
 
@@ -72 +60 @@
     /// </summary>
     public ApplicationAttribute(string name)
-      : this(name, "") {
+      : this(name, String.Empty) {
     }
 
@@ -84 +72 @@
     }
 
-    ///// <summary>
-    ///// Initializes a new instance of <see cref="ApplicationAttribute"/>.
-    ///// <param name="name">Name of the application</param>
-    ///// <param name="description">Description of the application</param>
-    ///// <param name="version">Version string of the application</param>
-    ///// </summary>
-    //public ApplicationAttribute(string name, string description, string version)
-    //  : this(name, description, version, false) {
-    //}
-
     /// <summary>
     /// Initializes a new instance of <see cref="ApplicationAttribute"/>.
@@ -102 +80 @@
     /// </summary>
     public ApplicationAttribute(string name, string description, bool restartOnErrors) {
+      if (name == null) throw new ArgumentNullException("name", "Application name is null.");
+      if (description == null) throw new ArgumentNullException("description", "Application description is null.");
       this.name = name;
       this.description = description;