Hi Denis,

> >+    g_at_chat_add_terminator(chat, "+EXT ERROR:", 11, FALSE);
> >+    g_at_chat_add_terminator(chat, "+CME ERROR:", 11, FALSE);
> >+    g_at_chat_add_terminator(chat, "+CMS ERROR:", 11, FALSE);
> >+    g_at_chat_add_terminator(chat, "NO ANSWER", -1, FALSE);
> >+    g_at_chat_add_terminator(chat, "CONNECT", -1, TRUE);
> >+    g_at_chat_add_terminator(chat, "NO CARRIER", -1, FALSE);
> >+    g_at_chat_add_terminator(chat, "BUSY", -1, FALSE);
> >+    g_at_chat_add_terminator(chat, "NO DIALTONE", -1, FALSE);
> >+    g_at_chat_add_terminator(chat, "ERROR", -1, FALSE);
> >+    g_at_chat_add_terminator(chat, "OK", -1, TRUE);
> 
> I really don't like this.  Lets keep the non-standard terminators in a 
> separate list.  I don't want the vast majority of the drivers incurring the 
> cost of multiple g_new/g_frees.

I have to agree on this. We should keep the penalty for well behaving
cards as small as possible.

Regards

Marcel


_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono

Reply via email to