<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think in off-nominal error cases the stats should be saved.<br>I think it is only when a media session is terminated after being<br>used that the stats in regards to it should be saved. As I<br>previously said the way it is now can result in stats for RTP<br>sessions that were never used, as well as stats for the same RTP<br>session at different points in time.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As a consumer of the functionality I think trying to use the data<br>would be problematic in its current state. You would need to go<br>through the data, correlate any duplicates, determine if any should<br>be ignored, and then it would be useful. Without doing so it can<br>lead you to wrong interpretations. For example if stats for an RTP<br>instance that was never used were present and I saw the statistics<br>in the vector I might wonder if there was a problem - but in<br>reality there wasn't, because the stream was never actually used.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua,</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thank you for your kind reviewing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, I've removed some off-nominal error cases. But to be honestly, I haven't look the pjsip code in deep level, so, I doubting about what I've done it. And I'm worry about may I couldn't catch up what you mentioned. So please give me a feedback if I was wrong(I think I made mistake again). I would fix it and learn something from that. :)</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>Thank you.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Kind regards,<br>Sungtae</p><p><a href="https://gerrit.asterisk.org/10899">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10899/5/include/asterisk/res_pjsip_session.h">File include/asterisk/res_pjsip_session.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10899/5/include/asterisk/res_pjsip_session.h@144">Patch Set #5, Line 144:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">};<br><br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The reason I mentioned passing this in, instead of storing it is because it alters the API and rippl […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Now I've got it. I was focused on the function more than structure. But this case, yes, you're right. The isolated structure is much more important. I've fixed it. :)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10899">change 10899</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10899"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib25c2d3fc4da084aecfde2a82c1b1d733bd64fa5 </div>
<div style="display:none"> Gerrit-Change-Number: 10899 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 11 Feb 2019 21:12:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>