dfaure added inline comments.

INLINE COMMENTS

> dfaure wrote in fdreceiver.cpp:36
> print out `m_path.toLocal8Bit()` here?

Possibly nitpicking, but toStdString converts to utf8 while toLocal8Bit() is 
what's used for SocketAddress and what's recommended for terminal output as 
well. If toLocal8Bit() doesn't compile, then you need 
m_path.toLocal8Bit().constData(). Or the short version for a cout/cerr 
statement:

  << qPrintable(m_path) << std::endl;

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D12744

To: chinmoyr, dfaure, ossi
Cc: #frameworks, michaelh, ngraham, bruns

Reply via email to