Changeset 1590


Ignore:
Timestamp:
1/19/2010 2:43:39 AM (4 years ago)
Author:
lowjoel
Message:

Fixed warning CA1800: Microsoft.Performance : 'control', a parameter, is cast to type multiple times in method 'Theming.ApplyTheme?(Control)'.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/eraser6/CodeReview/Eraser.Util/Theming.cs

    r1584 r1590  
    5757        public static void ApplyTheme(Control control) 
    5858        { 
    59             if (control is ContainerControl) 
    60                 ((ContainerControl)control).Font = SystemFonts.MessageBoxFont; 
     59            ContainerControl container = control as ContainerControl; 
     60            ButtonBase button = control as ButtonBase; 
     61            ListView listview = control as ListView; 
     62            ToolStrip toolstrip = control as ToolStrip; 
     63 
     64            if (container != null) 
     65                container.Font = SystemFonts.MessageBoxFont; 
    6166            else if (control.Font != SystemFonts.MessageBoxFont) 
    6267                control.Font = new Font(SystemFonts.MessageBoxFont.FontFamily, 
    6368                    control.Font.Size, control.Font.Style); 
    6469 
    65             if (control is ButtonBase) 
    66                 ApplyTheme((ButtonBase)control); 
    67             else if (control is ListView) 
    68                 ApplyTheme((ListView)control); 
    69             else if (control is ToolStrip) 
    70                 ApplyTheme((ToolStrip)control); 
     70            if (button != null) 
     71                ApplyTheme(button); 
     72            else if (listview != null) 
     73                ApplyTheme(listview); 
     74            else if (toolstrip != null) 
     75                ApplyTheme(toolstrip); 
    7176 
    7277            if (control.ContextMenuStrip != null) 
Note: See TracChangeset for help on using the changeset viewer.