On Sat 02. Sep - 02:20:51, Sebastian Kügler wrote:
> On Saturday 02 September 2006 00:44, Holger Macht wrote:
> > On Fri 01. Sep - 15:46:31, Pat Double wrote:
> > > If this is not already done, please consider this patch to detect
> > > suspend2 at the new /sys/power/suspend2 location.
>
> > > bool PM_Interface::haveSuspend2()
> > > {
> > > - if (access("/proc/suspend2", F_OK) < 0) {
> > > + if (access("/proc/suspend2", F_OK) < 0
> > > + && access("/sys/power/suspend2", F_OK) < 0) {
> > > return false;
> > > }
> >
> > Wouldn't a logical 'or' be better here? Or have both kernel interfaces to
> > exist always?
>
> No, only one of them is there at the same time.
Oh well, now I see that 'return false;' statement :-(
Thanks, committed.
Regards,
Holger
_______________________________________________
powersave-devel mailing list
powersave-devel@forge.novell.com
http://forge.novell.com/mailman/listinfo/powersave-devel
