From e821cdabd2930d301d3a12a5d4d1fe8904cd1c14 Mon Sep 17 00:00:00 2001 From: Simon Rozman Date: Mon, 4 Feb 2019 11:42:51 +0100 Subject: Unify certain variable names Signed-off-by: Simon Rozman --- setupapi/setupapi_windows.go | 14 ++++++-------- setupapi/setupapi_windows_test.go | 8 ++++---- 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/setupapi/setupapi_windows.go b/setupapi/setupapi_windows.go index 95c5ba6..f6b35d2 100644 --- a/setupapi/setupapi_windows.go +++ b/setupapi/setupapi_windows.go @@ -104,7 +104,7 @@ func SetupDiGetClassDevsEx(ClassGUID *windows.GUID, Enumerator string, hwndParen } // SetupDiGetDeviceInfoListDetail function retrieves information associated with a device information set including the class GUID, remote computer handle, and remote computer name. -func SetupDiGetDeviceInfoListDetail(DeviceInfoSet DevInfo) (data *DevInfoListDetailData, err error) { +func SetupDiGetDeviceInfoListDetail(DeviceInfoSet DevInfo) (DeviceInfoSetDetailData *DevInfoListDetailData, err error) { var _data _SP_DEVINFO_LIST_DETAIL_DATA _data.Size = uint32(unsafe.Sizeof(_data)) @@ -113,12 +113,11 @@ func SetupDiGetDeviceInfoListDetail(DeviceInfoSet DevInfo) (data *DevInfoListDet return } - data = &DevInfoListDetailData{ + return &DevInfoListDetailData{ ClassGUID: _data.ClassGUID, RemoteMachineHandle: _data.RemoteMachineHandle, RemoteMachineName: windows.UTF16ToString(_data.RemoteMachineName[:]), - } - return + }, nil } // SetupDiEnumDeviceInfo function returns a SP_DEVINFO_DATA structure that specifies a device information element in a device information set. @@ -136,7 +135,7 @@ func SetupDiOpenDevRegKey(DeviceInfoSet DevInfo, DeviceInfoData *SP_DEVINFO_DATA } // SetupDiGetDeviceInstallParams function retrieves device installation parameters for a device information set or a particular device information element. -func SetupDiGetDeviceInstallParams(DeviceInfoSet DevInfo, DeviceInfoData *SP_DEVINFO_DATA) (data *DevInstallParams, err error) { +func SetupDiGetDeviceInstallParams(DeviceInfoSet DevInfo, DeviceInfoData *SP_DEVINFO_DATA) (DeviceInstallParams *DevInstallParams, err error) { var _data _SP_DEVINSTALL_PARAMS _data.Size = uint32(unsafe.Sizeof(_data)) @@ -145,7 +144,7 @@ func SetupDiGetDeviceInstallParams(DeviceInfoSet DevInfo, DeviceInfoData *SP_DEV return } - data = &DevInstallParams{ + return &DevInstallParams{ Flags: _data.Flags, FlagsEx: _data.FlagsEx, hwndParent: _data.hwndParent, @@ -153,8 +152,7 @@ func SetupDiGetDeviceInstallParams(DeviceInfoSet DevInfo, DeviceInfoData *SP_DEV InstallMsgHandlerContext: _data.InstallMsgHandlerContext, FileQueue: _data.FileQueue, DriverPath: windows.UTF16ToString(_data.DriverPath[:]), - } - return + }, nil } // SetupDiSetDeviceInstallParams function sets device installation parameters for a device information set or a particular device information element. diff --git a/setupapi/setupapi_windows_test.go b/setupapi/setupapi_windows_test.go index 76257ac..062b6bd 100644 --- a/setupapi/setupapi_windows_test.go +++ b/setupapi/setupapi_windows_test.go @@ -21,17 +21,17 @@ func init() { } func TestSetupDiClassNameFromGuidEx(t *testing.T) { - className, err := SetupDiClassNameFromGuidEx(&deviceClassNetGUID, "") + deviceClassNetName, err := SetupDiClassNameFromGuidEx(&deviceClassNetGUID, "") if err != nil { t.Errorf("Error calling SetupDiClassNameFromGuidEx: %s", err.Error()) - } else if strings.ToLower(className) != "net" { + } else if strings.ToLower(deviceClassNetName) != "net" { t.Errorf("SetupDiClassNameFromGuidEx(%x) should return \"Net\"", deviceClassNetGUID) } - className, err = SetupDiClassNameFromGuidEx(&deviceClassNetGUID, computerName) + deviceClassNetName, err = SetupDiClassNameFromGuidEx(&deviceClassNetGUID, computerName) if err != nil { t.Errorf("Error calling SetupDiClassNameFromGuidEx: %s", err.Error()) - } else if strings.ToLower(className) != "net" { + } else if strings.ToLower(deviceClassNetName) != "net" { t.Errorf("SetupDiClassNameFromGuidEx(%x) should return \"Net\"", deviceClassNetGUID) } -- cgit v1.2.3-54-g00ecf