Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Win10] Implementation UWP own power management and storage provider #13254

Merged
merged 6 commits into from
Dec 28, 2017

Conversation

afedchin
Copy link
Member

let's make things right :)

@afedchin afedchin added this to the L 18.0-alpha1 milestone Dec 26, 2017
@afedchin afedchin requested a review from Paxxi December 26, 2017 19:28
@afedchin afedchin force-pushed the win10-fix branch 3 times, most recently from e355de6 to ade567f Compare December 27, 2017 15:16
Copy link
Member

@Paxxi Paxxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to in general.

Fix the unique_ptr<wchar_t[]> and filename and it's good to go.

ULARGE_INTEGER ULTotal = { { 0 } };
ULARGE_INTEGER ULTotalFree = { { 0 } };

std::unique_ptr<wchar_t> pcBuffer;

This comment was marked as spam.

This comment was marked as spam.

{
using KODI::PLATFORM::WINDOWS::FromW;

WCHAR* pcBuffer= NULL;

This comment was marked as spam.

#include "powermanagement/windows/Win32PowerSyscall.h"
extern HWND g_hWnd;
#elif defined(TARGET_WINDOWS_STORE)
#include "powermanagement/win10/PowerSyscallWin10.h"

This comment was marked as spam.


using namespace Windows::Devices::Power;

CPowerSyscall::CPowerSyscall()

This comment was marked as spam.

{
}

CPowerSyscall::~CPowerSyscall()

This comment was marked as spam.

@@ -100,7 +157,11 @@ bool CWin32PowerSyscall::CanReboot()

int CWin32PowerSyscall::BatteryLevel()
{
return CWIN32Util::BatteryLevel();
int result = 0;

This comment was marked as spam.

@afedchin
Copy link
Member Author

jenkins is happy see BuildMulti-PR-Manually#123

@MartijnKaijser MartijnKaijser merged commit ce9a784 into xbmc:master Dec 28, 2017
@afedchin afedchin deleted the win10-fix branch December 28, 2017 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants