Changeset 1705


Ignore:
Timestamp:
1/27/2010 5:58:28 AM (3 years ago)
Author:
lowjoel
Message:

Replace all Marshal.GetExceptionForHR with Win32ErrorCode.GetExceptionForWin32Error since we deal with Win32 errors not unlike COM errors, except with a few exceptions and that we should be throwing Win32Exception instead of COMException.

Location:
branches/eraser6/CodeReview/Eraser.Util
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • branches/eraser6/CodeReview/Eraser.Util/File.cs

    r1681 r1705  
    163163                throw new IOException(string.Format(CultureInfo.CurrentCulture, 
    164164                    "Could not load file icon from {0}", path), 
    165                     Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error())); 
     165                    Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error())); 
    166166        } 
    167167 

  • branches/eraser6/CodeReview/Eraser.Util/NtfsApi.cs

    r1681 r1705  
    7575                } 
    7676 
    77                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     77                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    7878            } 
    7979        } 

  • branches/eraser6/CodeReview/Eraser.Util/StreamInfo.cs

    r1701 r1705  
    362362            //Check that the handle is valid 
    363363            if (handle.IsInvalid) 
    364                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     364                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    365365 
    366366            //Return the FileStream 

  • branches/eraser6/CodeReview/Eraser.Util/VolumeInfo.cs

    r1701 r1705  
    7272 
    7373                    default: 
    74                         throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     74                        throw Win32ErrorCode.GetExceptionForWin32Error(lastError); 
    7575                } 
    7676            } 
    … …  
    326326                                break; 
    327327                            default: 
    328                                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     328                                throw Win32ErrorCode.GetExceptionForWin32Error( 
     329                                    Marshal.GetLastWin32Error()); 
    329330                        } 
    330331                    } 
    … …  
    382383                } 
    383384 
    384                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     385                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    385386            } 
    386387        } 
    … …  
    400401                } 
    401402 
    402                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     403                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    403404            } 
    404405        } 
    … …  
    423424                } 
    424425 
    425                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     426                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    426427            } 
    427428        } 
    … …  
    445446                } 
    446447 
    447                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     448                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    448449            } 
    449450        } 
    … …  
    462463                } 
    463464 
    464                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     465                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    465466            } 
    466467        } 
    … …  
    479480                } 
    480481 
    481                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     482                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    482483            } 
    483484        } 
    … …  
    587588            //Check that the handle is valid 
    588589            if (handle.IsInvalid) 
    589                 throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     590                throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    590591 
    591592            //Return the FileStream 
    … …  
    646647                    //Check that the handle is valid 
    647648                    if (handle.IsInvalid) 
    648                         throw Marshal.GetExceptionForHR(Marshal.GetHRForLastWin32Error()); 
     649                        throw Win32ErrorCode.GetExceptionForWin32Error(Marshal.GetLastWin32Error()); 
    649650 
    650651                    //This only works if the user has turned on the disk performance 

  • branches/eraser6/CodeReview/Eraser.Util/Win32ErrorCodes.cs

    r1701 r1705  
    2525using System.Runtime.InteropServices; 
    2626using System.IO; 
     27using System.ComponentModel; 
    2728 
    2829namespace Eraser.Util 
    … …  
    5354                case NoError:           return null; 
    5455                case SharingViolation:  return new IOException(); 
     56            } 
    5557 
    56                 default: 
    57                     int HR = GetHRForWin32Error(errorCode); 
    58                     return Marshal.GetExceptionForHR(HR); 
    59             } 
     58            int HR = GetHRForWin32Error(errorCode); 
     59            Exception exception = Marshal.GetExceptionForHR(HR); 
     60            if (exception.GetType() == typeof(COMException)) 
     61                throw new Win32Exception(errorCode); 
     62            else 
     63                throw exception; 
    6064        } 
    6165 
Note: See TracChangeset for help on using the changeset viewer.