feat: create SCM plugin and add line drawing tool (#14)(#15) (#21)

Author: xLPMG

.clang-format

@@ -1,100 +1,47 @@
 ---
-Language:        Cpp
-AccessModifierOffset: -3
-AlignAfterOpenBracket: Align
-AlignConsecutiveAssignments: AcrossEmptyLinesAndComments
-AlignConsecutiveDeclarations: AcrossEmptyLinesAndComments
-AlignEscapedNewlines: Right
-AlignOperands:   true
-AlignTrailingComments: true
-AllowAllParametersOfDeclarationOnNextLine: false
-AllowShortBlocksOnASingleLine: Empty
-AllowShortCaseLabelsOnASingleLine: false
-AllowShortFunctionsOnASingleLine: Inline
-AllowShortIfStatementsOnASingleLine: Never
-AllowShortLoopsOnASingleLine: false
-AlwaysBreakAfterReturnType: None
-AlwaysBreakBeforeMultilineStrings: false
-AlwaysBreakTemplateDeclarations: Yes
-BinPackArguments: false
-BinPackParameters: false
+BasedOnStyle: LLVM
+IndentWidth: 8
+TabWidth: 8
+UseTab: Always
+AlignConsecutiveAssignments: false
+AlignTrailingComments: false
+AllowShortFunctionsOnASingleLine: None
+BreakBeforeBraces: Allman
 BraceWrapping:
-  AfterClass:      true
-  AfterControlStatement: Never
-  AfterEnum:       true
-  AfterFunction:   true
-  AfterNamespace:  true
-  AfterObjCDeclaration: true
-  AfterStruct:     true
-  AfterUnion:      true
-  BeforeCatch:     false
-  BeforeElse:      false
-  IndentBraces:    false
-BreakBeforeBinaryOperators: None
-BreakBeforeBraces: Custom
-BreakBeforeInheritanceComma: true
-BreakBeforeTernaryOperators: true
-BreakConstructorInitializersBeforeComma: false
-BreakConstructorInitializers: BeforeComma
-BreakAfterJavaFieldAnnotations: false
-BreakStringLiterals: true
-ColumnLimit:     120
-CommentPragmas:  '^ IWYU pragma:'
-CompactNamespaces: false
-ConstructorInitializerAllOnOneLineOrOnePerLine: false
-ConstructorInitializerIndentWidth: 3
-ContinuationIndentWidth: 2
-Cpp11BracedListStyle: false
-DerivePointerAlignment: false
-DisableFormat:   false
-ExperimentalAutoDetectBinPacking: false
-FixNamespaceComments: false
-ForEachMacros:   [ foreach, Q_FOREACH, BOOST_FOREACH ]
-IncludeCategories:
-  - Regex:           '^"(llvm|llvm-c|clang|clang-c)/'
-    Priority:        2
-  - Regex:           '^(<|"(gtest|isl|json)/)'
-    Priority:        3
-  - Regex:           '.*'
-    Priority:        1
-IncludeIsMainRegex: '$'
+  AfterClass: true
+  AfterControlStatement: true
+  AfterEnum: true
+  AfterFunction: true
+  AfterNamespace: true
+  AfterStruct: true
+  AfterUnion: true
+  BeforeCatch: true
+  BeforeElse: true
+  IndentBraces: false
 IndentCaseLabels: true
-IndentWidth:     3
-IndentWrappedFunctionNames: false
-JavaScriptQuotes: Leave
-JavaScriptWrapImports: true
-KeepEmptyLinesAtTheStartOfBlocks: true
-MacroBlockBegin: ''
-MacroBlockEnd:   ''
-MaxEmptyLinesToKeep: 1
-NamespaceIndentation: All
-ObjCBlockIndentWidth: 2
-ObjCSpaceAfterProperty: true
-ObjCSpaceBeforeProtocolList: false
-PenaltyBreakAssignment: 2
-PenaltyBreakBeforeFirstCallParameter: 19
-PenaltyBreakComment: 300
-PenaltyBreakFirstLessLess: 120
-PenaltyBreakString: 1000
-PenaltyExcessCharacter: 1000000
-PenaltyReturnTypeOnItsOwnLine: 200
-PointerAlignment: Middle
-ReflowComments:  true
-SortIncludes: CaseSensitive
-SpaceAfterCStyleCast: false
-SpaceAfterTemplateKeyword: false
-SpaceBeforeAssignmentOperators: true
+AccessModifierOffset: -8
+IndentPPDirectives: None
+IndentAccessModifiers: false
+IndentWrappedFunctionNames: true
+ColumnLimit: 120
+PointerAlignment: Right
+SpacesInParentheses: false
 SpaceBeforeParens: ControlStatements
-SpaceInEmptyParentheses: false
-SpacesBeforeTrailingComments: 1
-SpacesInAngles:  false
-SpacesInContainerLiterals: true
+SpaceBeforeRangeBasedForLoopColon: true
+SpaceBeforeSquareBrackets: false
+SpacesInAngles: false
+SpacesInContainerLiterals: false
 SpacesInCStyleCastParentheses: false
-SpacesInParentheses: false
+SpacesBeforeTrailingComments: 2
+AllowShortIfStatementsOnASingleLine: false
+AllowShortLoopsOnASingleLine: false
+AllowShortBlocksOnASingleLine: false
+SortIncludes: false
+NamespaceIndentation: None
+BreakConstructorInitializers: BeforeComma
+ConstructorInitializerIndentWidth: 8
 SpacesInSquareBrackets: false
-Standard: c++11
-TabWidth:        3
-UseTab:          Never
-...
-
-
+SpaceAfterCStyleCast: false
+DerivePointerAlignment: false
+BinPackParameters: false  # Each parameter is placed on its own line.
+BinPackArguments: false   # Each argument is placed on its own line.

.clang-tidy

@@ -1,37 +1,77 @@
 ---
-Checks:                 '*,-google-default-arguments,-fuchsia-*,-zircon-*,-abseil-*,-llvmlibc-*'
-WarningsAsErrors:       ''
-HeaderFilterRegex:      ''
-AnalyzeTemporaryDtors:  false
-FormatStyle:            file
+Checks: >
+  -*, 
+  readability-misleading-indentation,
+  readability-redundant-declaration,
+  readability-redundant-member-init,
+  readability-use-anyofallof,
+  readability-identifier-naming,
+  readability-braces-around-statements,
+  readability-function-cognitive-complexity,
+  readability-magic-numbers,
+  readability-else-after-return,
+  readability-avoid-const-params-in-decls,
+  readability-container-size-empty,
+  readability-redundant-string-cstr,
+  readability-redundant-control-flow,
+  readability-simplify-boolean-expr,
+  readability-uppercase-literal-suffix,
+  readability-static-accessed-through-instance,
+  readability-named-parameter,
+  modernize-use-auto,
+  modernize-use-nullptr,
+  modernize-use-override,
+  modernize-deprecated-headers,
+  modernize-use-using,
+  modernize-avoid-c-arrays,
+  modernize-make-unique,
+  modernize-make-shared,
+  cppcoreguidelines-avoid-magic-numbers,
+  cppcoreguidelines-pro-bounds-constant-array-index,
+  cppcoreguidelines-pro-type-static-cast-downcast,
+  cppcoreguidelines-pro-type-reinterpret-cast,
+  cppcoreguidelines-pro-type-cstyle-cast,
+  cppcoreguidelines-pro-type-member-init,
+  cppcoreguidelines-pro-bounds-pointer-arithmetic,
+  cppcoreguidelines-pro-type-vararg,
+  cppcoreguidelines-no-malloc,
+  cppcoreguidelines-special-member-functions,
+  cppcoreguidelines-avoid-non-const-global-variables,
+  cppcoreguidelines-init-variables,
+  hicpp-no-array-decay,
+  hicpp-signed-bitwise,
+  hicpp-uppercase-literal-suffix,
+  hicpp-use-auto,
+  hicpp-use-nullptr,
+  hicpp-use-override,
+  hicpp-deprecated-headers,
+  hicpp-explicit-conversions,
+  hicpp-multiway-paths-covered,
+  hicpp-no-malloc,
+  hicpp-special-member-functions,
+  hicpp-avoid-goto,
+  performance-for-range-copy,
+  performance-implicit-conversion-in-loop,
+  performance-inefficient-vector-operation,
+  performance-unnecessary-value-param
+WarningsAsErrors: "*"
+HeaderFilterRegex: ".*"
+AnalyzeTemporaryDtors: true
+FormatStyle: file
 CheckOptions:
-  - key:                cert-dcl16-c.NewSuffixes
-    value:              'L;LL;LU;LLU'
-  - key:                cppcoreguidelines-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
-    value:              '1'
-  - key:                google-readability-braces-around-statements.ShortStatementLines
-    value:              '1'
-  - key:                google-readability-function-size.StatementThreshold
-    value:              '800'
-  - key:                google-readability-namespace-comments.ShortNamespaceLines
-    value:              '10'
-  - key:                google-readability-namespace-comments.SpacesBeforeComments
-    value:              '2'
-  - key:                modernize-loop-convert.MaxCopySize
-    value:              '16'
-  - key:                modernize-loop-convert.MinConfidence
-    value:              reasonable
-  - key:                modernize-loop-convert.NamingStyle
-    value:              CamelCase
-  - key:                modernize-pass-by-value.IncludeStyle
-    value:              llvm
-  - key:                modernize-replace-auto-ptr.IncludeStyle
-    value:              llvm
-  - key:                modernize-use-nullptr.NullMacros
-    value:              'NULL'
-  - key:                misc-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
-    value:              '1'
-  - key:                modernize-use-default-member-init.UseAssignment
-    value:              '1'
-...
-
+  - key: readability-identifier-naming.ClassCase
+    value: CamelCase
+  - key: readability-identifier-naming.MethodCase
+    value: camelBack
+  - key: readability-identifier-naming.VariableCase
+    value: camelBack
+  - key: readability-identifier-naming.ConstantCase
+    value: UPPER_CASE
+  - key: readability-identifier-naming.MacroDefinitionCase
+    value: UPPER_CASE
+  - key: readability-identifier-naming.EnumCase
+    value: CamelCase
+  - key: readability-identifier-naming.EnumConstantCase
+    value: CamelCase
+  - key: readability-braces-around-statements.ShortStatementLines
+    value: 1

.gitignore

@@ -75,3 +75,5 @@ qrc_*.cpp
 .kdev4
 # Ignore Eclipse project files
 .project
+# Ignore vscode project files
+.vscode

BUILDING.md

@@ -444,6 +444,7 @@ List of supported parameters (passed as `-DPARAMETER=VALUE`):
 | USE_PLUGIN_SATELLITES              | bool   | ON      | Enable building the Satellites plugin
 | USE_PLUGIN_SCENERY3D               | bool   | ON      | Enable building the 3D Scenery plugin
 | USE_PLUGIN_SIMPLEDRAWLINE          | bool   | OFF     | Enable building the SimpleDrawLine plugin (example of simple graphics plugin)
+| USE_PLUGIN_SKYCULTUREMAKER         | bool   | ON      | Enable building the Sky Culture Maker plugin
 | USE_PLUGIN_SOLARSYSTEMEDITOR       | bool   | ON      | Enable building the Solar System Editor plugin
 | USE_PLUGIN_SUPERNOVAE              | bool   | ON      | Enable building the Historical Supernovae plugin
 | USE_PLUGIN_TELESCOPECONTROL        | bool   | ON      | Enable building the Telescope Control plugin

CMakeLists.txt

@@ -566,6 +566,7 @@ ENDIF()
 ADD_PLUGIN(RemoteSync 1)
 ADD_PLUGIN(Satellites 1)
 ADD_PLUGIN(Scenery3d 1)
+ADD_PLUGIN(SkyCultureMaker 1)
 ADD_PLUGIN(SolarSystemEditor 1)
 ADD_PLUGIN(Supernovae 1)
 ADD_PLUGIN(LensDistortionEstimator 1)

plugins/SkyCultureMaker/CMakeLists.txt

@@ -0,0 +1,7 @@
+# This is the cmake config file for the Sky Culture Maker plugin
+SET(SCM_VERSION "0.1.0")
+
+ADD_DEFINITIONS(-DSKYCULTUREMAKER_PLUGIN_VERSION="${SCM_VERSION}")
+ADD_DEFINITIONS(-DSKYCULTUREMAKER_PLUGIN_LICENSE="GNU GPLv2 or later")
+
+ADD_SUBDIRECTORY( src )