summaryrefslogtreecommitdiffstats
path: root/modules/gdscript/gdscript_parser.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge pull request #28099 from lupoDharkael/fix-completionRémi Verschelde2019-06-031-1/+1
|\ \ \ | |/ / |/| | Fix code completion not working with class_name
| * | Don't allow class_name in built-in scriptslupoDharkael2019-04-171-1/+1
| | |
* | | Merge pull request #27987 from lupoDharkael/classname-duplicateRémi Verschelde2019-05-281-1/+7
|\ \ \ | | | | | | | | Prevent using multiple class_name in a class
| * | | Prevent using multiple class_name in a classlupoDharkael2019-04-161-1/+7
| | | |
* | | | Fix parse error when extends from subclass of named classesGeequlim2019-05-241-3/+4
| | | |
* | | | Fix typos with codespellRémi Verschelde2019-05-191-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using codespell 1.15.0. Method: ``` $ cat > ../godot-word-whitelist.txt << EOF ang curvelinear doubleclick leapyear lod merchantibility nd numer ois ony que seeked synching te uint unselect webp EOF $ codespell -w -q 3 -I ../godot-word-whitelist.txt --skip="./thirdparty,*.po" $ git diff // undo unwanted changes ```
* | | | Fix typed arrays and dictionaries getting their values sharedBojidar Marinov2019-05-021-29/+16
| | | |
* | | | Merge pull request #27676 from qarmin/small_fixes_2Rémi Verschelde2019-05-011-9/+4
|\ \ \ \ | |_|/ / |/| | | Small fixes to static analyzer bugs
| * | | Small fixes to static analyzer bugsqarmin2019-04-041-9/+4
| | | |
* | | | GDScript: add variable shadowing warninglupoDharkael2019-04-151-0/+16
| |/ / |/| |
* | | Merge pull request #27863 from bojidar-bg/27460-constant-class-clashRémi Verschelde2019-04-121-0/+33
|\ \ \ | | | | | | | | Check subclasses too when checking for name clashes
| * | | Check for subclasses when checking for name clashesBojidar Marinov2019-04-101-0/+33
| | | | | | | | | | | | | | | | Fixes #27460
* | | | GDScript: Don't allow built-in scripts to use class_nameGeorge Marques2019-04-101-0/+4
|/ / /
* | | Style: Apply new changes from clang-format 8.0Rémi Verschelde2019-04-091-5/+12
| | | | | | | | | | | | | | | | | | It seems to stay compatible with formatting done by clang-format 6.0 and 7.0, so contributors can keep using those versions for now (they will not undo those changes).
* | | Fix -Wimplicit-fallthrough warnings from GCC 8Rémi Verschelde2019-04-051-6/+12
|/ / | | | | | | | | | | | | | | | | | | | | | | Adds `FALLTHROUGH` macro to specify when a fallthrough is intentional. Can be replaced by `[[fallthrough]]` if/when we switch to C++17. The warning is now enabled by default for GCC on `extra` warnings level (part of GCC's `-Wextra`). It's not enabled in Clang's `-Wextra` yet, but we could enable it manually once we switch to C++11. There's no equivalent feature in MSVC for now. Fixes #26135.
* | Merge pull request #26097 from ↵Rémi Verschelde2019-04-011-2/+4
|\ \ | | | | | | | | | | | | danielspaniol/25955-wrong-unreachable-warning-after-returning-from-matchs-wildcard-pattern Fix: Wrong unreachable warning after returning from match's wildcard pattern #25955
| * | Require `return` in all match branchesDaniel Spaniol2019-02-201-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before the parser only checked if the catch-all branch has a return in order to determine if the entire match block has a return. This code block was assumed to always return. match value: "test": print("test") _: return Now as soon as one of the branches has no return, the entire match block is marked to not have a return.
* | | Fix duplicated lines in GDScript bytecodeBojidar Marinov2019-03-141-1/+3
| | | | | | | | | | | | Fixes #26789
* | | Fix enums coming from other classes without preloadBojidar Marinov2019-03-051-0/+12
| | | | | | | | | | | | Fix #19704, fix #26001
* | | Revert "Forbid implicit type conversion in GDScript"Rémi Verschelde2019-03-041-10/+95
| | |
* | | Merge pull request #26562 from vnen/gdscript-no-implicit-castRémi Verschelde2019-03-041-95/+10
|\ \ \ | | | | | | | | Forbid implicit type conversion in GDScript
| * | | GDScript: Forbid implicit type conversionGeorge Marques2019-03-031-95/+10
| | | | | | | | | | | | | | | | | | | | Since types are not present in release builds, this could cause issues where a variable does not have the exact defined type.
* | | | GDScript: Fix issue when detecting file class in inner classGeorge Marques2019-03-031-1/+1
| | | |
* | | | GDScript: Allow `for` iterator to be rededefinedGeorge Marques2019-03-031-1/+0
|/ / /
* | | Merge pull request #26547 from vnen/gdscript-dependency-parseJuan Linietsky2019-03-031-28/+46
|\ \ \ | | | | | | | | Add a parse mode for GDScript which doesn't load dependencies
| * | | Add a dependency search mode for GDScript parserGeorge Marques2019-03-031-28/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - This mode avoids loading any other resource. - Search for class_name now uses this mode, to avoid loading in the scan thread. - Implement get_dependencies() for GDScript loader, now exporting dependencies only should include the preloaded resources.
* | | | Fix GDScript checking for assigning to a constant only in releaseBojidar Marinov2019-03-031-7/+10
|/ / /
* | | Fix GDScript exports having the wrong type of default value by converting itBojidar Marinov2019-02-261-0/+10
| | | | | | | | | | | | | | | Also, initialize elements of PoolArrays when resizing them in the editor. Fixes #26066.
* | | Merge pull request #25018 from AllanDaemon/#24895George Marques2019-02-241-2/+3
|\ \ \ | | | | | | | | Fix support for optional parameters in setters
| * | | Fix #24895 (support for optional parameters in setters)Allan Daemon2019-01-231-2/+3
| | | |
* | | | Request to use load when cyclic reference is found, closes #26119Juan Linietsky2019-02-211-1/+1
| | | |
* | | | Add -Wshadow=local to warnings and fix reported issues.marxin2019-02-201-32/+32
| |/ / |/| | | | | | | | Fixes #25316.
* | | Fix typos with codespellRémi Verschelde2019-02-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using codespell 1.14.0. Method: ``` $ cat > ../godot-word-whitelist.txt << EOF ang doubleclick lod nd numer que te unselect EOF $ codespell -w -q 3 -I ../godot-word-whitelist.txt --skip="./thirdparty,*.po" $ git diff // undo unwanted changes ```
* | | Merge pull request #25481 from hpvb/fix-ubsan-asan-reportsRémi Verschelde2019-02-121-0/+4
|\ \ \ | | | | | | | | Fix many asan and ubsan reported issues
| * | | Fix many asan and ubsan reported issuesHein-Pieter van Braam2019-01-301-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows most demos to run without any ubsan or asan errors. There are still some things in thirdpart/ and some things in AudioServer that needs a look but this fixes a lot of issues. This should help debug less obvious issues, hopefully. This fixes #25217 and fixes #25218
* | | | Merge pull request #25550 from DualMatrix/fix-25357Rémi Verschelde2019-02-121-0/+19
|\ \ \ \ | | | | | | | | | | Fixed Null appearing inside export variables with type hints and no default value
| * | | | Fixed Null appearing inside export variables with type hints and no default ↵DualMatrix2019-02-011-0/+19
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | value The default value of the type is now used to initialise it. export(int) A Will now have A be 0 istead of Null even though it still showed as 0 before in the inspector, fixes #25357
* | | | Merge pull request #25650 from willnationsdev/script-iconRémi Verschelde2019-02-091-8/+12
|\ \ \ \ | |/ / / |/| | | Fix script class icons looking for paths at runtime
| * | | Fix script class icon filepath lookups at runtime.Will Nations2019-02-061-8/+12
| |/ /
* | | GDScript: do second pass of parsing on releaseGeorge Marques2019-01-231-6/+9
| | | | | | | | | | | | | | | | | | Some construct (like match) actually depends on the second pass. This adds some extra checks to not perform specific type-checks on release since not all type information is available.
* | | GDScript: fix default value for autoexported typed varsGeorge Marques2019-01-231-1/+1
| | |
* | | GDScript: read constants from parent scriptsGeorge Marques2019-01-231-0/+15
| | | | | | | | | | | | | | | This is needed to create export variables from enums defined in a parent class.
* | | GDScript: allow local classes to be used as typesGeorge Marques2019-01-231-0/+6
|/ /
* | Merge pull request #25069 from vnen/gdscript-fixesHein-Pieter van Braam2019-01-171-26/+16
|\ \ | | | | | | A bit more of GDScript fixes
| * | GDScript: clarify error message about cyclesGeorge Marques2019-01-161-3/+3
| | | | | | | | | | | | They may happen with any cyclic dependency, not only with inheritance.
| * | GDScript: fix type-check of indexed valuesGeorge Marques2019-01-161-23/+13
| | |
* | | Merge pull request #25001 from jlahman/gdscript-export-var-fixRémi Verschelde2019-01-161-0/+4
|\ \ \ | |/ / |/| | Fixes export PackedScene "reset to default" throwing errors
| * | Fixes export PackedScene "reset to default" throwing errorsjlahman2019-01-151-0/+4
| |/ | | | | | | | | | | When exporting variables from a gdscript, default values of uninitialized variables would never be set. This caused the default value to be Variant::NIL, and when a user tried to reset the variable through the editor, an error would be thrown because too few arguments would be counted(end of argument list for calls are detected by NIL values). Fixed by simply setting default value to an empty variant of the proper type in gdscript parser.
* | GDScript: consider constructors as always existingGeorge Marques2019-01-151-8/+7
| | | | | | | | | | | | | | | | There's always a constructor, even if implicit, especially for native types. Also don't check for signature match on function call, since this information is not available in release builds.
* | GDScript: allow objects to be keys of dictionariesGeorge Marques2019-01-151-1/+1
| | | | | | | | | | The engine allows this already, so the parser should not fail in this case.