Re: [PATCH] callmanager: change state of waiting call

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1084 bytes --]

Hi Nicolas,

On 04/15/2011 07:29 AM, Nicolas Bertrand wrote:
> becomes incoming when all active and held are released
> 
> ---
>  src/callmanager.cpp |   25 +++++++++++++++++++++----
>  src/callmanager.h   |    3 +++
>  2 files changed, 24 insertions(+), 4 deletions(-)
> 

Patch has been applied, thanks.

> @@ -709,6 +716,16 @@ void CallManager::dialingToAlerting()
>      sendState( callList[index] );
>  }
>  
> +void CallManager::waitingToIncoming()
> +{
> +	int index = indexForId( idForState( CallState_Waiting ) );
> +	if ( index < 0 )
> +		return;

Quick nitpick: could you please make sure to use spaces for indentation
in phonesim.  Phonesim codebase was developed according to a different
coding style and we should just conform to it, at least as much as
possible.  I fixed this for you and amended the commit.

> +
> +    callList[index].state = CallState_Incoming;
> +    sendState( callList[index] );
> +}
> +
>  void CallManager::dialBack()
>  {
>      startIncomingCall( "1234567", "7654321", "Alice", true );

Regards,
-Denis