D26317: Port endl to "\n". endl in qt5.15 is namespaced. We don't need to flush as when QFile is deleted it flush data
    Albert Astals Cid 
    noreply at phabricator.kde.org
       
    Tue Dec 31 11:21:49 GMT 2019
    
    
  
aacid added a comment.
  Isn't it better to just use `Qt::endl` ?
  
  I think it's much clearer to understand `Qt::endl` than `QLatin1Char('\n')`
  
  But if we prefer to change to use \n it should be merged into the existing strings, doesn't make much sense to do 
  `outStream << "</l:i18n>" << QLatin1Char('\n');`
  instead of
  `outStream << "</l:i18n>\n';`
REPOSITORY
  R238 KDocTools
REVISION DETAIL
  https://phabricator.kde.org/D26317
To: mlaurent, dfaure
Cc: aacid, anthonyfieroni, kde-frameworks-devel, kde-doc-english, LeGast00n, gennad, fbampaloukas, GB_2, michaelh, ngraham, bruns, skadinna
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20191231/32a4126b/attachment.html>
    
    
More information about the kde-doc-english
mailing list