Refactoring #2466

Use full paths in #include directives

Added by wuttke about 1 month ago. Updated 29 days ago.

Status:ResolvedStart date:23 Jun 2020
Priority:NormalDue date:
Assignee:wuttke% Done:

0%

Category:-
Target version:v1.18

Description

Instead of

#include "SessionItem.h" 

we'll have
#include "GUI/coregui/Models/SessionItem.h" 

This is recommended by https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes.

Advantages:
  • Provides valuable information to the reader of the code.
  • File names must be no longer unique. This allows in some cases for shorter or more natural names.
  • Slight simplification of CMake code.
  • Conformity with our other projects.
Disavantage:
  • Renaming files requires more typing, or use of a script.

The required script mv-src is available in :mlz/intern/text-processing-scripts.


Related issues

Related to BornAgain - Refactoring #1766: Replace all form factors in sample creation by shapes Sprint 16 Feb 2017

History

#1 Updated by wuttke about 1 month ago

  • Description updated (diff)

#2 Updated by wuttke about 1 month ago

  • Related to Refactoring #1766: Replace all form factors in sample creation by shapes added

#3 Updated by wuttke about 1 month ago

  • Status changed from New to Sprint
  • Target version set to v1.18

#4 Updated by wuttke 29 days ago

  • Status changed from Sprint to Resolved

Also available in: Atom PDF