<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/116623/">https://git.reviewboard.kde.org/r/116623/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 15th, 2014, 2:04 p.m. NOVT, <b>Vadim A. Misbakh-Soloviov</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Wasn't it a typo in
>> if (handledCallChannels.contains(callChannel)) {
I bet, it should be "if (!handledCallChannels.contains" (i.e. if NOT contains). Isn't it.
// It brakes call-ui with currently commited behaviour (it doesn't create windows/doesn't call at all).</pre>
</blockquote>
<p>On May 15th, 2014, 4:22 p.m. NOVT, <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Urgh, it looks like it :S Well that's awkward.
Mayank - fancy doing another patch fixing this? ;)</pre>
</blockquote>
<p>On May 15th, 2014, 4:25 p.m. NOVT, <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Crap. Looks like we're making a 0.8.2.</pre>
</blockquote>
<p>On May 15th, 2014, 4:26 p.m. NOVT, <b>Vadim A. Misbakh-Soloviov</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I can make patch in a few hours if any :)
// I need to increase my conributions count to get a right to ask for developer access ;)</pre>
</blockquote>
<p>On May 15th, 2014, 4:34 p.m. NOVT, <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Oh sure, feel free to :)
(also note that you don't get dev account just by commit count but you must have a dev that will sign your application...if you stay around and do couple good patches, we're happy to put our names there ;)</pre>
</blockquote>
<p>On May 15th, 2014, 9:53 p.m. NOVT, <b>Vadim A. Misbakh-Soloviov</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">https://reviewboard.kde.org/r/118151/
If any.
// just set up RBTools with 6th sense and created that review req ;)</pre>
</blockquote>
<p>On May 16th, 2014, 1:16 a.m. NOVT, <b>mayank jha</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks for pointing it out @Vadim.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Actually, it still need some clearance:
Now (with fixed if-condition) Dialout UI make calls just fineb but, I, for example, still being able to call same number couple times (althought, in different channels), so, in some inperpretation, windows is duplicating (two windows with two calls to same number). I just tried to call my cellphone, and, as I noticed, it was made in two separate threads: one keeps ringing, while other says that number is already busy.
Moreover, despite I don't experience original bug ever, I didn't even now: when I close dup-window (that was created by second press on "audio/videocall" buttons), call in the first window keeps going fine.</pre>
<br />
<p>- Vadim A.</p>
<br />
<p>On March 18th, 2014, 10:23 p.m. NOVT, mayank jha wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By mayank jha.</div>
<p style="color: grey;"><i>Updated March 18, 2014, 10:23 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=309146">309146</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-call-ui
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">We store the channels handled by a callManager in a list, and upon a new request we check if it already exists, only if it is new we open a new call-ui window.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Made a video call with another person. Clicked the audio and video call buttons. The window was not duplicated.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/call-handler.h <span style="color: grey">(5fea003)</span></li>
<li>src/call-handler.cpp <span style="color: grey">(f4ced53)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116623/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>