<div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">><br>
>  def setup(tray):<br>
> -    tray.add_device(SpeechDeviceView())<br>
> +    try:<br>
> +        tray.add_device(SpeechDeviceView())<br>
> +    except:<br>
> +        logging.warning('Text to speech no available. Missing dependencies')<br>
<br>
</div></div>Ideally we shouldn't be just catching generic exceptions like above but<br>
rather specific...<br>
<br>
except <SomeException>:<br>
<br>
Also, instead of logging.warning, maybe a logging.exception would be<br>
better?<br>
<div class="im"><br></div></blockquote><div><br></div><div>In this case, any exception will mean the service is unavailable.</div><div>I think is better see the error in the log, and a exception hide the problem.</div><div>
Really, we have more important problems to solve....</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> diff --git a/extensions/globalkey/speech.py b/extensions/globalkey/speech.py<br>
> index 2879b69..e47c778 100644<br>
> --- a/extensions/globalkey/speech.py<br>
> +++ b/extensions/globalkey/speech.py<br>
> @@ -14,10 +14,9 @@<br>
>  # along with this program; if not, write to the Free Software<br>
>  # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA<br>
><br>
> -from jarabe.model import speech<br>
> -<br>
>  BOUND_KEYS = ['<alt><shift>s']<br>
><br>
> +from jarabe.model import speech<br>
><br>
<br>
</div>Why the need to move this import below the const declaration? Its not<br>
that way in other extension modules.<br>
<div class="im"><br></div></blockquote><div><br></div><div>My mistake :(</div><div><br></div><div>Gonzalo </div></div>
</div>