Re: [PATCH 3/3] doc: Add new D-Bus methods to service interface

[Ronald Tessier <ronald.tessier@linux.intel.com>]

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

On 05/22/2012 03:22 PM, Lucas De Marchi wrote:
> On Mon, May 21, 2012 at 12:59 PM, Ronald Tessier
> <ronald.tessier@linux.intel.com>  wrote:
>> ---
>>   doc/service-api.txt |   20 ++++++++++++++++++++
>>   1 files changed, 20 insertions(+), 0 deletions(-)
>>
>> diff --git a/doc/service-api.txt b/doc/service-api.txt
>> index 36e1161..02396ff 100644
>> --- a/doc/service-api.txt
>> +++ b/doc/service-api.txt
>> @@ -63,6 +63,26 @@ Methods              array{object,dict} GetMessages()
>>                                 [service].Error.PermanentContentNotAccepted
>>                                 [service].Error.PermanentLackOfPrepaid
>>
>> +               void DeleteMessages(array{object})
>> +
>> +                       Delete an array of message objects that represents
>> +                       the currently stored messages to delete.
>> +
>> +                       Possible Errors: [service].Error.InvalidArguments
>> +
>> +               void DeleteConversation(string number)
>
>
> Could we mark these as [TODO], so we don't  get confused while reading
> the doc + code and realizing they don't match?
>

Sure, I re-sent a set of patches with the [TODO].
Thanks for the remark,

Ronald