FBMREMOTE - Utilisation de 100% du CPU

[GoogleCodeExporter]

A l'utilisation de la télécommande, le processus prend très rapidement 100% 
du CPU (visible avec ASTRO entre autre, processus EarlyPropale).

Détails de l'environnement :
- Android 2.2
- Mobiles : HTC Desire et LG Optimus 2X
- Operateur : Bouygues et Virgin
- Version : EarlyPropale 1.0.2
- Wifi uniquement
- Aucun log particulier dans Logcat

Pour reproduire ce bug il suffit de lancer la télécommande et de 
sélectionner un boitier valide (configuré).

Je ne me suis rendu compte du problème qu'en voyant ma propre application en 
cours de développement être particulièrement peu performante. La fluidité 
du mobile n'étant pas particulièrement impactée (par contre je suppose que 
la batterie en prend un coup).

- Explications techniques :

Le problème vient de la classe Main du projet EarlyProbale.
Pour être plus précis, du popup demandant le choix du boitier.
En effet, lors d'un click sur un des boitiers (seulement le 1 dans mon cas), le 
boitier est sélectionné et la méthode dialog.cancel() est appelée.
Un "cancelListener" étant positionné, il est logiquement appelé (extrait de 
la javadoc : Cancel the dialog. This is essentially the same as calling 
dismiss(), but it will also call your DialogInterface.OnCancelListener (if 
registered)).
Le problème se trouve justement dans le "cancelListener" car il y a de nouveau 
un appel à la méthode dialog.cancel() => boucle infinie

- Résolution :

Dans le "cancelListener", pour faire disparaitre le popup il faut appeler la 
méthode dismiss() et non pas cancel() afin d'éviter la récursivité. Je 
joins un patch (vraiment minimaliste :) ) qui corrige le problème chez moi. Je 
ne pense pas qu'il puisse y avoir d'effets collatéraux mais à tester tout de 
même.

Original issue reported on code.google.com by charles....@gmail.com on 30 Apr 2011 at 9:24

Attachments:

• Main.diff

[GoogleCodeExporter]

Bonjour,

merci beaucoup de votre aide, nous sommes en train de corriger l'ensemble des 
lignes ou cela apparait

Original comment by clement....@gmail.com on 10 May 2011 at 2:47

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by clement....@gmail.com on 10 May 2011 at 2:48