D26877: Simplify calls to whitespace() and use it in more places.

Tomaz Canabrava noreply at phabricator.kde.org
Thu Jan 23 16:22:49 GMT 2020


tcanabrava created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
tcanabrava requested review of this revision.

REVISION SUMMARY
  whitespace now returns a textStream so we don't need to stream() << whitespace()
  Use whitespace where we had stream() << " followed by 4 spaces
  
  There are plenty of manual space manipulation on the code that I plan to
  get rid of, but that will have to wait as it will break every single
  testcase.

TEST PLAN
  Run unittests

REPOSITORY
  R237 KConfig

BRANCH
  simplify_whitespace

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

AFFECTED FILES
  src/kconfig_compiler/KConfigCodeGeneratorBase.cpp
  src/kconfig_compiler/KConfigCodeGeneratorBase.h
  src/kconfig_compiler/KConfigHeaderGenerator.cpp

To: tcanabrava
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200123/4e440083/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list