plasma-desktop-5.10.5/kcms/kfontinst/kcmfontinst/FontList.cpp:1415: suspicious if ?

David Edmundson david at davidedmundson.co.uk
Tue Aug 29 09:00:59 UTC 2017


On Wed, Aug 23, 2017 at 10:25 AM, David Binderman <dcb314 at hotmail.com>
wrote:

> Hello there,
>
> plasma-desktop-5.10.5/kcms/kfontinst/kcmfontinst/FontList.cpp:1415]:
> (style) Same expression on both sides of '||'.
>
> It only controls whether a separator gets added in a menu. Nothing
terribly exciting.

Code beneath makes it clear that we're trying to check that first
expression and nothing else.
Will fix.

David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170829/a1e3466c/attachment.html>


More information about the Plasma-devel mailing list