<table><tr><td style="">ivan 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/D10691" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/hein/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;" rel="noreferrer">@hein</a></p>
<p>This is not really as simple as 'accept without a comment'.</p>
<p>I don't mind this patch to be merged, but keep in mind that it just turns users for which plasma crashes (for which we get an explicit message as to why) into users who do not have certain parts of Plasma working where the error message will get lost in the Plasma's log. Now, as a maintainer of said parts of Plasma, that is your call.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R159 KActivities Statistics</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10691" rel="noreferrer">https://phabricator.kde.org/D10691</a></div></div><br /><div><strong>To: </strong>kpiwowarski, Plasma, Frameworks, hein<br /><strong>Cc: </strong>ivan, plasma-devel, Frameworks, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>