[liboxygen] fix coverity issue
Review Request #126048 - Created Nov. 12, 2015 and submitted
this patch fixes coverity issue #1335195:
setEnabled() call is removed completely because setMaxCost does essentialy the same.
the diff also contains two minor changes for overall code beauty ;)
Please revert to the original formating.
It is used consistently throughout the entire code.
ok. Thanks for removing the comment.
It must be a relic from old code.
I disagree with this change.
First, at the minimum you should add parenthesis around the two && blocks (if only for readability), and second, I do not see the improvement in term of either readability or "beauty" with respect to the previous code.
Also, this is unrelated to the problem at hand.
Please revert. Lets not fix things that are working.
Thanks for the RR!
Is there any way to have access to the coverity review without a login ? or can I use my "kde" login ? (seems no). Or can you paste the actual report here ?