(MSVC, C++11) Add Network information(interface index, MAC, IPv4, Description) #65
Conversation
include/hwinfo/bios.h
Outdated
There was a problem hiding this comment.
Can you add dummy implementations for Linux (and OSX) for this classes member functions? They can just return "" but they need to be implemented.
include/hwinfo/cpu.h
Outdated
| @@ -36,6 +36,7 @@ class CPU { | |||
| ~CPU() = default; | |||
|
|
|||
| int id() const; | |||
| const std::string& processorId() const; | |||
There was a problem hiding this comment.
Not sure if this information is useful at all. Maybe just remove it for the time being
include/hwinfo/network.h
Outdated
| @@ -0,0 +1,35 @@ | |||
| // Copyright MRsoymilk | |||
There was a problem hiding this comment.
I cannot accept copyrighted content for this project
include/hwinfo/network.h
Outdated
There was a problem hiding this comment.
Could you add Linux (and maybe OSX) dummy implementations for this classes member functions too?
include/hwinfo/network.h
Outdated
| Network() = default; | ||
| std::string _interfaceIndex{}; | ||
| std::string _mac{}; | ||
| std::string _ipv4{}, _ipv6{}; |
There was a problem hiding this comment.
please write them in separate statements for readability
src/CMakeLists.txt
Outdated
There was a problem hiding this comment.
Once you have added the dummy implementations, you can add the linux and OSX cpp files here too
src/network.cpp
Outdated
| @@ -0,0 +1,24 @@ | |||
| // Copyright MRsoymilk | |||
There was a problem hiding this comment.
I cannot accept Copyright code for this project
src/windows/cpu.cpp
Outdated
| @@ -91,7 +91,7 @@ std::vector<double> CPU::threadsUtilisation() const { | |||
| std::vector<CPU> getAllCPUs() { | |||
| utils::WMI::_WMI wmi; | |||
| const std::wstring query_string( | |||
| L"SELECT Name, Manufacturer, NumberOfCores, NumberOfLogicalProcessors, MaxClockSpeed, L2CacheSize, L3CacheSize " | |||
| L"SELECT Name, Manufacturer, NumberOfCores, NumberOfLogicalProcessors, MaxClockSpeed, ProcessorId " | |||
src/windows/network.cpp
Outdated
| @@ -0,0 +1,80 @@ | |||
| // Copyright (c) MRsoymilk <[email protected]> | |||
There was a problem hiding this comment.
i cannot accept copyright code for this project
src/windows/utils/wmi_wrapper.cpp
Outdated
| @@ -11,9 +11,10 @@ namespace utils { | |||
| namespace WMI { | |||
|
|
|||
| _WMI::_WMI() { | |||
| auto res = CoInitializeSecurity(nullptr, -1, nullptr, nullptr, RPC_C_AUTHN_LEVEL_DEFAULT, RPC_C_IMP_LEVEL_IMPERSONATE, | |||
| HRESULT res{-1}; | |||
There was a problem hiding this comment.
better initialize res with the return value of the first statement
|
it seems a lot of things that need to be modified and it will take me some time to do |
|
I am a time traveler from 3023. The spacecraft malfunction makes it impossible to modify... |
The information shows below:
