Differential D15472

[sftp] replace memset(0) hack with c++ initialization
ClosedPublic
Actions

Authored by sitter on Sep 13 2018, 9:20 AM.

Details

Reviewers
broulik
Commits
R320:0f92b54ca21a: [sftp] replace memset(0) hack with c++ initialization
Summary

new T() in c++11 in a class with a default constructor that is not user
provided will zero initialize the object.
(see docs on value initialization)

Diff Detail

Repository
R320 KIO Extras
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter requested review of this revision.Sep 13 2018, 9:20 AM
sitter created this revision.
broulik accepted this revision.Sep 13 2018, 9:54 AM

Sweet.

This revision is now accepted and ready to land.Sep 13 2018, 9:54 AM
Closed by commit R320:0f92b54ca21a: [sftp] replace memset(0) hack with c++ initialization (authored by sitter). · Explain WhySep 13 2018, 10:09 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msftp/kio_sftp.cpp (5 lines)

Diff 41526

View Options

sftp/kio_sftp.cpp

Loading...