<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="http://git.reviewboard.kde.org/r/107640/">http://git.reviewboard.kde.org/r/107640/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 10th, 2013, 5:08 p.m. UTC, <b>Kurt Hindenburg</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 get crashing when there are multiple windows and try to edit profile dialog - can you see if you can reproduce?
0 libkonsoleprivate.dylib 0x000000010214bf26 Konsole::ColorScheme::foregroundColor() const + 22 (qcolor.h:293)
1 libkonsoleprivate.dylib 0x000000010215e7cf Konsole::ColorSchemeViewDelegate::paint(QPainter*, QStyleOptionViewItem const&, QModelIndex const&) const + 367 (EditProfileDialog.cpp:1273)</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;">Hmm, yes I can reproduce the crash but the new patch should fix the issue.</pre>
<br />
<p>- Francesco</p>
<br />
<p>On January 13th, 2013, 12:03 p.m. UTC, Francesco Cecconi wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Konsole.</div>
<div>By Francesco Cecconi.</div>
<p style="color: grey;"><i>Updated Jan. 13, 2013, 12:03 p.m.</i></p>
<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;">Looking at the code, the best solution for the issue described in the bug is:
Fixing the wrong behaviour to open two or more "edit dialog" for one session/tab with
the same profile (it should be fixed anyway and this is the issue of the bug 311270 ),
and the crash with two or more "edit dialog", one for different session(tabs)
with the same profile.
With the patch:
one tab/session -> only one edit profile dialog at the same time ( 311270 )
tab1/session -> edit profile (profile1)
tab2/session -> edit profile (profile1)
...
tab[i]/session -> edit profile (profile1)
many tab with the same profile, only one edit dialog is shown
NOTE:The issue(crash) appears only when transparency property is modified by one of the two profiles.</pre>
</td>
</tr>
</table>
<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=311270">311270</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/EditProfileDialog.h <span style="color: grey">(c85bd11)</span></li>
<li>src/SessionController.h <span style="color: grey">(aefdbc6)</span></li>
<li>src/SessionController.cpp <span style="color: grey">(fd58ecc)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107640/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>