Ignore:
Timestamp:
4/28/2009 10:34:04 AM (6 years ago)
Author:
lowjoel
Message:

Ran Static code analysis on the Eraser project and implemented a few recommendations.

-Use more specific exceptions to allow code to determine the type of exception.
-Made GUIProgram implement IDisposable since the global mutex must be freed.
-Mde AboutForm? implement IDisposable, clearing the caching bitmaps on dispose.
-Made a few CommandLineProgram? functions static since they don't reference this.
-Name parameters/local with more unique, descriptive names for clarity.
-Use EventHandler?<EventArg?> instead of declaring our own delegate and event types as the type of the evenr handler

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/eraser6/Eraser/AboutForm.Designer.cs

    r856 r903  
    3939                components.Dispose(); 
    4040            } 
     41 
     42            ParentBitmap.Dispose(); 
     43            AboutBitmap.Dispose(); 
     44            AboutTextBitmap.Dispose(); 
     45            DoubleBufferBitmap.Dispose(); 
    4146            base.Dispose(disposing); 
    4247        } 
Note: See TracChangeset for help on using the changeset viewer.