linphone-developers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Linphone-developers] [Bug] Call pausing


From: Johan Pascal
Subject: Re: [Linphone-developers] [Bug] Call pausing
Date: Tue, 25 Aug 2015 22:16:26 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.8.0

Dear Fabrizio,
there is no bug there:
- when you press call on your side, the UI will change button "Pause" to "Resume"
AND change the |>(play) shaped logo in the call view to a ||(paused) shaped logo.

- when the call is paused by remote, you will just get the second change as you're not given the possibility to resume the call.

Try to implement your suggestion and you will see that the logo displaying the current call status won't be modified anymore when you pause the call.

regards

Johan


On 25/08/15 20:15, Fabrizio Carrai wrote:
Here a bug I found (another will follow), please feel free to comment.

 
Version:
Linphone 3.8.5 (Linux)

Description:
The bug has been found during a code review. No test have been done yet, although the error seems to be evident.

Module:

gtk/main.c


Code:

static void linphone_gtk_call_state_changed(LinphoneCore *lc, LinphoneCall *call, LinphoneCallState cs, const char *msg){
switch(cs){

[...]
case LinphoneCallPausing:
linphone_gtk_enable_hold_button(call,TRUE,FALSE);
linphone_gtk_call_update_tab_header(call,FALSE);
<<<<----- 'break' is missing
case LinphoneCallPausedByRemote:
linphone_gtk_in_call_view_set_paused(call);
linphone_gtk_call_update_tab_header(call,TRUE);
break;

[...]
linphone_gtk_update_call_buttons (call);
}



--
Fabrizio


_______________________________________________
Linphone-developers mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/linphone-developers


reply via email to

[Prev in Thread] Current Thread [Next in Thread]