<table><tr><td style="">aacid added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16978">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16978#362018" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D16978#362018</a>, <a href="https://phabricator.kde.org/p/yurchor/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@yurchor</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16978#361763" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D16978#361763</a>, <a href="https://phabricator.kde.org/p/aacid/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@aacid</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16978#361614" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D16978#361614</a>, <a href="https://phabricator.kde.org/p/yurchor/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@yurchor</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16978#361561" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D16978#361561</a>, <a href="https://phabricator.kde.org/p/aacid/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@aacid</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>the wall sound is not played, any idea when that was lost form the code?</p>
<p>Also i thnk we should eventually move to kgsound, but that's for another time.</p>
<p>I guess you can commit, but if you could investigate why the wall sound was removed it'd be nice, since it feels weird not to have it playing (the wall.wav is there)</p></div>
</blockquote>
<p>It seems that it was removed in <a href="https://phabricator.kde.org/R407:5fb04e3feca8c15b4deb6d86ce37fd4e5e898e99" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">5fb04e3feca8c15b4deb6d86ce37fd4e5e898e99</a> and never have been restored. On the overloaded machine (e.g. when Baloo has decided to take the rule) even now sounds can lead to the tearing in the ball movement. Anyway, it is better explicitly switch off the sound than trying to decide why it is not working (and even not compiled), imho.</p></div>
</blockquote>
<p>Do you think you can give it a try porting to kgsound? technically it's supposed to be much less resource intensive and shouldn't be "too hard" (TM)</p></div>
</blockquote>
<p>I do not know (it might need a very heavy fuel as we see on the KolourPaint example). No hard promises in our business. ;)</p>
<p>First things first. Kolf should be ported away from kdelibs4support to make it safe for the next several years and Kolf should keep its full set of features. Optimizations can wait, imho</p></div>
</blockquote>
<p>kdelibs4support is as sake as KF5, they will both disappear at the same time, so porting away from kdelibs4support doesn't really give you a "lot" of future proofing.</p>
<p>But up to you :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R407 Kolf</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16978">https://phabricator.kde.org/D16978</a></div></div><br /><div><strong>To: </strong>yurchor, KDE Games, ltoscano, arojas, aacid<br /><strong>Cc: </strong>aacid, kde-games-devel<br /></div>