All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] doc: Add message state API
@ 2011-01-06 19:50 Faiyaz Baxamusa
  2011-01-13 18:04 ` Denis Kenzior
  0 siblings, 1 reply; 3+ messages in thread
From: Faiyaz Baxamusa @ 2011-01-06 19:50 UTC (permalink / raw)
  To: ofono

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

---
 Makefile.am               |    2 +-
 doc/cdma-message-api.txt  |    5 +++++
 doc/message-api.txt       |   24 +-----------------------
 doc/message-state-api.txt |   27 +++++++++++++++++++++++++++
 4 files changed, 34 insertions(+), 24 deletions(-)
 create mode 100644 doc/message-state-api.txt

diff --git a/Makefile.am b/Makefile.am
index 09dc9ad..14f53ef 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -369,7 +369,7 @@ doc_files = doc/overview.txt doc/ofono-paper.txt doc/release-faq.txt \
 			doc/phonebook-api.txt doc/radio-settings-api.txt \
 			doc/sim-api.txt doc/stk-api.txt \
 			doc/audio-settings-api.txt doc/text-telephony-api.txt \
-			doc/calypso-modem.txt
+			doc/calypso-modem.txt doc/message-state-api.txt
 
 
 test_scripts = test/backtrace \
diff --git a/doc/cdma-message-api.txt b/doc/cdma-message-api.txt
index 8e6b9ea..c3d03f8 100644
--- a/doc/cdma-message-api.txt
+++ b/doc/cdma-message-api.txt
@@ -110,3 +110,8 @@ Properties	boolean UseDeliveryAcknowledgement
 			The call back number for the user.  If the number is
 			empty, then the optional field is not included
 			in the encoded PDU.
+
+Message hierarchy
+===============
+
+Please refer to message-state-api.txt
diff --git a/doc/message-api.txt b/doc/message-api.txt
index f7ab22a..e6c8522 100644
--- a/doc/message-api.txt
+++ b/doc/message-api.txt
@@ -90,26 +90,4 @@ Properties	string ServiceCenterAddress
 Message hierarchy
 ===============
 
-Service		org.ofono
-Interface	org.ofono.Message
-Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
-
-Methods		dict GetProperties()
-
-			Returns properties for the message object. See
-			the properties section for available properties.
-
-			Possible Errors: [service].Error.InvalidArguments
-
-Signals		PropertyChanged(string name, variant value)
-
-			This signal indicates a changed value of the given
-			property.
-
-Properties	string State
-
-			Contains the state of the message object.  Possible
-			values are:
-				"pending",
-				"sent",
-				"failed"
+Please refer to message-state-api.txt
diff --git a/doc/message-state-api.txt b/doc/message-state-api.txt
new file mode 100644
index 0000000..edd0cbf
--- /dev/null
+++ b/doc/message-state-api.txt
@@ -0,0 +1,27 @@
+Message hierarchy
+===============
+
+Service		org.ofono
+Interface	org.ofono.Message
+Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
+
+Methods		dict GetProperties()
+
+			Returns properties for the message object. See
+			the properties section for available properties.
+
+			Possible Errors: [service].Error.InvalidArguments
+
+Signals		PropertyChanged(string name, variant value)
+
+			This signal indicates a changed value of the given
+			property.
+
+Properties	string State
+
+			Contains the state of the message object.  Possible
+			values are:
+				"pending",
+				"sent",
+				"failed"
+				"invalid"
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/3] doc: Add message state API
  2011-01-06 19:50 [PATCH 1/3] doc: Add message state API Faiyaz Baxamusa
@ 2011-01-13 18:04 ` Denis Kenzior
  2011-01-13 21:56   ` Faiyaz Baxamusa
  0 siblings, 1 reply; 3+ messages in thread
From: Denis Kenzior @ 2011-01-13 18:04 UTC (permalink / raw)
  To: ofono

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

Hi Faiyaz,

On 01/06/2011 01:50 PM, Faiyaz Baxamusa wrote:
> ---
>  Makefile.am               |    2 +-
>  doc/cdma-message-api.txt  |    5 +++++
>  doc/message-api.txt       |   24 +-----------------------

Please rename this file into doc/messagemanager-api.txt

>  doc/message-state-api.txt |   27 +++++++++++++++++++++++++++

And this one can be message-api.txt

>  4 files changed, 34 insertions(+), 24 deletions(-)
>  create mode 100644 doc/message-state-api.txt
> 
> diff --git a/Makefile.am b/Makefile.am
> index 09dc9ad..14f53ef 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -369,7 +369,7 @@ doc_files = doc/overview.txt doc/ofono-paper.txt doc/release-faq.txt \
>  			doc/phonebook-api.txt doc/radio-settings-api.txt \
>  			doc/sim-api.txt doc/stk-api.txt \
>  			doc/audio-settings-api.txt doc/text-telephony-api.txt \
> -			doc/calypso-modem.txt
> +			doc/calypso-modem.txt doc/message-state-api.txt
>  
>  
>  test_scripts = test/backtrace \
> diff --git a/doc/cdma-message-api.txt b/doc/cdma-message-api.txt
> index 8e6b9ea..c3d03f8 100644
> --- a/doc/cdma-message-api.txt
> +++ b/doc/cdma-message-api.txt
> @@ -110,3 +110,8 @@ Properties	boolean UseDeliveryAcknowledgement
>  			The call back number for the user.  If the number is
>  			empty, then the optional field is not included
>  			in the encoded PDU.
> +
> +Message hierarchy
> +===============
> +
> +Please refer to message-state-api.txt

Please get rid of this chunk.

> diff --git a/doc/message-api.txt b/doc/message-api.txt
> index f7ab22a..e6c8522 100644
> --- a/doc/message-api.txt
> +++ b/doc/message-api.txt
> @@ -90,26 +90,4 @@ Properties	string ServiceCenterAddress
>  Message hierarchy
>  ===============
>  
> -Service		org.ofono
> -Interface	org.ofono.Message
> -Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
> -
> -Methods		dict GetProperties()
> -
> -			Returns properties for the message object. See
> -			the properties section for available properties.
> -
> -			Possible Errors: [service].Error.InvalidArguments
> -
> -Signals		PropertyChanged(string name, variant value)
> -
> -			This signal indicates a changed value of the given
> -			property.
> -
> -Properties	string State
> -
> -			Contains the state of the message object.  Possible
> -			values are:
> -				"pending",
> -				"sent",
> -				"failed"
> +Please refer to message-state-api.txt
> diff --git a/doc/message-state-api.txt b/doc/message-state-api.txt
> new file mode 100644
> index 0000000..edd0cbf
> --- /dev/null
> +++ b/doc/message-state-api.txt
> @@ -0,0 +1,27 @@
> +Message hierarchy
> +===============
> +
> +Service		org.ofono
> +Interface	org.ofono.Message
> +Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
> +
> +Methods		dict GetProperties()
> +
> +			Returns properties for the message object. See
> +			the properties section for available properties.
> +
> +			Possible Errors: [service].Error.InvalidArguments
> +
> +Signals		PropertyChanged(string name, variant value)
> +
> +			This signal indicates a changed value of the given
> +			property.
> +
> +Properties	string State
> +
> +			Contains the state of the message object.  Possible
> +			values are:
> +				"pending",
> +				"sent",
> +				"failed"
> +				"invalid"

Please get rid of this invalid state.

Regards,
-Denis


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/3] doc: Add message state API
  2011-01-13 18:04 ` Denis Kenzior
@ 2011-01-13 21:56   ` Faiyaz Baxamusa
  0 siblings, 0 replies; 3+ messages in thread
From: Faiyaz Baxamusa @ 2011-01-13 21:56 UTC (permalink / raw)
  To: ofono

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

Hi Denis,

On 01/13/2011 10:04 AM, ext Denis Kenzior wrote:
> Hi Faiyaz,
>
> On 01/06/2011 01:50 PM, Faiyaz Baxamusa wrote:
>> ---
>>   Makefile.am               |    2 +-
>>   doc/cdma-message-api.txt  |    5 +++++
>>   doc/message-api.txt       |   24 +-----------------------
>
> Please rename this file into doc/messagemanager-api.txt

I will rename current doc/message-api.txt to
doc/messagemanager-api.txt file and also rename doc/cdma-message-api.txt 
to doc/cdma-messagemanager-api.txt

>
>>   doc/message-state-api.txt |   27 +++++++++++++++++++++++++++
>
> And this one can be message-api.txt

I will rename the new file to doc/message-api.txt

>
>>   4 files changed, 34 insertions(+), 24 deletions(-)
>>   create mode 100644 doc/message-state-api.txt
>>
>> diff --git a/Makefile.am b/Makefile.am
>> index 09dc9ad..14f53ef 100644
>> --- a/Makefile.am
>> +++ b/Makefile.am
>> @@ -369,7 +369,7 @@ doc_files = doc/overview.txt doc/ofono-paper.txt doc/release-faq.txt \
>>   			doc/phonebook-api.txt doc/radio-settings-api.txt \
>>   			doc/sim-api.txt doc/stk-api.txt \
>>   			doc/audio-settings-api.txt doc/text-telephony-api.txt \
>> -			doc/calypso-modem.txt
>> +			doc/calypso-modem.txt doc/message-state-api.txt
>>
>>
>>   test_scripts = test/backtrace \
>> diff --git a/doc/cdma-message-api.txt b/doc/cdma-message-api.txt
>> index 8e6b9ea..c3d03f8 100644
>> --- a/doc/cdma-message-api.txt
>> +++ b/doc/cdma-message-api.txt
>> @@ -110,3 +110,8 @@ Properties	boolean UseDeliveryAcknowledgement
>>   			The call back number for the user.  If the number is
>>   			empty, then the optional field is not included
>>   			in the encoded PDU.
>> +
>> +Message hierarchy
>> +===============
>> +
>> +Please refer to message-state-api.txt
>
> Please get rid of this chunk.
>

Sure.

>> diff --git a/doc/message-api.txt b/doc/message-api.txt
>> index f7ab22a..e6c8522 100644
>> --- a/doc/message-api.txt
>> +++ b/doc/message-api.txt
>> @@ -90,26 +90,4 @@ Properties	string ServiceCenterAddress
>>   Message hierarchy
>>   ===============
>>
>> -Service		org.ofono
>> -Interface	org.ofono.Message
>> -Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
>> -
>> -Methods		dict GetProperties()
>> -
>> -			Returns properties for the message object. See
>> -			the properties section for available properties.
>> -
>> -			Possible Errors: [service].Error.InvalidArguments
>> -
>> -Signals		PropertyChanged(string name, variant value)
>> -
>> -			This signal indicates a changed value of the given
>> -			property.
>> -
>> -Properties	string State
>> -
>> -			Contains the state of the message object.  Possible
>> -			values are:
>> -				"pending",
>> -				"sent",
>> -				"failed"
>> +Please refer to message-state-api.txt
>> diff --git a/doc/message-state-api.txt b/doc/message-state-api.txt
>> new file mode 100644
>> index 0000000..edd0cbf
>> --- /dev/null
>> +++ b/doc/message-state-api.txt
>> @@ -0,0 +1,27 @@
>> +Message hierarchy
>> +===============
>> +
>> +Service		org.ofono
>> +Interface	org.ofono.Message
>> +Object path	[variable prefix]/{modem0,modem1,...}/{message_01,...}
>> +
>> +Methods		dict GetProperties()
>> +
>> +			Returns properties for the message object. See
>> +			the properties section for available properties.
>> +
>> +			Possible Errors: [service].Error.InvalidArguments
>> +
>> +Signals		PropertyChanged(string name, variant value)
>> +
>> +			This signal indicates a changed value of the given
>> +			property.
>> +
>> +Properties	string State
>> +
>> +			Contains the state of the message object.  Possible
>> +			values are:
>> +				"pending",
>> +				"sent",
>> +				"failed"
>> +				"invalid"
>
> Please get rid of this invalid state.
>

Sure, will remove "invalid"

> Regards,
> -Denis
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-01-13 21:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-06 19:50 [PATCH 1/3] doc: Add message state API Faiyaz Baxamusa
2011-01-13 18:04 ` Denis Kenzior
2011-01-13 21:56   ` Faiyaz Baxamusa

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.