aleksejshilin added a comment.

  In D12490#252959 <https://phabricator.kde.org/D12490#252959>, @anthonyfieroni 
wrote:
  
  > path1 *should* never be empty
  
  
  Hm... I thought concatPaths() is supposed to be used like this: 
`concatPaths(basePath, relativePath)` - then an empty basePath makes sense (and 
means 'current directory'). If this is not the case, then:
  
  - KUrlCompletion (and probably KCompletion) needs to be fixed;
  - `Q_ASSERT(!path1.isEmpty())` should be added here so that such bugs get 
noticed.
  
  Is this correct?

REPOSITORY
  R241 KIO

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

To: aleksejshilin, #frameworks, anthonyfieroni, dfaure
Cc: michaelh, bruns

Reply via email to