Bug 22321 - Enshrine keepalive-interval in the spec
Summary: Enshrine keepalive-interval in the spec
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/wj...
Whiteboard:
Keywords: patch
Depends on: 22322
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-16 11:38 UTC by Will Thompson
Modified: 2009-06-23 03:34 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Will Thompson 2009-06-16 11:38:12 UTC
telepathy-sofiasip has had a 'keepalive-interval' parameter for a while, and telepathy-gabble will gain one in the next release.

The former requires a int32, with -1 and 0 representing two different defaults. Gabble will require an uint32, with 0 meaning no keepalive, and a default value of 30 (corresponding to the previously hard-coded interval).

Mikhail agreed that Gabble's definition is probably more sensible; telepathy-sofiasip should be brought in line, and then we should add this to the well-known parameters in the spec.
Comment 1 Simon McVittie 2009-06-22 08:05:57 UTC
Looks good.
Comment 2 Will Thompson 2009-06-23 03:34:02 UTC
Reviewed by smcv and merged, will be in spec version 0.17.27.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.