search

freeswitch / mod_unimrcp

Other
27 stars 28 forks source link

I think there should be a 'break' after line 1063 #9

Open kelehaishiyao opened 1 year ago

kelehaishiyao commented 1 year ago 
while (schannel->state == SPEECH_CHANNEL_CLOSED) {
    if (switch_thread_cond_timedwait(schannel->cond, schannel->mutex, SPEECH_CHANNEL_TIMEOUT_USEC) == SWITCH_STATUS_TIMEOUT && !warned) {
        warned = 1;
        switch_log_printf(SWITCH_CHANNEL_UUID_LOG(schannel->session_uuid), SWITCH_LOG_WARNING, "(%s) MRCP session has not opened after %d ms\n", schannel->name, SPEECH_CHANNEL_TIMEOUT_USEC / (1000));
                    break; // <----------------------------------------------------
    }
}
seven1240 commented 1 year ago

I think it is by design, do you hit a problem when it runs forever? what if you comment out warned = 1?