<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p style="margin-top:0;margin-bottom:0">Hello there,<br>
</p>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><br>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;"></span></font>>however, please consider to use bugzilla at bugs.kde.orgproduct: plasmashell<br>
<font size="2"><span style="font-size:11pt;">
<div class="PlainText"><br>
Done.<br>
<br>
<a href="https://bugs.kde.org/show_bug.cgi?id=390087" class="OWAAutoLink" id="LPlnk809831" previewremoved="true">https://bugs.kde.org/show_bug.cgi?id=390087</a><br>
<br>
Regards<br>
<br>
David Binderman<br>
<br>
On giovedì 8 febbraio 2018 10:07:23 CET David Binderman wrote:<br>
> Hello there,<br>
> <br>
> <br>
> <br>
> plasma-desktop-5.12.0/kcms/kfontinst/lib/Misc.cpp:294]: (style) Same<br>
> expression on both sides of '||'.<br>
> <br>
> Source code is<br>
> <br>
> <br>
>     return "font/otf"==mime || "font/ttf"==mime ||<br>
>            "application/x-font-type1"==mime ||<br>
> "application/x-font-ttf"==mime || "application/x-font-otf"==mime ||<br>
> "application/x-font-type1"==mime;<br>
> <br>
> x-font-type1 gets mentioned twice. Suggest code rework.<br>
> <br>
> <br>
> Regards<br>
> <br>
> <br>
> David Binderman<br>
<br>
<br>
-- <br>
Marco Martin<br>
</div>
</span></font></div>
</div>
</div>
</body>
</html>