Bug 362709

Summary: kio_sftp.cpp:1239]: (style) Suspicious condition
Product: [Frameworks and Libraries] kio-extras Reporter: dcb314
Component: defaultAssignee: Plasma Development Mailing List <plasma-devel>
Status: RESOLVED NOT A BUG    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: RedHat Enterprise Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description dcb314 2016-05-05 15:30:54 UTC
../../../kioslave/sftp/kio_sftp.cpp:1239]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.

Source code is

    } else if ((errorCode = writeToFile(fd, filedata.constData(), filedata.size()) != 0)) {

Suggest new code

    } else if ((errorCode = writeToFile(fd, filedata.constData(), filedata.size())) != 0) {


Reproducible: Always
Comment 1 Christoph Feck 2016-06-20 04:01:11 UTC
Replaced with bug 364381.