Hi Pekka,

On 10/08/2010 10:36 AM, [email protected] wrote:
> From: Pekka Pessi <[email protected]>
> 
> ---
>  doc/call-barring-api.txt     |   10 -------
>  doc/call-forwarding-api.txt  |    9 +++++++
>  doc/voicecall-api.txt        |   55 
> +++++++++++++++++++++++++++++++++++++++++-
>  doc/voicecallmanager-api.txt |    5 ++++
>  4 files changed, 68 insertions(+), 11 deletions(-)
> 
> diff --git a/doc/call-barring-api.txt b/doc/call-barring-api.txt
> index 41ae4b1..1534494 100644
> --- a/doc/call-barring-api.txt
> +++ b/doc/call-barring-api.txt
> @@ -37,16 +37,6 @@ Signals            PropertyChanged(string property, 
> variant value)
>                       Signal is emitted whenever a property has changed.
>                       The new value is passed as the signal argument.
>  
> -             IncomingBarringInEffect()
> -
> -                     Signal is emitted when a call is made and an
> -                     incoming call barring supplementary service is in use.
> -
> -             OutgoingBarringInEffect()
> -
> -                     Signal is emitted when a call is made and an
> -                     outgoing call barring supplementary service is in use.
> -
>  Properties   string VoiceIncoming [readwrite]
>  
>                       Contains the value of the barrings for the incoming
> diff --git a/doc/call-forwarding-api.txt b/doc/call-forwarding-api.txt
> index 067531a..a97165b 100644
> --- a/doc/call-forwarding-api.txt
> +++ b/doc/call-forwarding-api.txt
> @@ -26,6 +26,15 @@ Signals            PropertyChanged(string property, 
> variant value)
>                       Signal is emitted whenever a property has changed.
>                       The new value is passed as the signal argument.
>  
> +             ForwardingInEffect(string condition)
> +
> +                     This informational signal is emitted when an outgoing
> +                     call is made. It indicates that a voice call
> +                     forwarding for incoming calls is in use. Condition can
> +                     be either of:
> +                             "unconditional" - VoiceUnconditional is active
> +                             "conditional" - A conditional rule is active
> +

So this maps to CSSI <code1> 0 or 1.  Sounds like this belongs on the
voicecall interface (or perhaps voicecallmanager) as well.

>  Properties   string VoiceUnconditional [readwrite]
>  
>                       Contains the value of the voice unconditional call
> diff --git a/doc/voicecall-api.txt b/doc/voicecall-api.txt
> index f0ba316..a6c0b42 100644
> --- a/doc/voicecall-api.txt
> +++ b/doc/voicecall-api.txt
> @@ -73,6 +73,54 @@ Signals            PropertyChanged(string property, 
> variant value)
>                       of this signal as optional. This signal will be
>                       emitted before the PropertyChanged signal.
>  
> +             JoinedMultiparty()
> +
> +                     Signal is emitted when remote party joins the
> +                     call into a multiparty conference.
> +
> +                     Note that there is no corresponding event when
> +                     remote party splits call from the multiparty
> +                     conference.
> +
> +             Waiting()
> +
> +                     Signal is emitted when an outgoing call is waiting.

I'm fine with this one, but have you considered putting this on a the
voicecallmanager interface?  This can only be emitted for dialing /
alerting calls, so the scope is inherently known.  But I'm fine either way.

> +
> +             OutgoingBarred()
> +
> +                     Signal is emitted when caller's outgoing barring
> +                     services prevent call from being connected.
> +
> +                     This signal can be emitted while call is being
> +                     disconnected or while a recorded announcement is
> +                     played.
> +

I'm not so sure about this assessment on AT modems...

> +             IngomingBarred()
> +
> +                     Signal is emitted when callee's incoming barring
> +                     services prevent call from being connected.
> +
> +                     This signal can be emitted while call is being
> +                     disconnected or while a recorded announcement is
> +                     played.

Please name this RemoteBarred()

> +
> +             IncomingRedirected(string)
> +
> +                     Signal is emitted when network indicates that an
> +                     incoming call has been redirected.  The value
> +                     contains the phone number of the party that
> +                     redirected the call if available.

Why is this one on the voice call interface while IncomingForwarded is
on the voicecall manager interface?  I assume this refers to an incoming
or waiting call being signaled as part of a deflection procedure?

> +
> +             OutgoingRedirected()
> +
> +                     Signal is emitted when an outgoing call has been
> +                     redirected.

Please stick with our terminology and name this Deflected().

> +
> +             CallerIDRestrictionRejected()
> +
> +                     Signal is emitted when caller requested Caller
> +                     ID Restriction but network rejects it.

Again for consistency, please use CallingLineRestrictionRejected()

> +
>  Properties   string LineIdentification [readonly]
>  
>                       Contains the Line Identification information returned
> @@ -90,7 +138,7 @@ Properties string LineIdentification [readonly]
>               boolean Multiparty [readonly]
>  
>                       Contains the indication if the voice call is part
> -                     of a multiparty call or not.
> +                     of a multiparty call set up by user or not.
>  
>                       Notifications if a call becomes part or leaves a
>                       multipart call are sent.
> @@ -115,6 +163,11 @@ Properties       string LineIdentification [readonly]
>                       Client applications can use this to infer somewhat
>                       reliable call duration information.
>  
> +             boolean OnHold [readonly]
> +
> +                     Contains the indication if the remote party has
> +                     put call on hold.
> +

I'm not sure you can ever make this work on AT modems.  Perhaps making
this a signal on the VoiceCallManager interface is better.

>               string Information [readonly, optional]
>  
>                       Contains information related to the call for the
> diff --git a/doc/voicecallmanager-api.txt b/doc/voicecallmanager-api.txt
> index 11af23a..34ed9de 100644
> --- a/doc/voicecallmanager-api.txt
> +++ b/doc/voicecallmanager-api.txt
> @@ -132,6 +132,11 @@ Signals          CallAdded(object path, dict properties)
>                       released.  The object path is no longer accessible
>                       after this signal and only emitted for reference.
>  
> +             IncomingCallForwarded
> +
> +                     This signal is sent when an incoming voice call
> +                     has been forwarded by the network.
> +

What is this one referring to?  Is it a result of a transfer procedure?

>  Properties   array{string} EmergencyNumbers
>  
>                       Contains the list of emergency numbers recognized

Regards,
-Denis
_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono

Reply via email to