[asterisk-bugs] [Asterisk 0015807]: rtt should be stored as double in struct ast_rtp_instance_stats

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15807 
====================================================================== 
Reported By:                klaus3000
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15807
Category:                   Core/RTP
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
JIRA:                       SWP-1143 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 214938 
Request Review:              
====================================================================== 
Date Submitted:             2009-09-01 09:36 CDT
Last Modified:              2010-06-07 14:52 CDT
====================================================================== 
Summary:                    rtt should be stored as double in struct
ast_rtp_instance_stats
Description: 
Hi!

AFAIS member "rtt" in struct ast_rtp_instance_stats stored the RTT in
seconds, using an usigned int. 

As RTTs usually are below 0.5 (at least on earth and for VoIP useable
links) this member is useless as it is always 0.

Shouldn't that be changed to double?
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0015819 [patch] buggy output in "sip show ...
====================================================================== 

---------------------------------------------------------------------- 
 (0123079) svnbot (reporter) - 2010-06-07 14:52
 https://issues.asterisk.org/view.php?id=15807#c123079 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 268773

U   trunk/channels/chan_sip.c
U   trunk/channels/sip/dialplan_functions.c
U   trunk/include/asterisk/rtp_engine.h
U   trunk/main/rtp_engine.c

------------------------------------------------------------------------
r268773 | tilghman | 2010-06-07 14:52:38 -0500 (Mon, 07 Jun 2010) | 5
lines

Seems strange (and the code backs up) that if the max and min of a
statistic is expressed as a double, the last value would not also need to
be a double.

(closes issue https://issues.asterisk.org/view.php?id=15807)
 Reported by: klaus3000

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=268773 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-06-07 14:52 svnbot         Checkin                                      
2010-06-07 14:52 svnbot         Note Added: 0123079                          
======================================================================