[Differential] [Updated] D4537: EditorConfig support

[Grzegorz Szymaszek]

gszymaszek marked 6 inline comments as done.
gszymaszek added inline comments.

INLINE COMMENTS

> dhaumann wrote in editorconfig.cpp:23
> Better is to write:
> 
>   EditorConfig::EditorConfig(KTextEditor::DocumentPrivate *document)
>       : m_document(document)
>       , m_handle(0)
>   {
>       // ...
>   }

Is it OK to initialize `m_handle` in the constructor? If so, is `m_handle(0)` necessary?

REPOSITORY
  R39 KTextEditor

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: gszymaszek, #ktexteditor
Cc: dhaumann, kwrite-devel, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170213/0699630b/attachment.html>