Ticket #163 (accepted planned)

Opened 3 years ago

Last modified 16 months ago

tidy up namespace indentation

Reported by: ichthyo Owned by: ichthyo
Priority: nice Milestone: 2beta
Component: lumieraProc Keywords:
Cc: Blocked By: #162
Blocking:

Description (last modified by ichthyo) (diff)

Namespaces should either be not indented at all, or be indented in a simple way to support readability. Contrary to this, the existing code is indented according to the behaviour of the "astyle" utility,
because initially a lot of boilerplate code was generated from UML.

Ichthyo meanwhile thinks that some judicious use of indentation actually improves readability, while fine grained indentation slavishly following the number of namespaces tends to get into the way. Thus, when doing this cleanup, within Proc-Layer the goal should be

  • to indent anything within any namespaces by one indentation step (2 spaces)
  • to indent a private/implementation namespace by two steps

Possibly, this should be combined with the namespace cleanup #162

Change History

comment:1 Changed 3 years ago by ichthyo

  • Description modified (diff)
  • Blocked By 162 added; 161 removed

comment:2 Changed 2 years ago by ichthyo

  • Status changed from new to accepted

doing that already on occasion since some time

comment:3 Changed 16 months ago by ichthyo

  • Type changed from fixme to planned

pushing to future (and re-indenting on occasion)

Note: See TracTickets for help on using tickets.