dexapier / android-scripting

Automatically exported from code.google.com/p/android-scripting
0 stars 0 forks source link

App frozen when adding a phone trigger #544

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What device(s) are you experiencing the problem on?
HTC Desire Z

What firmware version are you running on the device?
Android 2.2.1
2.6.32.21 kernel
1.82.405.1 software number

What steps will reproduce the problem?
1. Open SL4A
2. Menu -> View -> Triggers
3. Menu -> Add
4. Select a script
5. Select "phone" as the trigger kind

What is the expected output? What do you see instead?
The app is frozen. It should not freeze and create the trigger.

What version of the product are you using? On what operating system?
SL4A R4 on Android 2.2.1.
Also tried with source build, and got the same defect.

Please provide any additional information below.

It looks like a deadlock, as the MainThread.run() method is executed in the 
main thread associated with the supplied context, so the supplied task cannot 
be run until the MainThread.run() finishes, and MainThread.run() does not 
finish if the task is not ran.
I solved the problem with the attached patch.

Original issue reported on code.google.com by alessand...@gmail.com on 4 May 2011 at 8:31

Attachments:

GoogleCodeExporter commented 9 years ago
....sorry for the log file extension... obviously it's a txt.
And the patch I attached is wrong... I am deleting it. This is the good one.

Original comment by alessand...@gmail.com on 4 May 2011 at 8:37

Attachments:

GoogleCodeExporter commented 9 years ago
If someone is interested, have created a patched apk that fixes this issue 
which can be downloaded here:

http://www.alessandropira.org/sl4a/sl4a.html

Original comment by alessand...@gmail.com on 27 May 2011 at 11:57

GoogleCodeExporter commented 9 years ago
Thanks. Will investigate.

Looking at incorporating your fixes in the next release, but you'll need to 
sing a CLA:

http://code.google.com/p/android-scripting/wiki/DeveloperGuide#Contributor_Licen
se_Agreements

You must sign a Contributor License Agreement (CLA) before we can
accept any code. The CLA protects you and us. It's quick and painless,
just type in your name and click submit.

Let us know when it's done.

Original comment by rjmatthews62 on 21 Aug 2011 at 12:40

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Done.

Original comment by alessand...@gmail.com on 23 Aug 2011 at 7:10