mirror of
https://github.com/godotengine/godot-visual-script.git
synced 2025-12-31 21:48:42 +03:00
9a4df15d3e
We've made several updates to improve our codebase and application performance. - Clang format was applied for better readability and consistency, aiding other developers in understanding and contributing to the code. - We resolved issues that were causing visual script compilation failure. The visual script now compiles successfully, ensuring application functionality. - Broken sections within the codebase were fixed, improving overall stability. - Built-in functions from the visual script were removed to simplify the code and enhance readability. - Generic search performance was improved to provide faster results, enhancing user experience. - Missing flow nodes were added to the Visual Script, ensuring all necessary components are present for correct functioning. In an effort to streamline the codebase: - `VisualScriptComment` class and related code were removed, reducing complexity and improving maintainability. - Error messages were optimized for quicker feedback when errors occur. - Licenses were updated to reflect recent changes, ensuring legal compliance and project transparency. - The `get_global_name()` override in `visual_script.h` was fixed, and `TYPE_BUILTIN_FUNC` in `visual_script_expression.h` was removed, improving code functionality. - Search logic was refactored to avoid double searching, enhancing performance. - Documentation was updated to reflect recent changes, providing accurate information to users and developers. - Property selection logic in `VisualScriptPropertySelector` was refactored for easier understanding and modification. - Code was refactored to avoid variable shadowing, improving readability and reducing potential errors. - `.clang-format` and `.clang-tidy` configuration files were added to ensure consistent code styling. To make the code more robust and easier to understand: - Variable names were corrected for clarity, and error handling in `visual_script_expression.cpp` was improved. - Function and variable names were refactored for better readability and maintainability. - Member editing logic in `VisualScriptEditor` was simplified. - Name variables were updated to be unique, avoiding potential conflicts and errors. - The `VisualScriptSubCall` class was refactored for simplicity and ease of understanding. For macOS workflow: - It was updated to use the latest version and correct path for installing Vulkan SDK, enabling the application to leverage the latest features and improvements from the Vulkan SDK. - Mac Vulkan SDK was installed to support Vulkan-based functionalities. Lastly, we made necessary changes to pass CI/CD tests, ensuring the code quality and stability of the application.
200 lines
6.0 KiB
YAML
200 lines
6.0 KiB
YAML
# Commented out parameters are those with the same value as base LLVM style.
|
|
# We can uncomment them if we want to change their value, or enforce the
|
|
# chosen value in case the base style changes (last sync: Clang 14.0).
|
|
---
|
|
### General config, applies to all languages ###
|
|
BasedOnStyle: LLVM
|
|
AccessModifierOffset: -4
|
|
AlignAfterOpenBracket: DontAlign
|
|
# AlignArrayOfStructures: None
|
|
# AlignConsecutiveMacros: None
|
|
# AlignConsecutiveAssignments: None
|
|
# AlignConsecutiveBitFields: None
|
|
# AlignConsecutiveDeclarations: None
|
|
# AlignEscapedNewlines: Right
|
|
AlignOperands: DontAlign
|
|
AlignTrailingComments: false
|
|
# AllowAllArgumentsOnNextLine: true
|
|
AllowAllParametersOfDeclarationOnNextLine: false
|
|
# AllowShortEnumsOnASingleLine: true
|
|
# AllowShortBlocksOnASingleLine: Never
|
|
# AllowShortCaseLabelsOnASingleLine: false
|
|
# AllowShortFunctionsOnASingleLine: All
|
|
# AllowShortLambdasOnASingleLine: All
|
|
# AllowShortIfStatementsOnASingleLine: Never
|
|
# AllowShortLoopsOnASingleLine: false
|
|
# AlwaysBreakAfterDefinitionReturnType: None
|
|
# AlwaysBreakAfterReturnType: None
|
|
# AlwaysBreakBeforeMultilineStrings: false
|
|
# AlwaysBreakTemplateDeclarations: MultiLine
|
|
# AttributeMacros:
|
|
# - __capability
|
|
# BinPackArguments: true
|
|
# BinPackParameters: true
|
|
# BraceWrapping:
|
|
# AfterCaseLabel: false
|
|
# AfterClass: false
|
|
# AfterControlStatement: Never
|
|
# AfterEnum: false
|
|
# AfterFunction: false
|
|
# AfterNamespace: false
|
|
# AfterObjCDeclaration: false
|
|
# AfterStruct: false
|
|
# AfterUnion: false
|
|
# AfterExternBlock: false
|
|
# BeforeCatch: false
|
|
# BeforeElse: false
|
|
# BeforeLambdaBody: false
|
|
# BeforeWhile: false
|
|
# IndentBraces: false
|
|
# SplitEmptyFunction: true
|
|
# SplitEmptyRecord: true
|
|
# SplitEmptyNamespace: true
|
|
# BreakBeforeBinaryOperators: None
|
|
# BreakBeforeConceptDeclarations: true
|
|
# BreakBeforeBraces: Attach
|
|
# BreakBeforeInheritanceComma: false
|
|
# BreakInheritanceList: BeforeColon
|
|
# BreakBeforeTernaryOperators: true
|
|
# BreakConstructorInitializersBeforeComma: false
|
|
BreakConstructorInitializers: AfterColon
|
|
# BreakStringLiterals: true
|
|
ColumnLimit: 0
|
|
# CommentPragmas: '^ IWYU pragma:'
|
|
# QualifierAlignment: Leave
|
|
# CompactNamespaces: false
|
|
ConstructorInitializerIndentWidth: 8
|
|
ContinuationIndentWidth: 8
|
|
Cpp11BracedListStyle: false
|
|
# DeriveLineEnding: true
|
|
# DerivePointerAlignment: false
|
|
# DisableFormat: false
|
|
# EmptyLineAfterAccessModifier: Never
|
|
# EmptyLineBeforeAccessModifier: LogicalBlock
|
|
# ExperimentalAutoDetectBinPacking: false
|
|
# PackConstructorInitializers: BinPack
|
|
ConstructorInitializerAllOnOneLineOrOnePerLine: true
|
|
# AllowAllConstructorInitializersOnNextLine: true
|
|
# FixNamespaceComments: true
|
|
# ForEachMacros:
|
|
# - foreach
|
|
# - Q_FOREACH
|
|
# - BOOST_FOREACH
|
|
# IfMacros:
|
|
# - KJ_IF_MAYBE
|
|
# IncludeBlocks: Preserve
|
|
IncludeCategories:
|
|
- Regex: '".*"'
|
|
Priority: 1
|
|
- Regex: '^<.*\.h>'
|
|
Priority: 2
|
|
- Regex: '^<.*'
|
|
Priority: 3
|
|
# IncludeIsMainRegex: '(Test)?$'
|
|
# IncludeIsMainSourceRegex: ''
|
|
# IndentAccessModifiers: false
|
|
IndentCaseLabels: true
|
|
# IndentCaseBlocks: false
|
|
# IndentGotoLabels: true
|
|
# IndentPPDirectives: None
|
|
# IndentExternBlock: AfterExternBlock
|
|
# IndentRequires: false
|
|
IndentWidth: 4
|
|
# IndentWrappedFunctionNames: false
|
|
# InsertTrailingCommas: None
|
|
# JavaScriptQuotes: Leave
|
|
# JavaScriptWrapImports: true
|
|
KeepEmptyLinesAtTheStartOfBlocks: false
|
|
# LambdaBodyIndentation: Signature
|
|
# MacroBlockBegin: ''
|
|
# MacroBlockEnd: ''
|
|
# MaxEmptyLinesToKeep: 1
|
|
# NamespaceIndentation: None
|
|
# PenaltyBreakAssignment: 2
|
|
# PenaltyBreakBeforeFirstCallParameter: 19
|
|
# PenaltyBreakComment: 300
|
|
# PenaltyBreakFirstLessLess: 120
|
|
# PenaltyBreakOpenParenthesis: 0
|
|
# PenaltyBreakString: 1000
|
|
# PenaltyBreakTemplateDeclaration: 10
|
|
# PenaltyExcessCharacter: 1000000
|
|
# PenaltyReturnTypeOnItsOwnLine: 60
|
|
# PenaltyIndentedWhitespace: 0
|
|
# PointerAlignment: Right
|
|
# PPIndentWidth: -1
|
|
# ReferenceAlignment: Pointer
|
|
# ReflowComments: true
|
|
# RemoveBracesLLVM: false
|
|
# SeparateDefinitionBlocks: Leave
|
|
# ShortNamespaceLines: 1
|
|
# SortIncludes: CaseSensitive
|
|
# SortJavaStaticImport: Before
|
|
# SortUsingDeclarations: true
|
|
# SpaceAfterCStyleCast: false
|
|
# SpaceAfterLogicalNot: false
|
|
# SpaceAfterTemplateKeyword: true
|
|
# SpaceBeforeAssignmentOperators: true
|
|
# SpaceBeforeCaseColon: false
|
|
# SpaceBeforeCpp11BracedList: false
|
|
# SpaceBeforeCtorInitializerColon: true
|
|
# SpaceBeforeInheritanceColon: true
|
|
# SpaceBeforeParens: ControlStatements
|
|
# SpaceBeforeParensOptions:
|
|
# AfterControlStatements: true
|
|
# AfterForeachMacros: true
|
|
# AfterFunctionDefinitionName: false
|
|
# AfterFunctionDeclarationName: false
|
|
# AfterIfMacros: true
|
|
# AfterOverloadedOperator: false
|
|
# BeforeNonEmptyParentheses: false
|
|
# SpaceAroundPointerQualifiers: Default
|
|
# SpaceBeforeRangeBasedForLoopColon: true
|
|
# SpaceInEmptyBlock: false
|
|
# SpaceInEmptyParentheses: false
|
|
# SpacesBeforeTrailingComments: 1
|
|
# SpacesInAngles: Never
|
|
# SpacesInConditionalStatement: false
|
|
# SpacesInContainerLiterals: true
|
|
# SpacesInCStyleCastParentheses: false
|
|
## Godot TODO: We'll want to use a min of 1, but we need to see how to fix
|
|
## our comment capitalization at the same time.
|
|
SpacesInLineCommentPrefix:
|
|
Minimum: 0
|
|
Maximum: -1
|
|
# SpacesInParentheses: false
|
|
# SpacesInSquareBrackets: false
|
|
# SpaceBeforeSquareBrackets: false
|
|
# BitFieldColonSpacing: Both
|
|
# StatementAttributeLikeMacros:
|
|
# - Q_EMIT
|
|
# StatementMacros:
|
|
# - Q_UNUSED
|
|
# - QT_REQUIRE_VERSION
|
|
TabWidth: 4
|
|
# UseCRLF: false
|
|
UseTab: Always
|
|
# WhitespaceSensitiveMacros:
|
|
# - STRINGIZE
|
|
# - PP_STRINGIZE
|
|
# - BOOST_PP_STRINGIZE
|
|
# - NS_SWIFT_NAME
|
|
# - CF_SWIFT_NAME
|
|
---
|
|
### C++ specific config ###
|
|
Language: Cpp
|
|
Standard: c++17
|
|
---
|
|
### ObjC specific config ###
|
|
Language: ObjC
|
|
# ObjCBinPackProtocolList: Auto
|
|
ObjCBlockIndentWidth: 4
|
|
# ObjCBreakBeforeNestedBlockParam: true
|
|
# ObjCSpaceAfterProperty: false
|
|
# ObjCSpaceBeforeProtocolList: true
|
|
---
|
|
### Java specific config ###
|
|
Language: Java
|
|
# BreakAfterJavaFieldAnnotations: false
|
|
JavaImportGroups: ['org.godotengine', 'android', 'androidx', 'com.android', 'com.google', 'java', 'javax']
|
|
...
|