Re: [orca-list] heads up/please test
- From: "ashley" <ash cox talktalk net>
- To: "Joanmarie Diggs" <joanmarie diggs gmail com>
- Cc: orca mailing list <orca-list gnome org>
- Subject: Re: [orca-list] heads up/please test
- Date: Tue, 27 Apr 2010 07:48:25 +0100
Just out of curiosity, will this make orca's performance faster?
Thanks,
Ash
Website:
http://www.ashleycox.co.uk/
Podcast:
http://www.bavis.co.uk/theactechpodcast
Blog:
http://daysofthegeek.blogspot.com
----- Original Message -----
From: "Joanmarie Diggs" <joanmarie diggs gmail com>
To: <orca-list gnome org>
Sent: Tuesday, April 27, 2010 4:56 AM
Subject: [orca-list] heads up/please test
Hey all.
Today I was looking at some performance-related issues in Orca. One of
them is that we regularly spend 0.01 to 0.25 seconds in updateBraille().
We do this automatically, whether the user has braille and/or braille
monitor support enabled or disabled.
So there are two questions:
* Why up to 0.25 seconds? Answer: Some of that seems to be related to
Mozilla bugs and other issues. (We didn't spend that much time when in
other applications.)
* Why are we spending any time whatsoever in updateBraille() if the user
doesn't have braille and/or braille monitor support enabled? Answer:
Yeah.... This is where I decided to start.
Unfortunately, it's not as simple as adding a check in updateBraille()
which returns immediately if braille support is disabled. Doing so winds
up breaking things elsewhere. We need to add checks pretty much anywhere
there is a braille-related call. And that, it turns out, is all over the
place throughout Orca. The majority are in various and sundry scripts
which for often-valid reasons are cobbling together their own braille
presentation.
So....
I opened bug 616848 - Application and toolkit scripts should not call
methods in braille.py. And I've started the conversion. The conversion
is a work in progress which will be committed to master in pieces. The
first chunk has just been committed -- after pylinting, full regression
testing, and a lot of careful examination of debug.outs to be sure the
changes being made were sound. Nonetheless, your testing would be
greatly appreciated.
Please note:
1. Just because you don't use braille doesn't mean you can't test these
changes. Remember, right now Orca's braille functionality is kicking in
everywhere, even if you don't have a braille display connected and even
if you've disabled braille output. The *ultimate* goal is to stop that
from happening. Bug 616848's goal is simply to make it possible for us
to work on the ultimate goal.
2. I can see many of you thinking, "I better test Firefox!" I'm all for
that. But please test the other apps as well. For one thing those other
apps need testing. For another, the chunk I've committed to master
doesn't touch the Gecko script. The Gecko script is currently unchanged.
I'll send out another "heads up/please test" message when I change stuff
for bug 616848 in the Gecko script.
As a reminder, it's extremely helpful when testing if you can verify
that the bug you find does exist in the current revision from master,
but doesn't exist in the previous revision.
Thanks very much in advance!
--joanie
_______________________________________________
orca-list mailing list
orca-list gnome org
http://mail.gnome.org/mailman/listinfo/orca-list
Visit http://live.gnome.org/Orca for more information on Orca.
The manual is at
http://library.gnome.org/users/gnome-access-guide/nightly/ats-2.html
The FAQ is at http://live.gnome.org/Orca/FrequentlyAskedQuestions
Netiquette Guidelines are at
http://live.gnome.org/Orca/FrequentlyAskedQuestions/NetiquetteGuidelines
Log bugs and feature requests at http://bugzilla.gnome.org
Find out how to help at http://live.gnome.org/Orca/HowCanIHelp
--------------------------------------------------------------------------------
No virus found in this incoming message.
Checked by AVG - www.avg.com
Version: 9.0.814 / Virus Database: 271.1.1/2836 - Release Date: 04/26/10
07:31:00
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]