summaryrefslogtreecommitdiffstats
path: root/modules
Commit message (Collapse)AuthorAgeFilesLines
* Fix GDNative wrapper type sizes (RID, Variant, Packed*Array), add size ↵bruvzg2020-05-1621-9/+53
| | | | checking static asserts.
* websocket: Fix crash at exit when not running the editorIgnacio Etcheverry2020-05-161-1/+8
| | | | | | | EditorDebuggerServer::register_protocol_handler must not be called before editor initialization. Otherwise, if the editor is never initialized, the added StringName will not be released until static destructors are called (instead of being release during editor deinitialization).
* Fix incorrect capabilities notification in LSPOliver Frank2020-05-151-3/+3
| | | | (cherry picked from commit 2173d041af711c111d60d1fc1f45f96f4f8c7271)
* Improve jsonrpc error reportingOliver Frank2020-05-151-1/+1
| | | | (cherry picked from commit dac8b7b6f4fec0136a3dc95afae0aa8aa8ce9fd9)
* break, continue outside of a loop, match statement handledThakee Nathees2020-05-152-0/+37
|
* Merge pull request #38738 from akien-mga/cause-we-never-go-out-of-styleRémi Verschelde2020-05-14267-5678/+3098
|\ | | | | Style: Remove new line at block start, enforce line between functions, enforce braces in if and loop blocks
| * Style: Enforce braces around if blocks and loopsRémi Verschelde2020-05-14101-1443/+2707
| | | | | | | | | | Using clang-tidy's `readability-braces-around-statements`. https://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html
| * Style: Enforce separation line between function definitionsRémi Verschelde2020-05-1441-0/+259
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I couldn't find a tool that enforces it, so I went the manual route: ``` find -name "thirdparty" -prune \ -o -name "*.cpp" -o -name "*.h" -o -name "*.m" -o -name "*.mm" \ -o -name "*.glsl" > files perl -0777 -pi -e 's/\n}\n([^#])/\n}\n\n\1/g' $(cat files) misc/scripts/fix_style.sh -c ``` This adds a newline after all `}` on the first column, unless they are followed by `#` (typically `#endif`). This leads to having lots of places with two lines between function/class definitions, but clang-format then fixes it as we enforce max one line of separation. This doesn't fix potential occurrences of function definitions which are indented (e.g. for a helper class defined in a .cpp), but it's better than nothing. Also can't be made to run easily on CI/hooks so we'll have to be careful with new code. Part of #33027.
| * Style: clang-format: Disable KeepEmptyLinesAtTheStartOfBlocksRémi Verschelde2020-05-14257-4223/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Which means that reduz' beloved style which we all became used to will now be changed automatically to remove the first empty line. This makes us lean closer to 1TBS (the one true brace style) instead of hybridating it with some Allman-inspired spacing. There's still the case of braces around single-statement blocks that needs to be addressed (but clang-format can't help with that, but clang-tidy may if we agree about it). Part of #33027.
| * Style: Fix missing/invalid copyright headersRémi Verschelde2020-05-148-12/+132
| |
* | Merge pull request #38610 from ThakeeNathees/infer-type-null-errorRémi Verschelde2020-05-141-0/+8
|\ \ | |/ |/| set parser error when infer type is null
| * set parser error when infer type is nullThakee Nathees2020-05-131-0/+8
| |
* | Merge pull request #38611 from ThakeeNathees/shadow-var-warning-bug-fixRémi Verschelde2020-05-141-11/+5
|\ \ | | | | | | shadowed var warning in nested block bug fix
| * | shadowed var warning in nested block bug fixThakee Nathees2020-05-101-11/+5
| |/ | | | | | | Fix: #38552
* | Remove redundant void argument listsRémi Verschelde2020-05-143-3/+3
| | | | | | | | | | Using clang-tidy's `modernize-redundant-void-arg`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-redundant-void-arg.html
* | Enforce use of bool literals instead of integersRémi Verschelde2020-05-149-24/+24
| | | | | | | | | | Using clang-tidy's `modernize-use-bool-literals`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-bool-literals.html
* | Modernize remaining uses of 0/NULL instead of nullptr (C++11)Rémi Verschelde2020-05-145-6/+6
| | | | | | | | | | Using clang-tidy's `modernize-use-nullptr`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-nullptr.html
* | Port member initialization from constructor to declaration (C++11)Rémi Verschelde2020-05-1450-615/+357
| | | | | | | | | | | | | | | | | | | | Using `clang-tidy`'s `modernize-use-default-member-init` check and manual review of the changes, and some extra manual changes that `clang-tidy` failed to do. Also went manually through all of `core` to find occurrences that `clang-tidy` couldn't handle, especially all initializations done in a constructor without using initializer lists.
* | Merge pull request #38609 from ↵Rémi Verschelde2020-05-131-0/+2
|\ \ | | | | | | | | | | | | ThakeeNathees/range-crash-with-non-numeric-const-fix range() with non-numeric const argument crash fix
| * | range() with non-numeric const argument crash fixThakee Nathees2020-05-101-0/+2
| |/
* | Merge pull request #38708 from ThakeeNathees/init-assign-type-parser-bug-fixRémi Verschelde2020-05-131-0/+1
|\ \ | | | | | | regression: var declaration type info parser bug fix
| * | regression: var declaration type info parser bug fixThakee Nathees2020-05-131-0/+1
| | |
* | | regression: static func can't access const fixThakee Nathees2020-05-132-4/+7
|/ /
* | Add WebSocket debugger, use it for Javascript.Fabio Alessandrelli2020-05-125-0/+363
| |
* | Remove WebSocket defaults from project settings.Fabio Alessandrelli2020-05-125-35/+14
| | | | | | | | | | Can be manually set, let's not pollute them further. Should also be done for WebRTC.
* | Merge pull request #38672 from madmiraal/fix-physicsw-warningRémi Verschelde2020-05-111-19/+19
|\ \ | | | | | | Fix 'physicsw' may be used uninitialized warning in csg_shape.cpp
| * | Fix 'physicsw' may be used uninitialized warning in csg_shape.cppMarcel Admiraal2020-05-111-19/+19
| | |
* | | thirdparty: Cleanup after #38386, document provenance and copyrightRémi Verschelde2020-05-111-2/+2
| | | | | | | | | | | | Also renamed `delaunay.h` to `delaunay_2d.h` to match the class name.
* | | Merge pull request #38386 from reduz/new-lightmapperRémi Verschelde2020-05-1121-4/+3581
|\ \ \ | | | | | | | | New GPU lightmapper
| * | | New lightmapperJuan Linietsky2020-05-1021-4/+3581
| | | | | | | | | | | | | | | | | | | | | | | | | | | | -Added LocalVector (needed it) -Added stb_rect_pack (It's pretty cool, we could probably use it for other stuff too) -Fixes and changes all around the place -Added library for 128 bits fixed point (required for Delaunay3D)
* | | | Merge pull request #38587 from Faless/js/improvementsRémi Verschelde2020-05-112-6/+6
|\ \ \ \ | | | | | | | | | | DisplayServerJavaScript, improvements to HTML5 build (still dummy renderer).
| * | | | Fix Closure compiler build, python style.Fabio Alessandrelli2020-05-102-6/+6
| |/ / / | | | | | | | | | | | | Move copyToFS into utils.js library included with '--pre-js'.
* | | | Merge pull request #38638 from pepegadeveloper123/mono_inherited_scene_bugRémi Verschelde2020-05-112-68/+94
|\ \ \ \ | | | | | | | | | | Fix inherited C# scene not inheriting parent's fields
| * | | | Inherited C# scene not inheriting parent's fieldspepegadeveloper1232020-05-112-68/+94
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a child scene inherits a parent scene with a C# root node, the parent scene's export variables appear to assume values set in the parent scene, in the child scene's Inspector. However, when the child scene is played, the parent scene's export variables assume default values. When a node is created, it inherits its parent C# script's fields from the map CSharpScriptInstance::script->member_info. However this map was not initialized outside the editor, and this commit ensured it is. This fixes issues #36480 and #37581.
* | | | Merge pull request #38650 from neikeq/dotnet-cli-supportRémi Verschelde2020-05-1115-95/+198
|\ \ \ \ | | | | | | | | | | C#: Support for building with the dotnet CLI
| * | | | C#: Support for building with the dotnet CLIIgnacio Etcheverry2020-05-1115-95/+198
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | By adding a reference to the 'Microsoft.NETFramework.ReferenceAssemblies' nuget package, we can build projects targeting .NET Framework with the dotnet CLI. By referencing this package we also don't need to install Mono on Linux/macOS or .NET Framework on Windows, as the assemblies are taken from the package.
* / / / Implement CSharpScript::inherits_scriptIgnacio Etcheverry2020-05-101-4/+9
|/ / /
* / / Revert "Renamed plane's d to distance"Rémi Verschelde2020-05-108-41/+41
|/ / | | | | | | | | | | | | This reverts commit ec7b481170dcd6a7b4cf0e6c1221e204ff7945f3. This was wrong, `d` is not a distance but the `d` constant in the parametric equation `ax + by + cz = d` describing the plane.
* | Merge pull request #38621 from akien-mga/stylé-comme-jamaisRémi Verschelde2020-05-1036-433/+1115
|\ \ | | | | | | Style: clang-format: Disable if statements and case labels on single line
| * | Style: clang-format: Disable AllowShortCaseLabelsOnASingleLineRémi Verschelde2020-05-1016-312/+877
| | | | | | | | | | | | Part of #33027.
| * | Style: clang-format: Disable AllowShortIfStatementsOnASingleLineRémi Verschelde2020-05-1024-121/+238
| | | | | | | | | | | | | | | | | | | | | Part of #33027, also discussed in #29848. Enforcing the use of brackets even on single line statements would be preferred, but `clang-format` doesn't have this functionality yet.
* | | Merge pull request #38600 from neikeq/noIgnacio Roldán Etcheverry2020-05-1071-1645/+2510
|\ \ \ | |/ / |/| | Switch to nuget Microsoft.Build and rewrite GodotTools messasing protocol
| * | C#: Rewrite GodotTools messaging protocolIgnacio Etcheverry2020-05-0942-1047/+2239
| | |
| * | C#: Use Sdks in GodotTools csprojs and switch to nuget Microsoft.BuildIgnacio Etcheverry2020-05-0926-531/+140
| | |
| * | C#/Mono: Check assembly version when loadingIgnacio Etcheverry2020-05-099-72/+136
| | | | | | | | | | | | | | | Not sure if we should check revision too, but this is good enough for what we want. This will be needed to load the correct Microsoft.Build when we switch to the nuget version.
* | | Merge pull request #37160 from ↵Rémi Verschelde2020-05-102-4/+27
|\ \ \ | | | | | | | | | | | | | | | | ThakeeNathees/saveing-cyclic-inheritance-crash-fix Fix: saving gdscript with cyclic inheritance crash the editor
| * | | fixed: saving gdscript with cyclic inheritance crash the editorThakee Nathees2020-05-102-4/+27
| | |/ | |/| | | | | | | Fix: #9609
* | | Merge pull request #37020 from aaronfranke/rangeRémi Verschelde2020-05-102-9/+9
|\ \ \ | | | | | | | | Allow using integer vectors for iteration and make range() use them
| * | | Change get_completion_identifier_is_function to return a boolAaron Franke2020-05-092-2/+2
| | | | | | | | | | | | | | | | A minor bugfix
| * | | Allow using integer vectors for iteration and make range() use themAaron Franke2020-05-091-7/+7
| | | |