> On Oct. 24, 2012, 11:09 a.m., Oliver Henshaw wrote:
> > powerdevil/daemon/powerdevilactionpool.cpp, lines 102-109
> > <http://git.reviewboard.kde.org/r/106863/diff/3/?file=90668#file90668line102>
> >
> > Is isSupported checked everywhere it should be? Remember that this
> > should work when DPMS is configured in but the display doesn't support DPMS
> > (e.g. qxl or vmware displays in a VM)
I ifdef'd out all the DPMS-related stuff but it would probably also be good to
add a
if (action) {
if (action->isSupported()) {
action->onProfileLoad();
}
} else { …
to powerdevilcore so it doesn't load an action if it is not supported.
- Kai Uwe
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106863/#review20781
-----------------------------------------------------------
On Oct. 15, 2012, 4:17 p.m., Kai Uwe Broulik wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106863/
> -----------------------------------------------------------
>
> (Updated Oct. 15, 2012, 4:17 p.m.)
>
>
> Review request for Solid.
>
>
> Description
> -------
>
> That message usually appears when starting before the Desktop is up, causing
> an ugly 1990's passivepopup dialog on the screen, and its contents are not
> really novice-user-resolvable.
> On my machine it always claims "The profile Battery tried to activate
> DPMSControl which is a non-existent action.", which is when I compile
> powerdevil myself that DPMS stuff is not compiled (DPMS build requirements
> not met here) and so the action floats around in the config but cannot be
> triggered anyways. (Imho this is a really infamous message, have seen it
> quite often on other machines *duck*). All the other actions seem to be
> installed anyways, so this missing action poses no threat. I guess a kWarning
> would be sufficient for this.
>
>
> Diffs
> -----
>
> powerdevil/daemon/actions/CMakeLists.txt db9ca47
> powerdevil/daemon/actions/dpms/powerdevildpmsaction.cpp a16bf7e
> powerdevil/daemon/powerdevilactionpool.cpp 484c271
>
> Diff: http://git.reviewboard.kde.org/r/106863/diff/
>
>
> Testing
> -------
>
> Compiles.
> The previous passivepopup does not appear anymore. Did not test whether the
> kwarning is triggered, though. (Dunno how to get powerdevil debug console
> output)
>
>
> Thanks,
>
> Kai Uwe Broulik
>
>
_______________________________________________
Kde-hardware-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-hardware-devel
