Skip to content

Commit

Permalink
Reworked API breaking change
Browse files Browse the repository at this point in the history
  • Loading branch information
nefarius committed Aug 9, 2024
1 parent 0df94e3 commit 317b5ab
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 14 deletions.
28 changes: 15 additions & 13 deletions src/NefConUtil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,9 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

GUID clID;
const auto guid = nefarius::winapi::GUIDFromString(classGuid);

if (!nefarius::winapi::GUIDFromString(classGuid, &clID))
if (!guid)
{
logger->error(
"Device Class GUID format invalid, expected format (with or without brackets): xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx");
Expand All @@ -138,7 +138,7 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

auto ret = AddDeviceClassFilter(&clID, nefarius::utilities::ConvertAnsiToWide(serviceName), pos);
auto ret = AddDeviceClassFilter(&guid.value(), nefarius::utilities::ConvertAnsiToWide(serviceName), pos);

if (ret)
{
Expand Down Expand Up @@ -173,9 +173,9 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

GUID clID;
const auto guid = nefarius::winapi::GUIDFromString(classGuid);

if (!nefarius::winapi::GUIDFromString(classGuid, &clID))
if (!guid)
{
logger->error(
"Device Class GUID format invalid, expected format (with or without brackets): xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx");
Expand All @@ -200,7 +200,7 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

auto ret = RemoveDeviceClassFilter(&clID, nefarius::utilities::ConvertAnsiToWide(serviceName), pos);
auto ret = RemoveDeviceClassFilter(&guid.value(), nefarius::utilities::ConvertAnsiToWide(serviceName), pos);

if (ret)
{
Expand Down Expand Up @@ -393,17 +393,18 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

GUID clID;
const auto guid = nefarius::winapi::GUIDFromString(classGuid);

if (!nefarius::winapi::GUIDFromString(classGuid, &clID))
if (!guid)
{
logger->error(
"Device Class GUID format invalid, expected format (with or without brackets): xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx");
return EXIT_FAILURE;
}

auto ret = nefarius::devcon::Create(nefarius::utilities::ConvertAnsiToWide(className), &clID,
nefarius::utilities::WideMultiStringArray(nefarius::utilities::ConvertAnsiToWide(hwId)));
auto ret = nefarius::devcon::Create(nefarius::utilities::ConvertAnsiToWide(className), &guid.value(),
nefarius::utilities::WideMultiStringArray(
nefarius::utilities::ConvertAnsiToWide(hwId)));

if (!ret)
{
Expand Down Expand Up @@ -435,9 +436,9 @@ int main(int argc, char* argv[])
return EXIT_FAILURE;
}

GUID clID;
const auto guid = nefarius::winapi::GUIDFromString(classGuid);

if (!nefarius::winapi::GUIDFromString(classGuid, &clID))
if (!guid)
{
logger->error(
"Device Class GUID format invalid, expected format (with or without brackets): xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx");
Expand All @@ -446,7 +447,8 @@ int main(int argc, char* argv[])

bool rebootRequired;

auto results = devcon::uninstall_device_and_driver(&clID, nefarius::utilities::ConvertAnsiToWide(hwId), &rebootRequired);
auto results = devcon::uninstall_device_and_driver(&guid.value(), nefarius::utilities::ConvertAnsiToWide(hwId),
&rebootRequired);

// TODO: finish proper error propagation!
for (const auto& item : results)
Expand Down
2 changes: 1 addition & 1 deletion vcpkg-configuration.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
{
"kind": "git",
"repository": "https://github.com/nefarius/nefarius-vcpkg-registry.git",
"baseline": "f05fbd1c3c85e36094132725878d3d03e3baf1fb",
"baseline": "710c5dff329f590618dc22c39823356238465499",
"packages": [ "neflib" ]
}
],
Expand Down

0 comments on commit 317b5ab

Please sign in to comment.