Merge lp:~pete-woods/hud/voice-bug-1177226 into lp:hud/13.10

Proposed by Pete Woods
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 253
Merged at revision: 253
Proposed branch: lp:~pete-woods/hud/voice-bug-1177226
Merge into: lp:hud/13.10
Diff against target: 27 lines (+12/-5)
1 file modified
src/hudsphinx.c (+12/-5)
To merge this branch: bzr merge lp:~pete-woods/hud/voice-bug-1177226
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Indicator Applet Developers Pending
Review via email: mp+162792@code.launchpad.net

Commit message

Fix voice on the device

Have to duplicate the call to cmd_ln_init to handle each case, it won't accept NULL arguments.

Description of the change

Fix voice on the device

Have to duplicate the call to cmd_ln_init to handle each case, it won't accept NULL arguments.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:253
http://jenkins.qa.ubuntu.com/job/hud-ci/17/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/hud-raring-amd64-ci/21
    SUCCESS: http://jenkins.qa.ubuntu.com/job/hud-raring-i386-ci/6

Click here to trigger a rebuild:
http://s-jenkins:8080/job/hud-ci/17/rebuild

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Works

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/hudsphinx.c'
--- src/hudsphinx.c 2013-03-13 17:31:24 +0000
+++ src/hudsphinx.c 2013-05-07 15:01:49 +0000
@@ -122,11 +122,18 @@
122 gchar *hmm = "/usr/share/pocketsphinx/model/hmm/en_US/hub4wsj_sc_8k";122 gchar *hmm = "/usr/share/pocketsphinx/model/hmm/en_US/hub4wsj_sc_8k";
123 gchar *dict = "/usr/share/pocketsphinx/model/lm/en_US/cmu07a.dic";123 gchar *dict = "/usr/share/pocketsphinx/model/lm/en_US/cmu07a.dic";
124124
125 self->config = cmd_ln_init(NULL, sphinx_cmd_ln, TRUE,125 if (device != NULL) {
126 "-hmm", hmm,126 self->config = cmd_ln_init(NULL, sphinx_cmd_ln, TRUE,
127 "-dict", dict,127 "-hmm", hmm,
128 "-adcdev", device == NULL ? "" : device,128 "-dict", dict,
129 NULL);129 "-adcdev", device,
130 NULL);
131 } else {
132 self->config = cmd_ln_init(NULL, sphinx_cmd_ln, TRUE,
133 "-hmm", hmm,
134 "-dict", dict,
135 NULL);
136 }
130137
131 if (self->config == NULL) {138 if (self->config == NULL) {
132 g_warning("Sphinx command line arguments failed to initialize");139 g_warning("Sphinx command line arguments failed to initialize");

Subscribers

People subscribed via source and target branches