This bug was fixed in the package gnome-power-manager - 2.24.2-2ubuntu1
---
gnome-power-manager (2.24.2-2ubuntu1) jaunty; urgency=low
[ Ted Gould ]
* Merge from Debian experimental.
* Remove 20_fix_precentage_closures.patch as already applied upstream.
* Merge in the mactel
Repackaged the patch a bit, hopefully fits together with the ubuntu
patches a bit better. The two following attachments replace the old
patch.
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug notification because you are a
** Attachment added: 90-Add-guarded-brightness-stepping-functions.patch
http://launchpadlibrarian.net/18940342/90-Add-guarded-brightness-stepping-functions.patch
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug
** Attachment added: 91-Using-guarded-and-scaled-stepping-when-dimming.patch
http://launchpadlibrarian.net/18940344/91-Using-guarded-and-scaled-stepping-when-dimming.patch
** Changed in: mactel-support
Assignee: (unassigned) = rydberg (rydberg)
** Changed in: mactel-support
I don't understand why the patch has to change the type of loop being
used. Why can't it just use the new function as the increment or
decrement functions in the for loops? It would make the patch much
smaller.
--
GPM unresponsive for two minutes after startup on Macbooks
I presume you are talking about patch 91.
The current code is executing the body of the for loop for both end
points, cur and shared_value_abs, Besides being unnecessary, it means
the loop exits for a value outside of the valid range. This makes it
impossible to use the guarded_up/down functions,
... make that 20 lines; the going down comment had been strangely
misplaced.
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
On Mon, 2008-10-27 at 21:40 +, rydberg wrote:
It *is* possible to make the patch smaller, by 18 lines, by leaving the
if statements untouched, like this:
snip
However, the resulting code looks rather artifical. Would you prefer a
patch base on the above?
Yes, I think that's better. Not
** Attachment removed: 91-Using-guarded-and-scaled-stepping-when-
dimming.patch
http://launchpadlibrarian.net/18940344/91-Using-guarded-and-scaled-
stepping-when-dimming.patch
** Attachment removed: use-scaled-stepping-in-xrand.patch
True. I removed the old attachments and added new ones accordingly. I
also took the liberty of changing patch 90 to use non-linear stepping;
it makes every step roughly equal in perceived brightness change, and
allows for much finer tuning at low brightness.
** Attachment added:
** Attachment added: 91-Using-guarded-and-scaled-stepping-when-dimming.patch
http://launchpadlibrarian.net/18957594/91-Using-guarded-and-scaled-stepping-when-dimming.patch
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this
** Attachment added: use-scaled-stepping-in-xrand.patch
http://launchpadlibrarian.net/18921173/use-scaled-stepping-in-xrand.patch
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug notification because you are a member
The patch has been uploaded to the mactel PPA.
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
ubuntu-bugs mailing list
** Also affects: mactel-support
Importance: Undecided
Status: New
--
GPM unresponsive for two minutes after startup on Macbooks
https://bugs.launchpad.net/bugs/289520
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
14 matches
Mail list logo