ltoscano added a comment.

  In D26317#585315 <https://phabricator.kde.org/D26317#585315>, @aacid wrote:
  
  > 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';`
  
  
  I agree with Albert here; I think it's a bit more painful but it's probably 
going to be cleaner.

REPOSITORY
  R238 KDocTools

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

To: mlaurent, dfaure
Cc: ltoscano, aacid, anthonyfieroni, kde-frameworks-devel, kde-doc-english, 
LeGast00n, gennad, fbampaloukas, GB_2, michaelh, ngraham, bruns, skadinna

Reply via email to